Bug 77017 - [Chromium]: rebaseline after r105878
Summary: [Chromium]: rebaseline after r105878
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Stephen Chenney
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-25 08:52 PST by Stephen Chenney
Modified: 2012-01-26 11:20 PST (History)
4 users (show)

See Also:


Attachments
Patch (1.92 KB, patch)
2012-01-25 09:25 PST, Stephen Chenney
no flags Details | Formatted Diff | Diff
Patch (51.88 KB, patch)
2012-01-25 13:01 PST, Stephen Chenney
no flags Details | Formatted Diff | Diff
Patch (33.99 KB, patch)
2012-01-26 09:54 PST, Stephen Chenney
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Stephen Chenney 2012-01-25 08:52:12 PST
Test expectations after r105878 are not right. This bug tracks rebaselining.
Comment 1 Stephen Chenney 2012-01-25 09:07:19 PST
This is more complicated than I thought. Windows and Mac results are wrong for one test. Investigating further.
Comment 2 anton muhin 2012-01-25 09:11:21 PST
maybe suppress tests for time being?
Comment 3 Stephen Chenney 2012-01-25 09:25:57 PST
Created attachment 123957 [details]
Patch
Comment 4 Levi Weintraub 2012-01-25 09:44:01 PST
Comment on attachment 123957 [details]
Patch

Works for now.
Comment 5 anton muhin 2012-01-25 10:31:27 PST
Comment on attachment 123957 [details]
Patch

Clearing flags on attachment: 123957

Committed r105895: <http://trac.webkit.org/changeset/105895>
Comment 6 anton muhin 2012-01-25 10:31:36 PST
All reviewed patches have been landed.  Closing bug.
Comment 7 anton muhin 2012-01-25 10:32:37 PST
Suppressions committed as http://trac.webkit.org/changeset/105895.  Reopening for proper rebaseline
Comment 8 Stephen Chenney 2012-01-25 13:01:12 PST
Created attachment 123992 [details]
Patch
Comment 9 Levi Weintraub 2012-01-25 13:04:05 PST
Comment on attachment 123992 [details]
Patch

Let's see how this goes.
Comment 10 Levi Weintraub 2012-01-25 13:09:13 PST
Comment on attachment 123992 [details]
Patch

Clearing flags on attachment: 123992

Committed r105921: <http://trac.webkit.org/changeset/105921>
Comment 11 Levi Weintraub 2012-01-25 13:09:18 PST
All reviewed patches have been landed.  Closing bug.
Comment 12 Stephen Chenney 2012-01-26 09:52:18 PST
Expectations were broken again, and the lines in test_expectations also needed to be removed.
Comment 13 Stephen Chenney 2012-01-26 09:54:41 PST
Created attachment 124132 [details]
Patch
Comment 14 Levi Weintraub 2012-01-26 09:57:15 PST
(In reply to comment #13)
> Created an attachment (id=124132) [details]
> Patch

Why only text expectations this time?
Comment 15 WebKit Review Bot 2012-01-26 11:20:43 PST
Comment on attachment 124132 [details]
Patch

Clearing flags on attachment: 124132

Committed r106017: <http://trac.webkit.org/changeset/106017>
Comment 16 WebKit Review Bot 2012-01-26 11:20:49 PST
All reviewed patches have been landed.  Closing bug.