Bug 82925 - Add layout test expectations for Chromium to exhibit support WebM types
Summary: Add layout test expectations for Chromium to exhibit support WebM types
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: David Dorwin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-04-02 12:22 PDT by David Dorwin
Modified: 2012-04-19 15:50 PDT (History)
1 user (show)

See Also:


Attachments
Patch (4.74 KB, patch)
2012-04-02 12:27 PDT, David Dorwin
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Dorwin 2012-04-02 12:22:21 PDT
Chromium layout tests have been running without WebM support because of layout test-specific code. That is being removed (http://crbug.com/119667), so some tests that previously used the default expectations need to be overrides.
Comment 1 David Dorwin 2012-04-02 12:27:06 PDT
Created attachment 135153 [details]
Patch
Comment 2 Andrew Scherkus 2012-04-02 13:36:32 PDT
Comment on attachment 135153 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=135153&action=review

> LayoutTests/platform/chromium/test_expectations.txt:4300
> +BUGCR119667 : media/W3C/video/canPlayType/canPlayType_codecs_order_1.html = PASS FAIL

why are these PASS FAIL?

should we wait for the chromium side change to land first?
Comment 3 David Dorwin 2012-04-02 13:56:57 PDT
Per offline discussion, I'm committing the WebKit side first, so these will start to pass when the Chromium change lands. I'll then remove these lines.
Comment 4 Andrew Scherkus 2012-04-02 14:05:50 PDT
Committed as http://trac.webkit.org/changeset/112937
Comment 5 Eric Seidel (no email) 2012-04-19 15:50:14 PDT
Comment on attachment 135153 [details]
Patch

Cleared review? from attachment 135153 [details] so that this bug does not appear in http://webkit.org/pending-review.  If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).