Bug 41079 - [Qt] Qt's DumpRenderTree formats SVG rectangles differently from other ports for no good reason
Summary: [Qt] Qt's DumpRenderTree formats SVG rectangles differently from other ports ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks:
 
Reported: 2010-06-23 10:51 PDT by James Robinson
Modified: 2011-04-18 10:39 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description James Robinson 2010-06-23 10:51:48 PDT
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="M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 Z"]
-    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#008000]}] [data="M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 Z"]
+    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#FF0000]}] [data="M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 L250.00,10.00"]
+    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#008000]}] [data="M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 L250.00,10.00"]

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.
Comment 1 Dirk Schulze 2010-06-23 11:48:17 PDT
(In reply to comment #0)
> 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="M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 Z"]
> -    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#008000]}] [data="M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 Z"]
> +    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#FF0000]}] [data="M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 L250.00,10.00"]
> +    RenderPath {rect} at (258,10) size 250x250 [fill={[type=SOLID] [color=#008000]}] [data="M250.00,10.00 L500.00,10.00 L500.00,260.00 L250.00,260.00 L250.00,10.00"]
> 
> 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't change this behavior. So I think we can close this bug, or if this test is broken, upload a new baseline for Qt.
Comment 2 James Robinson 2010-06-23 11:50:51 PDT
I've added a new baseline.  That's quite unfortunate, are you sure you can't normalize this in Qt's DRT?
Comment 3 Nikolas Zimmermann 2010-07-09 07:28:51 PDT
Changed component to SVG, so it shows up in my all-svg-bugs search.
Comment 4 Andreas Kling 2011-04-18 10:39:23 PDT
I fixed this with bug 46051 :)