Bug 77178 - Update mac-lion canvas results
Summary: Update mac-lion canvas results
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Matthew Delaney
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-26 19:54 PST by Matthew Delaney
Modified: 2012-01-27 23:17 PST (History)
3 users (show)

See Also:

Attachments
Patch (7.27 KB, patch)
2012-01-26 20:05 PST, Matthew Delaney 		no flags Details | Formatted Diff | Diff
Patch (11.67 KB, patch)
2012-01-27 22:49 PST, Matthew Delaney 		fpizlo: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Delaney 2012-01-26 19:54:14 PST 
The mac-lion Skipped list needs to be updated for canvas tests.
Comment 1 Matthew Delaney 2012-01-26 20:05:42 PST 
Created attachment 124247 [details]
Patch
Comment 2 Andy Estes 2012-01-27 17:28:16 PST 
As bdash pointed out on IRC, the results you added in platform/mac-lion/ should be moved to platform/mac/. If platform/mac/ has expected results already that apply to Snow Leopard, they should be moved to platform/mac-snowleopard/. r=me if you do this.
Comment 3 Matthew Delaney 2012-01-27 20:09:10 PST 
Committed r106179: <http://trac.webkit.org/changeset/106179>
Comment 4 Matthew Delaney 2012-01-27 20:26:32 PST 
Landed with Andy's suggested fixes. Though, I had to leave that one test in mac-lion since those results are truly lion-only.
Comment 5 Matthew Delaney 2012-01-27 22:37:37 PST 
Reopening to update more of the tests' expected outputs.
Comment 6 Matthew Delaney 2012-01-27 22:49:23 PST 
Created attachment 124429 [details]
Patch
Comment 7 Filip Pizlo 2012-01-27 23:06:12 PST 
Comment on attachment 124429 [details]
Patch

rs=me
Comment 8 Matthew Delaney 2012-01-27 23:17:34 PST 
Committed r106184: <http://trac.webkit.org/changeset/106184>