Bug 90585

Summary: Skip platform/mac/editing/deleting/backward-delete.html on android
Product: WebKit Reporter: Xingnan Wang <xingnan.wang>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, ap, james.wei, wangxianzhu, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Android   
OS: Android   
Attachments:
Description Flags
Patch none

Xingnan Wang
Reported 2012-07-04 22:03:39 PDT
As this test only for mac.
Attachments
Patch (1.42 KB, patch)
2012-07-04 22:05 PDT, Xingnan Wang 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Xingnan Wang
Comment 1 2012-07-04 22:05:30 PDT
Created attachment 150874 [details] Patch
Adam Barth
Comment 2 2012-07-04 22:25:57 PDT
Is there a public buildbot for chromium-android? It's not going to be possible to keep the TestExpectations file up-to-date without a public buildbot.
Xingnan Wang
Comment 3 2012-07-04 22:36:31 PDT
(In reply to comment #2) > Is there a public buildbot for chromium-android? It's not going to be possible to keep the TestExpectations file up-to-date without a public buildbot. Do you mean this http://build.webkit.org/builders/Chromium%20Android%20Release?
Alexey Proskuryakov
Comment 4 2012-07-04 22:43:01 PDT
I think that Android should skip everything in platform/mac, not just a single test.
Xingnan Wang
Comment 5 2012-07-04 23:11:43 PDT
(In reply to comment #4) > I think that Android should skip everything in platform/mac, not just a single test. Agree, so I think we can use this WONTFIX SKIP ANDROID : platform/mac = TEXT IMAGE IMAGE+TEXT PASS TIMEOUT CRASH at top?
Adam Barth
Comment 6 2012-07-04 23:35:30 PDT
> Do you mean this http://build.webkit.org/builders/Chromium%20Android%20Release? I mean one that runs tests. > > I think that Android should skip everything in platform/mac, not just a single test. > > Agree, so I think we can use this > WONTFIX SKIP ANDROID : platform/mac = TEXT IMAGE IMAGE+TEXT PASS TIMEOUT CRASH > at top? That's not consistent with other Chromium ports. We should ask the folks who made that decisions, but for some reason Chromium runs all the platform-specific tests for all the platforms.
Xingnan Wang
Comment 7 2012-08-08 20:21:12 PDT
(In reply to comment #6) > > Do you mean this http://build.webkit.org/builders/Chromium%20Android%20Release? > > I mean one that runs tests. > > > > I think that Android should skip everything in platform/mac, not just a single test. > > > > Agree, so I think we can use this > > WONTFIX SKIP ANDROID : platform/mac = TEXT IMAGE IMAGE+TEXT PASS TIMEOUT CRASH > > at top? > > That's not consistent with other Chromium ports. We should ask the folks who made that decisions, but for some reason Chromium runs all the platform-specific tests for all the platforms. As Bug 92653 landed, can the one be reviewed now?
WebKit Review Bot
Comment 8 2012-08-08 22:21:43 PDT
Comment on attachment 150874 [details] Patch Clearing flags on attachment: 150874 Committed r125150: <http://trac.webkit.org/changeset/125150>
WebKit Review Bot
Comment 9 2012-08-08 22:21:47 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.