Transcode test-par-16-9 LayoutTest video to .ogv
Created attachment 95507 [details] Patch
Attachment 95507 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/medi..." exit_code: 1 LayoutTests/ChangeLog:7: Line contains tab character. [whitespace/tab] [5] LayoutTests/ChangeLog:8: Line contains tab character. [whitespace/tab] [5] LayoutTests/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5] Total errors found: 3 in 1 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 95509 [details] Patch
I can never remember who does chrome video stuff... maybe atwilson?
+scherkus
tiny nit.. the original file included audio if the transcoded copy also includes audio then it's extension should be .ogg as opposed to .ogv
hrmm.. ignore my comment media-file.js expects ogv
File is good! Landed by hand due to binary content: http://trac.webkit.org/changeset/88053
Comment on attachment 95509 [details] Patch Cleared review? from attachment 95509 [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).
Nope we should be good here as I landed the patch by hand due to binary content. Apologies about not clearing the review? earlier!