RESOLVED FIXED 60721
Marker test from ietestcenter fails
https://bugs.webkit.org/show_bug.cgi?id=60721
Summary Marker test from ietestcenter fails
Rob Buis
Reported 2011-05-12 13:25:01 PDT
See the test url, this test also is part of SVG1.12nd edition testsuite.
Attachments
Patch (23.07 KB, patch)
2011-05-12 13:46 PDT, Rob Buis
no flags
Archive of layout-test-results from ec2-cr-linux-02 (1.24 MB, application/zip)
2011-05-12 14:10 PDT, WebKit Review Bot
no flags
Patch (23.49 KB, patch)
2011-05-12 14:28 PDT, Rob Buis
no flags
Patch (55.38 KB, patch)
2011-05-12 14:56 PDT, Rob Buis
no flags
Patch (41.27 KB, patch)
2011-05-12 15:13 PDT, Rob Buis
zimmermann: review+
webkit.review.bot: commit-queue-
Archive of layout-test-results from ec2-cr-linux-01 (1.24 MB, application/zip)
2011-05-12 15:54 PDT, WebKit Review Bot
no flags
Rob Buis
Comment 1 2011-05-12 13:46:47 PDT
Geoffrey Garen
Comment 2 2011-05-12 13:50:33 PDT
Your ChangeLog is missing an explanation of why this change is correct.
Dirk Schulze
Comment 3 2011-05-12 13:57:50 PDT
Comment on attachment 93330 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=93330&action=review > Source/WebCore/svg/SVGMarkerElement.cpp:251 > + return style->display() != NONE || (parentNode() && parentNode()->renderer() && parentNode()->renderer()->isSVGHiddenContainer()); What about visibility?
WebKit Review Bot
Comment 4 2011-05-12 14:10:10 PDT
Comment on attachment 93330 [details] Patch Attachment 93330 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/8684889 New failing tests: svg/W3C-SVG-1.1-SE/painting-marker-07-f.svg
WebKit Review Bot
Comment 5 2011-05-12 14:10:16 PDT
Created attachment 93336 [details] Archive of layout-test-results from ec2-cr-linux-02 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-02 Port: Chromium Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Rob Buis
Comment 6 2011-05-12 14:28:03 PDT
Dirk Schulze
Comment 7 2011-05-12 14:37:03 PDT
Comment on attachment 93344 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=93344&action=review I'd like to see at least one more test, where you use inherit for display <marker style="display:inherit" ... r- because of the new test and the changelog. > Source/WebCore/ChangeLog:9 > + > + Allow <marker> to be referenced in case the display:none is set on it > + or any of the ancestors. You're doing more now. Marker gets a Renderer, always. Also not sure what you mean with referenced in the combination with display:none. Please add a more detailed description what you changed, why you did it, Why it fixes the problem. Add a link to the specification text.
Rob Buis
Comment 8 2011-05-12 14:56:58 PDT
Rob Buis
Comment 9 2011-05-12 15:13:55 PDT
WebKit Review Bot
Comment 10 2011-05-12 15:54:26 PDT
Comment on attachment 93353 [details] Patch Attachment 93353 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/8689847 New failing tests: svg/W3C-SVG-1.1-SE/types-dom-05-b.svg svg/W3C-SVG-1.1-SE/painting-marker-07-f.svg
WebKit Review Bot
Comment 11 2011-05-12 15:54:31 PDT
Created attachment 93361 [details] Archive of layout-test-results from ec2-cr-linux-01 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-01 Port: Chromium Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Nikolas Zimmermann
Comment 12 2011-05-18 07:26:47 PDT
Comment on attachment 93353 [details] Patch Looks great, r=me.
Rob Buis
Comment 13 2011-05-18 14:40:36 PDT
Ryosuke Niwa
Comment 14 2011-05-18 18:55:21 PDT
Rob Buis
Comment 16 2011-05-19 07:46:59 PDT
Hi Ryosuke, (In reply to comment #14) > It seems like we need a Windows rebaseline: > http://build.webkit.org/builders/Windows%20XP%20Debug%20%28Tests%29/builds/28783 Agreed! I want to make clear as well that the new result is an improvement. Cheers, Rob.
Note You need to log in before you can comment on or make changes to this bug.