<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>41079</bug_id>
          
          <creation_ts>2010-06-23 10:51:48 -0700</creation_ts>
          <short_desc>[Qt] Qt&apos;s DumpRenderTree formats SVG rectangles differently from other ports for no good reason</short_desc>
          <delta_ts>2011-04-18 10:39:23 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>SVG</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Qt, QtTriaged</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="James Robinson">jamesr</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>benjamin</cc>
    
    <cc>eric</cc>
    
    <cc>kling</cc>
    
    <cc>krit</cc>
    
    <cc>ossy</cc>
    
    <cc>zimmermann</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>241932</commentid>
    <comment_count>0</comment_count>
    <who name="James Robinson">jamesr</who>
    <bug_when>2010-06-23 10:51:48 -0700</bug_when>
    <thetext>Diff from a recent patch:

--- /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/fast/repaint/svg-layout-root-style-attr-update-expected.txt	2010-06-23 10:01:08.455453400 -0700
+++ /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/fast/repaint/svg-layout-root-style-attr-update-actual.txt	2010-06-23 10:01:08.455453400 -0700
@@ -11,5 +11,5 @@
   RenderBlock (positioned) {DIV} at (0,0) size 500x500
 layer at (8,0) size 500x500
   RenderSVGRoot {svg} at (8,0) size 500x500
-    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#FF0000]}] [data=&quot;M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 Z&quot;]
-    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#008000]}] [data=&quot;M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 Z&quot;]
+    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#FF0000]}] [data=&quot;M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 L250.00,10.00&quot;]
+    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#008000]}] [data=&quot;M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 L250.00,10.00&quot;]

See http://trac.webkit.org/changeset/61696.  Every other port formats the rectangle the same way except Qt.  From a spot check of LayoutTests/platform/qt this appears to be causing a lot of diffs.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241995</commentid>
    <comment_count>1</comment_count>
    <who name="Dirk Schulze">krit</who>
    <bug_when>2010-06-23 11:48:17 -0700</bug_when>
    <thetext>(In reply to comment #0)
&gt; Diff from a recent patch:
&gt; 
&gt; --- /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/fast/repaint/svg-layout-root-style-attr-update-expected.txt    2010-06-23 10:01:08.455453400 -0700
&gt; +++ /home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/layout-test-results/fast/repaint/svg-layout-root-style-attr-update-actual.txt    2010-06-23 10:01:08.455453400 -0700
&gt; @@ -11,5 +11,5 @@
&gt;    RenderBlock (positioned) {DIV} at (0,0) size 500x500
&gt;  layer at (8,0) size 500x500
&gt;    RenderSVGRoot {svg} at (8,0) size 500x500
&gt; -    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#FF0000]}] [data=&quot;M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 Z&quot;]
&gt; -    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#008000]}] [data=&quot;M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 Z&quot;]
&gt; +    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#FF0000]}] [data=&quot;M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 L250.00,10.00&quot;]
&gt; +    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#008000]}] [data=&quot;M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 L250.00,10.00&quot;]
&gt; 
&gt; See http://trac.webkit.org/changeset/61696.  Every other port formats the rectangle the same way except Qt.  From a spot check of LayoutTests/platform/qt this appears to be causing a lot of diffs.

This is caused by Qt itself. Qt closes paths automatically if a path crosses itself. Thats why DRT results will never match the results of other ports. We already have bug reports about the different results, the Qt guys told me, that they won&apos;t change this behavior. So I think we can close this bug, or if this test is broken, upload a new baseline for Qt.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>241998</commentid>
    <comment_count>2</comment_count>
    <who name="James Robinson">jamesr</who>
    <bug_when>2010-06-23 11:50:51 -0700</bug_when>
    <thetext>I&apos;ve added a new baseline.  That&apos;s quite unfortunate, are you sure you can&apos;t normalize this in Qt&apos;s DRT?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>249006</commentid>
    <comment_count>3</comment_count>
    <who name="Nikolas Zimmermann">zimmermann</who>
    <bug_when>2010-07-09 07:28:51 -0700</bug_when>
    <thetext>Changed component to SVG, so it shows up in my all-svg-bugs search.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>387847</commentid>
    <comment_count>4</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2011-04-18 10:39:23 -0700</bug_when>
    <thetext>I fixed this with bug 46051 :)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>