Summary: | [chromium] Check and rebaseline SVG test expectations for Chromium | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Yuzo Fujishima <yuzo> | ||||||
Component: | SVG | Assignee: | Jeremy Orlow <jorlow> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, commit-queue, dglazkov, eric, jorlow, tony, yaar, yuzo, zimmermann | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | 37986 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Yuzo Fujishima
2010-04-26 00:26:29 PDT
Committed r58239: <http://trac.webkit.org/changeset/58239> Just changed test expectation. The issue remains. Created attachment 54265 [details]
Patch
Committed r58241: <http://trac.webkit.org/changeset/58241> Comment on attachment 54265 [details]
Patch
Cleaned up some duplicate expectations.
http://trac.webkit.org/changeset/58244 may be related to this bug as well. Are you guys planning to submit new baselines for these? It's been a few days now. I don't get it....were you planning on doing the rebaslining Yuzo? If not, you either shouldn't have rolled the stuff in or (at very least) assigned a bug to someone to do it. Anyway, I'll take this on. Tomorrow that is. Sorry about this. It was too big a regression for me to handle myself, and I don't know how to find a good assignee. (And I assumed that filing a bug is enough, which seems to be wrong.) What is the good way to find a relevant (Chromium) developer? (In reply to comment #11) > Sorry about this. > It was too big a regression for me to handle myself, > and I don't know how to find a good assignee. > (And I assumed that filing a bug is enough, which seems to be wrong.) Filing a chrome bug used to just work out because Dimitri triaged them all, but most webkit bugs just get lost in the void. > What is the good way to find a relevant (Chromium) developer? I assume you were the last Gardener? If so, probably what you should have done was rebaseline everything but then send the review to someone who knows SVG. svn blameing one of the SVG directories for someone from Chrome would have been a start. You could have also cc'ed someone like Dimitri and asked for help. Anyway, we caught it early, so it's not that big of a deal. (In reply to comment #11) > Sorry about this. > It was too big a regression for me to handle myself, > and I don't know how to find a good assignee. > (And I assumed that filing a bug is enough, which seems to be wrong.) I am wondering why you felt it was a difficult problem. Let me try to explain my approach to diagnosing it, hopefully it'll be useful in the future gardening stints. The dashboard (see Jeremy's link in comment 6) clearly fingers http://trac.webkit.org/log/?verbose=on&rev=58212&stop_rev=58212, which is one revision that caused all these changes. When you look at the log, this revision changes expectations for a large list of tests that matches nearly one-to-one the list of failures on the canaries. If you look at the diffs on the new expectations, checked in at r58212 and compare them with the expectation changes on the dashboard, you'll see that they add the same type of RenderTree output: http://trac.webkit.org/changeset/58212/trunk/LayoutTests/platform/mac/svg/W3C-SVG-1.1/animate-elem-31-t-expected.txt http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showLargeExpectations=true&tests=svg%2FW3C-SVG-1.1%2Fanimate-elem-31-t.svg&showExpectations=true&useWebKitCanary=true Then we read up on the type of the change in the excellent ChangeLog that Nikolas provided and conclude that yes, all these are valid adjustments. You can even cc him (I just did) on the bug or ask on #webkit (he's WildFox) whether that's correct. The tedious task of determining that is, well, tedious, but shouldn't take more than 30 mins. The rest is easy -- just run rebaselining tool and commit the diffs. Does this make any sense? svg-as-background-2.html might be a real failure http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fbackgrounds%2Fsvg-as-background-2.html&showExpectations=true&useWebKitCanary=true (In reply to comment #14) > svg-as-background-2.html might be a real failure > > http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fbackgrounds%2Fsvg-as-background-2.html&showExpectations=true&useWebKitCanary=true Yes indeed! It fails for WebKit/mac too. Actually, https://bugs.webkit.org/show_bug.cgi?id=38108 is already open for addressing fallout from that change. Let's continue discussion about svg-as-background-2.html there. Created attachment 54570 [details]
Rebaseline SVG layout tests
Comment on attachment 54570 [details]
Rebaseline SVG layout tests
r=me (reviewed over shoulder)
Comment on attachment 54570 [details] Rebaseline SVG layout tests Rejecting patch 54570 from commit-queue. Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Jeremy Orlow', u'--force']" exit_code: 1 Last 500 characters of output: ests/platform/chromium-win/svg/transforms/text-with-pattern-inside-transformed-html-expected.txt patching file LayoutTests/platform/chromium-win/svg/transforms/text-with-pattern-with-svg-transform-expected.txt patching file LayoutTests/platform/chromium-win/traversal/node-iterator-prototype-expected.txt patching file LayoutTests/platform/chromium/test_expectations.txt Hunk #1 FAILED at 2799. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/chromium/test_expectations.txt.rej Full output: http://webkit-commit-queue.appspot.com/results/1906076 There are missing expectations: https://bugs.webkit.org/show_bug.cgi?id=38360 Unsure as to the status of this bug. Was it eventually landed? Changed component to SVG, so it shows up in my all-svg-bugs search. I'm assuming this was fixed since it's super old. |