Closed
Bug 1089484
Opened 10 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•10 years ago
|
||
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•10 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•10 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
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/160999d8824b
Keywords: checkin-needed
Comment 5•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/160999d8824b
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
•