[webkit-changes] [91915] trunk

[rwlbuis]

Title: [91915] trunk

Revision

91915

Author

rwlb...@webkit.org

Date

2011-07-28 04:42:29 -0700 (Thu, 28 Jul 2011)

Log Message

2011-07-28  Rob Buis  <rb...@rim.com>

        REGRESSION (r91125): Google Drawings is broken
        https://bugs.webkit.org/show_bug.cgi?id=65257

        Reviewed by Darin Adler.

        Only consider zero-length path rendering code path when a stroke is defined.

        Test: svg/custom/zero-path-square-cap-rendering2.svg

        * rendering/svg/RenderSVGPath.cpp:
        (WebCore::RenderSVGPath::shouldStrokeZeroLengthSubpath):

Modified Paths

• trunk/LayoutTests/ChangeLog
• trunk/Source/WebCore/ChangeLog
• trunk/Source/WebCore/rendering/svg/RenderSVGPath.cpp

Added Paths

• trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.png
• trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.txt
• trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2.svg

Diff

Modified: trunk/LayoutTests/ChangeLog (91914 => 91915)

--- trunk/LayoutTests/ChangeLog	2011-07-28 08:13:23 UTC (rev 91914)
+++ trunk/LayoutTests/ChangeLog	2011-07-28 11:42:29 UTC (rev 91915)
@@ -1,3 +1,14 @@
+2011-07-28  Rob Buis  <rb...@rim.com>
+
+        REGRESSION (r91125): Google Drawings is broken
+        https://bugs.webkit.org/show_bug.cgi?id=65257
+
+        Reviewed by Darin Adler.
+
+        * svg/custom/zero-path-square-cap-rendering2-expected.png: Added.
+        * svg/custom/zero-path-square-cap-rendering2-expected.txt: Added.
+        * svg/custom/zero-path-square-cap-rendering2.svg: Added.
+
 2011-07-22  Martin Robinson  <mrobin...@igalia.com>
 
         [Cairo] Switch from ContextShadow to ShadowBlur

Added: trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.png (0 => 91915)

--- trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.png	                        (rev 0)
+++ trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.png	2011-07-28 11:42:29 UTC (rev 91915)
@@ -0,0 +1,228 @@
+\x89PNG
+
+
+IHDR X\x9Av\x82p)tEXtchecksum10c7ee91dd13ee0179fbe34da613f37c\xBB\xF0ݳ\xEEiCCPICC Profilex
\x85T\xCFkA\xFE6n\xA9\xD0"Zk\xB2x\x90"IY\xABhE\xD46\xFDbk\xDB\xB6E\x90d3I\xD6n6\xEB\xEE&\xB5\xA5\x88\xE4\xE2\xD1*\xDEE\xED\xA1\xFF\x80z\xF0d/J\x85ZE(ޫ(b\xA1-\xF1\xCDnL\xB6\xA5\xEA\xC0\xCE~\xF3\xDE7\xEF}ov\xDF
+r\xD24\xF5\x80\xE4
+\xC7R\xA2il|Bj\xFC\x88\x8E\xA2	A4%U\xDB\xECN$A\x83s\xF9{\xE7\xD8z\x81[V\xC3{\xFBw\xB2w\xAD\x9AҶ\x9A\x84\xFD@\xE0G\x9A\xD9*\xB0\xEFq
+Y\x88<ߡ)\xC7t\xDF\xE3\xD8\xF2\xEC\x8F9Nyx\xC1\xB5+=\xC4Y"|@5-\xCEM\xB8S\xCD%\xD3@\x83H8\x94\xF5qR>\x9C׋\x94\xD7inf\xC6\xC8\xBDO\x90\xA6\xBB\xCC\xEE\xABb\xA1\x9CN\xF6\x90\xBD\x9D\xF4~N\xB3\xDE>\xC2!\xC2\xAD?F\xB8\x8D\x9E\xF5\x8C\xD5?\xE2a\xE1\xA4\xE6Ć=5\xF4\xF8`\xB7\xA9\xF85\xC2_M'\xA2Tq\xD9.
+\xF1\x98\xAE\xFDV\xF2J\x82p\x908\xCAda\x80sZHO\xD7Ln\xF8\xBA\x87}&\xD7\xAF\xE2wVQ\xE1y\x8Fg\xDE\xD4\xDD\xEFE\xDA\xAF0\x9A
+HPEa\x98\xB0P@\x86<14\xB2r?#\xAB\x93{2u$j\xBBtbD\xB1A{6\xDC=\xB7Q\xA4\xDD<\xFE("q\x94C\xB5\x92\xFCA\xFE*\xAF\xC9O\xE5y\xF9\xCB\\xB0\xD8V\x{1D4B5B}\x9A\xBA\xF2\xE0;\xC5噹\xD7\xD3\xC8\xE3sM^|\x95\xD4v\x93WG\x96\xACyz\xBC\x9A\xEC?\xECW\x971\xE6\x825\x8F\xC4s\xB0\xFB\xF1-_\x95̗)\x8C\xC5\xE3U\xF3\xEAK\x84uZ17ߟl;=\xE2.\xCF.\xB5\xD6s\xAD\x89\x8B7V\x9B\x97g\xFDjH\xFB\x93\xE6U\xF9O^\xF5\xF1\xFCg\xCD\xC4c\xE2)1&v\x8A\xE7!\x89\x97\xC5.\xF1\x92\xD8K\xAB\xE2`m\xC7\x9D\x95\x86)\xD2m\x91\xFA$\xD5``\x9A\xBC\xF5/]?[x\xBDF\xF5Q\x94\xCC\xD2T\x89\xF7\xC2*d4\x9D\xB9o\xFA\x8F\xDB\xC7\xFC\xE4\xF7\x8A\xE7\x9F(/l\xE0ș\xBAmSq\xEF\xA1e\xA5ns\xAE\xBF\xD1}\xF0\xB6nk\xA3~8\xFCX<\xAB\xADR5\x9F \xBC
v\x87z\xE8)\x98Ӗ\x96\xCD9R\x87,\x9F\x93\xBA\xE9\xCAbR\xCCP\xDBCRR\xD7%\xD7eK\xB3\x99Ub\xE9vؙ\xD3n\xA19B\xF7ħJe\x93\xFA\xAF\xF1\xB0\xFD\xB0R\xF9\xF9\xACR\xD9~Nց\x97\xFAo\xC0\xBC\xFDE\xC0x\x8B\x89 IDATx
\xEDڱ\xADXU\xD0o0"G\xA8\x92\x9C\xC41D\x9E\xCB̞\xD7\xC0\xC9H\xEFӷ\xBF
 @\x80
?fA\x80
 @\x80\x81\xFF\x81
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xA0\x80Ĩ
+"@\x80
 @@
\xF1 @\x80
\x88	( 1j\x83 @\x80
P@\xBC
 @\x80
b
+H\x8C\xDA  @\x80
o\x80
 @\x80\x81\x98\x80\xA36\x88
 @\x80
\xC4 @\x80
 @ &\xF0\xF9\xAF\xBF\xFF\x8A
+3\xA8C\xE0\xF3\x9F?\xBE\xFC\xF2\xA5cY[ @\x80
T	|\xFA\xF8\xE3\xE3[\xD5Ɩ\xFD\xEE\xBF\xFE\xFC\xEB\xC7\xD7߿~\xF79 @\x80
\xDC\xF0ֽ\xCC]L\x80
 @\xE0\x99\x80\xF2\x8C\xDE` @\x80
\xF7\x90{\x99\xBB\x98
 @\x80\xC03
\xE4\xBD\xC1 @\x80
\xEE	( \xF72w1
 @\x80\x81g
+\xC83z\x83	 @\x80
\xDCP@\xEEe\xEEb @\x80
\xCF\x90g\xF4 @\x80
\xB8'\xA0\x80\xDC\xCB\xDC\xC5 @\x80
\x9E	( \xCF\xE8
+&@\x80
 pO@
\xB9\x97\xB9\x8B	 @\x80
<P@\x9E\xD1L\x80
 @\xE0\x9E\x80r/s @\x80
x&\xA0\x80<\xA37\x98
 @\x80\xC0=
\xE4^\xE6.&@\x80
 \xF0L@
yFo0
 @\x80\x81{
+Ƚ\xCC]L\x80
 @\xE0\x99\x80\xF2\x8C\xDE` @\x80
\xF7\x90{\x99\xBB\x98
 @\x80\xC03
\xE4\xBD\xC1 @\x80
\xEE	( \xF72w1
 @\x80\x81g
+\xC83z\x83	 @\x80
\xDCP@\xEEe\xEEb @\x80
\xCF\x90g\xF4 @\x80
\xB8'\xA0\x80\xDC\xCB\xDC\xC5 @\x80
\x9E	( \xCF\xE8
+&@\x80
 pO@
\xB9\x97\xB9\x8B	 @\x80
<P@\x9E\xD1L\x80
 @\xE0\x9E\x80r/s @\x80
x&\xA0\x80<\xA37\x98
 @\x80\xC0=
\xE4^\xE6.&@\x80
 \xF0L@
yFo0
 @\x80\x81{
+Ƚ\xCC]L\x80
 @\xE0\x99\x80\xF2\x8C\xDE` @\x80
\xF7\x90{\x99\xBB\x98
 @\x80\xC03
\xE4\xBD\xC1 @\x80
\xEE	( \xF72w1
 @\x80\x81g
+\xC83z\x83	 @\x80
\xDCP@\xEEe\xEEb @\x80
\xCF\x90g\xF4 @\x80
\xB8'\xA0\x80\xDC\xCB\xDC\xC5 @\x80
\x9E	( \xCF\xE8
+&@\x80
 pO@
\xB9\x97\xB9\x8B	 @\x80
<P@\x9E\xD1L\x80
 @\xE0\x9E\x80r/s @\x80
x&\xA0\x80<\xA37\x98
 @\x80\xC0=
\xE4^\xE6.&@\x80
 \xF0L@
yFo0
 @\x80\x81{
+Ƚ\xCC]L\x80
 @\xE0\x99\x80\xF2\x8C\xDE` @\x80
\xF7\x90{\x99\xBB\x98
 @\x80\xC03
\xE4\xBD\xC1 @\x80
\xEE	( \xF72w1
 @\x80\x81g
+\xC83z\x83	 @\x80
\xDCP@\xEEe\xEEb @\x80
\xCF\x90g\xF4 @\x80
\xB8'\xA0\x80\xDC\xCB\xDC\xC5 @\x80
\x9E	( \xCF\xE8
+&@\x80
 pO@
\xB9\x97\xB9\x8B	 @\x80
<P@\x9E\xD1L\x80
 @\xE0\x9E\x80r/s @\x80
x&\xA0\x80<\xA37\x98
 @\x80\xC0=
\xE4^\xE6.&@\x80
 \xF0L@
yFo0
 @\x80\x81{
+Ƚ\xCC]L\x80
 @\xE0\x99\x80\xF2\x8C\xDE` @\x80
\xF7\x90{\x99\xBB\x98
 @\x80\xC03
\xE4\xBD\xC1 @\x80
\xEE	( \xF72w1
 @\x80\x81g
+\xC83z\x83	 @\x80
\xDCP@\xEEe\xEEb @\x80
\xCF\x90g\xF4 @\x80
\xB8'\xA0\x80\xDC\xCB\xDC\xC5 @\x80
\x9E	( \xCF\xE8
+&@\x80
 pO@
\xB9\x97\xB9\x8B	 @\x80
<P@\x9E\xD1L\x80
 @\xE0\x9E\x80r/s @\x80
x&\xA0\x80<\xA37\x98
 @\x80\xC0=
\xE4^\xE6.&@\x80
 \xF0L@
yFo0
 @\x80\x81{
+Ƚ\xCC]L\x80
 @\xE0\x99\x80\xF2\x8C\xDE` @\x80
\xF7\x90{\x99\xBB\x98
 @\x80\xC03
\xE4\xBD\xC1 @\x80
\xEE	( \xF72w1
 @\x80\x81g
+\xC83z\x83	 @\x80
\xDCP@\xEEe\xEEb @\x80
\xCF\x90g\xF4 @\x80
\xB8'\xA0\x80\xDC\xCB\xDC\xC5 @\x80
\x9E	( \xCF\xE8
+&@\x80
 pO@
\xB9\x97\xB9\x8B	 @\x80
<P@\x9E\xD1L\x80
 @\xE0\x9E\x80r/s @\x80
x&\xA0\x80<\xA37\x98
 @\x80\xC0=
\xE4^\xE6.&@\x80
 \xF0L@
yFo0
 @\x80\x81{
+Ƚ\xCC]L\x80
 @\xE0\x99\x80\xF2\x8C\xDE` @\x80
\xF7\x90{\x99\xBB\x98
 @\x80\xC03
\xE4\xBD\xC1 @\x80
\xEE	( \xF72w1
 @\x80\x81g
+\xC83z\x83	 @\x80
\xDCP@\xEEe\xEEb @\x80
\xCF\x90g\xF4 @\x80
\xB8'\xA0\x80\xDC\xCB\xDC\xC5 @\x80
\x9E	( \xCF\xE8
+&@\x80
 pO@
\xB9\x97\xB9\x8B	 @\x80
<P@\x9E\xD1L\x80
 @\xE0\x9E\x80r/s @\x80
x&\xA0\x80<\xA37\x98
 @\x80\xC0=
\xE4^\xE6.&@\x80
 \xF0L@
yFo0
 @\x80\x81{
+Ƚ\xCC]L\x80
 @\xE0\x99\x80\xF2\x8C\xDE` @\x80
\xF7\x90{\x99\xBB\x98
 @\x80\xC03
\xE4\xBD\xC1 @\x80
\xEE	( \xF72w1
 @\x80\x81g
+\xC83z\x83	 @\x80
\xDCP@\xEEe\xEEb @\x80
\xCF>\xFF\xF9۟φ\xFC\xFF\xF8\xE9ǟ\xFE\x9F\x8Bي
 @\xA0^\xE0ӷ\xBF\xFA+@\x80
 @\x80@\x85\x80_\xB0*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 \xAA_\xDA\xE7\xBAIDAT@\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8P@*b\xB2$
 @\x80\x81
+
d#GW @\x80
\xA8\xF8F\xDA\xC9\xEA\xC1?~IEND\xAEB`\x82
\ No newline at end of file

Added: trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.txt (0 => 91915)

--- trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.txt	                        (rev 0)
+++ trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.txt	2011-07-28 11:42:29 UTC (rev 91915)
@@ -0,0 +1,7 @@
+layer at (0,0) size 800x600
+  RenderView at (0,0) size 800x600
+layer at (0,0) size 800x600
+  RenderSVGRoot {svg} at (0,165) size 134x122
+    RenderSVGContainer {g} at (0,165) size 134x122 [transform={m=((1.09,0.00)(0.00,1.09)) t=(0.00,0.00)}]
+      RenderSVGPath {path} at (0,165) size 134x122 [fill={[type=SOLID] [color=#008000]}] [data="" 0 39403 L 103176 39403 L 103176 132091 L 0 132091 Z"]
+      RenderSVGPath {path} at (0,114) size 800x486 [fill={[type=SOLID] [color=#000000]}] [data="" 0 0"]

Added: trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2.svg (0 => 91915)

--- trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2.svg	                        (rev 0)
+++ trunk/LayoutTests/svg/custom/zero-path-square-cap-rendering2.svg	2011-07-28 11:42:29 UTC (rev 91915)
@@ -0,0 +1,6 @@
+<svg version="1.1" xmlns="http://www.w3.org/2000/svg" stroke="none" stroke-linecap="square" viewBox="0 0 672084 311277">
+  <g transform="scale(1.0902777777777777)">
+    <path fill="green" d="M 0 39403 L 103176 39403 103176 132091 0 132091 Z"></path>
+    <path d="M0 0"></path>
+  </g>
+</svg>

Modified: trunk/Source/WebCore/ChangeLog (91914 => 91915)

--- trunk/Source/WebCore/ChangeLog	2011-07-28 08:13:23 UTC (rev 91914)
+++ trunk/Source/WebCore/ChangeLog	2011-07-28 11:42:29 UTC (rev 91915)
@@ -1,3 +1,17 @@
+2011-07-28  Rob Buis  <rb...@rim.com>
+
+        REGRESSION (r91125): Google Drawings is broken
+        https://bugs.webkit.org/show_bug.cgi?id=65257
+
+        Reviewed by Darin Adler.
+
+        Only consider zero-length path rendering code path when a stroke is defined.
+
+        Test: svg/custom/zero-path-square-cap-rendering2.svg
+
+        * rendering/svg/RenderSVGPath.cpp:
+        (WebCore::RenderSVGPath::shouldStrokeZeroLengthSubpath):
+
 2011-07-22  Martin Robinson  <mrobin...@igalia.com>
 
         [Cairo] Switch from ContextShadow to ShadowBlur

Modified: trunk/Source/WebCore/rendering/svg/RenderSVGPath.cpp (91914 => 91915)

--- trunk/Source/WebCore/rendering/svg/RenderSVGPath.cpp	2011-07-28 08:13:23 UTC (rev 91914)
+++ trunk/Source/WebCore/rendering/svg/RenderSVGPath.cpp	2011-07-28 11:42:29 UTC (rev 91915)
@@ -155,7 +155,7 @@
 {
     // Spec(11.4): Any zero length subpath shall not be stroked if the ‘stroke-linecap’ property has a value of butt
     // but shall be stroked if the ‘stroke-linecap’ property has a value of round or square
-    return style()->svgStyle()->capStyle() != ButtCap && !m_fillBoundingBox.width() && !m_fillBoundingBox.height();
+    return style()->svgStyle()->hasStroke() && style()->svgStyle()->capStyle() != ButtCap && !m_fillBoundingBox.width() && !m_fillBoundingBox.height();
 }
 
 FloatRect RenderSVGPath::zeroLengthSubpathRect() const

_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes