WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Formatted Diff
Diff
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
Details
Patch
(23.49 KB, patch)
2011-05-12 14:28 PDT
,
Rob Buis
no flags
Details
Formatted Diff
Diff
Patch
(55.38 KB, patch)
2011-05-12 14:56 PDT
,
Rob Buis
no flags
Details
Formatted Diff
Diff
Patch
(41.27 KB, patch)
2011-05-12 15:13 PDT
,
Rob Buis
zimmermann
: review+
webkit.review.bot
: commit-queue-
Details
Formatted Diff
Diff
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
Details
Show Obsolete
(3)
View All
Add attachment
proposed patch, testcase, etc.
Rob Buis
Comment 1
2011-05-12 13:46:47 PDT
Created
attachment 93330
[details]
Patch
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
Created
attachment 93344
[details]
Patch
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
Created
attachment 93349
[details]
Patch
Rob Buis
Comment 9
2011-05-12 15:13:55 PDT
Created
attachment 93353
[details]
Patch
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
Committed
r86791
: <
http://trac.webkit.org/changeset/86791
>
Ryosuke Niwa
Comment 14
2011-05-18 18:55:21 PDT
It seems like we need a Windows rebaseline:
http://build.webkit.org/builders/Windows%20XP%20Debug%20%28Tests%29/builds/28783
Ryosuke Niwa
Comment 15
2011-05-18 18:55:52 PDT
Windows 7:
http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r86809%20(13005)/results.html
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.
Top of Page
Format For Printing
XML
Clone This Bug