Closed
Bug 1089484
Opened 11 years ago
Closed 10 years ago
Increase test coverage of test_reset_src.html
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: jwwang, Assigned: jwwang)
Details
Attachments
(1 file)
|
6.53 KB,
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
Have test_reset_src.html test more files to increase test coverage.
|
Assignee | |
Comment 1•11 years ago
|
||
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•11 years ago
|
||
Comment on attachment 8512443 [details] [diff] [review]
1089484_rewrite_test_reset_src_html_to_test_more_files.patch
Review of attachment 8512443 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/media/test/manifest.js
@@ +123,5 @@
> { name:"bug482461-theora.ogv", type:"video/ogg", duration:4.138 },
> // With first frame a "duplicate" (empty) frame.
> { name:"bug500311.ogv", type:"video/ogg", duration:1.96 },
> // Small audio file
> + { name:"small-shot.ogg", type:"audio/ogg", duration:0.276 },
In fact, this is an audio-only file.
Attachment #8512443 -
Flags: review?(kinetik)
|
||
Updated•11 years ago
|
Attachment #8512443 -
Flags: review?(kinetik) → review+
|
|
Assignee | |
Comment 3•10 years ago
|
||
Try: https://tbpl.mozilla.org/?tree=Try&rev=44a63ff6e518
Most green.
Keywords: checkin-needed
|
||
Comment 4•10 years ago
|
||
Keywords: checkin-needed
|
||
Comment 5•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•