WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
45991
[chromium] Unskip the canvas philip suite
https://bugs.webkit.org/show_bug.cgi?id=45991
Summary
[chromium] Unskip the canvas philip suite
James Robinson
Reported
2010-09-17 13:31:22 PDT
The philip canvas test suite was marked as SKIP in chromium's test_expectations.txt when it was initially checked in. We should be running these tests, there's no reason to skip them.
Attachments
Patch
(1.72 KB, patch)
2010-09-17 13:38 PDT
,
James Robinson
no flags
Details
Formatted Diff
Diff
specific expectations
(10.38 KB, patch)
2010-09-17 18:30 PDT
,
James Robinson
no flags
Details
Formatted Diff
Diff
Patch
(2.24 KB, patch)
2010-09-19 23:29 PDT
,
James Kozianski
no flags
Details
Formatted Diff
Diff
Show Obsolete
(3)
View All
Add attachment
proposed patch, testcase, etc.
James Robinson
Comment 1
2010-09-17 13:35:28 PDT
canvas/philip contains 782 tests. On my linux box 97 fail and 2 are flaky. The rest pass with the existing baselines. http/tests/canvas/philip contains 10 tests, all of which pass locally.
James Robinson
Comment 2
2010-09-17 13:38:50 PDT
Created
attachment 67942
[details]
Patch
Dimitri Glazkov (Google)
Comment 3
2010-09-17 13:39:29 PDT
Comment on
attachment 67942
[details]
Patch Weeee!
James Robinson
Comment 4
2010-09-17 13:42:36 PDT
Committed
r67744
: <
http://trac.webkit.org/changeset/67744
>
James Robinson
Comment 5
2010-09-17 18:30:11 PDT
Created
attachment 67989
[details]
specific expectations
Andreas Kling
Comment 6
2010-09-17 19:15:23 PDT
Comment on
attachment 67989
[details]
specific expectations Awesome stuff!
James Kozianski
Comment 7
2010-09-19 23:29:15 PDT
Created
attachment 68054
[details]
Patch
James Kozianski
Comment 8
2010-09-19 23:35:13 PDT
Comment on
attachment 68054
[details]
Patch This just takes the mac tests from the above patch (which seems to be out of date) as they are failing on the canaries right now.
James Robinson
Comment 9
2010-09-20 00:47:30 PDT
This is not right - I intentionally left the entire suite as = TEXT until I had a chance to land the 'specific expectations' patch. Looks like abarth messed up the expectations in
http://trac.webkit.org/changeset/67820
and the followup. For shame, Adam!
James Robinson
Comment 10
2010-09-20 00:49:31 PDT
A bunch fail on windows still. I recommend reverting the entire section specific expectations patch, which contains a list of the tests that fail on each platform as of last Friday. I won't be able to look at this for ~10 hours more.
Adam Barth
Comment 11
2010-09-20 00:50:35 PDT
> For shame, Adam!
Huh? I'm not sure what I did wrong. Maybe my attempt to help garden was misguided.
Ojan Vafai
Comment 12
2010-09-20 01:09:38 PDT
(In reply to
comment #9
)
> This is not right - I intentionally left the entire suite as = TEXT until I had a chance to land the 'specific expectations' patch. Looks like abarth messed up the expectations in
http://trac.webkit.org/changeset/67820
and the followup. For shame, Adam!
James, I'm not sure the history of this set of expectations, but someone else went ahead and took away the test suite listing. This patch just cleaned up some of the fallout on the mac bot. As best I can tell, there's only one failing test left on the webkit chromium canaries and thats canvas/philip/tests/2d.gradient.radial.touch2.html on the windows bot.
James Robinson
Comment 13
2010-09-20 11:33:27 PDT
James' patch looks fine, I'm just frustrated that you guys are having to clean up expectations after I generated expectations for each platform already. I can clean up the remaining fallout.
James Robinson
Comment 14
2010-09-20 12:40:57 PDT
Committed
r67872
: <
http://trac.webkit.org/changeset/67872
>
James Robinson
Comment 15
2010-09-20 14:16:13 PDT
Comment on
attachment 67989
[details]
specific expectations clearing flags, obsolete
James Robinson
Comment 16
2010-09-20 14:16:27 PDT
Comment on
attachment 68054
[details]
Patch Clearing flags, obsolete
James Robinson
Comment 17
2010-09-21 13:27:01 PDT
Not resolved. The remaining failures need to be triaged into separate bugs.
Justin Novosad
Comment 18
2011-03-25 08:22:44 PDT
Change set
http://trac.webkit.org/changeset/81489
resolved the following layout test failures on windows and linux, which were associated with this bug: canvas/philip/tests/2d.shadow.pattern.alpha.html canvas/philip/tests/2d.shadow.pattern.basic.html canvas/philip/tests/2d.shadow.pattern.transparent.2.html
Marcus Bulach
Comment 19
2011-06-06 06:30:45 PDT
(In reply to
comment #18
)
> Change set
http://trac.webkit.org/changeset/81489
resolved the following layout test failures on windows and linux, which were associated with this bug: > > canvas/philip/tests/2d.shadow.pattern.alpha.html > canvas/philip/tests/2d.shadow.pattern.basic.html > canvas/philip/tests/2d.shadow.pattern.transparent.2.html
Hi, I was gardening today, and canvas/philip/tests/2d.composite.canvas.source-over.html started failing on Windows GPU. Added a TEXT failure baseline at:
http://trac.webkit.org/changeset/88162
and filed a bug with more details at:
http://code.google.com/p/chromium/issues/detail?id=85051
Is anyone familiar with this test? Thanks!
Adam Klein
Comment 20
2011-11-22 10:52:11 PST
The Skia roll in
http://crrev.com/111151
caused the following tests to start passing: canvas/philip/tests/2d.shadow.canvas.alpha-expected.txt canvas/philip/tests/2d.shadow.canvas.basic-expected.txt canvas/philip/tests/2d.shadow.canvas.transparent.2-expected.txt canvas/philip/tests/2d.shadow.image.alpha-expected.txt canvas/philip/tests/2d.shadow.image.basic-expected.txt canvas/philip/tests/2d.shadow.image.scale-expected.txt canvas/philip/tests/2d.shadow.image.transparent.2-expected.txt
Stephen Chenney
Comment 21
2013-04-09 16:10:11 PDT
LayoutTest failures for Chromium are being marked WontFix. The Bug is still accessible and referenced from TestExpectations.
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