Closed Bug 1289742 Opened 3 years ago Closed 3 years ago

Intermittent dom/media/test/test_load_same_resource.html | Clone http://mochi.test:8888/tests/dom/media/test/dynamic_resource.sjs?key=50285108&res1=320x240.ogv&res2=short-video.ogv duration: 1.081179 expected: 0.266

Categories

(Core :: Audio/Video: Playback, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: jwwang)

References

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell fixed])

Attachments

(8 files)

Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
this has gone up and down over the last few weeks, but right now it seems to be very low frequency.
Whiteboard: [stockwell unknown]
Comment on attachment 8843852 [details]
Bug 1289742. P1 - remove tests that are not relevant. dynamic_resource.sjs is the only one we need to test media cache.

https://reviewboard.mozilla.org/r/117464/#review119104
Attachment #8843852 - Flags: review?(kaku) → review+
Comment on attachment 8843853 [details]
Bug 1289742. P2 - remove the e.canPlayType() test which is already done by MediaTestManager.

https://reviewboard.mozilla.org/r/117466/#review119106
Attachment #8843853 - Flags: review?(kaku) → review+
Comment on attachment 8843854 [details]
Bug 1289742. P3 - use the 'once' handler.

https://reviewboard.mozilla.org/r/117468/#review119108
Attachment #8843854 - Flags: review?(kaku) → review+
Comment on attachment 8843855 [details]
Bug 1289742. P4 - remove the load() call because the load algirithm will be triggered when .src is assigned.

https://reviewboard.mozilla.org/r/117470/#review119110
Attachment #8843855 - Flags: review?(kaku) → review+
Comment on attachment 8843856 [details]
Bug 1289742. P5 - test.duration is mandatory to run the test.

https://reviewboard.mozilla.org/r/117472/#review119112
Attachment #8843856 - Flags: review?(kaku) → review+
Comment on attachment 8843857 [details]
Bug 1289742. P6 - give the cloned test a different token for better logging messages.

https://reviewboard.mozilla.org/r/117474/#review119114
Attachment #8843857 - Flags: review?(kaku) → review+
Comment on attachment 8843858 [details]
Bug 1289742. P7 - print PR logs for debugging

https://reviewboard.mozilla.org/r/117476/#review119094

::: dom/media/test/test_load_same_resource.html:74
(Diff revision 1)
>    var elemType = /^audio/.test(test.type) ? "audio" : "video";
>    var e = document.createElement(elemType);

Since we are cleaning up this file, condisder using the helper funtion in manifest.js, `function getMajorMimeType(mimetype) {...}`, here.
Attachment #8843858 - Flags: review?(kaku) → review+
Comment on attachment 8843858 [details]
Bug 1289742. P7 - print PR logs for debugging

https://reviewboard.mozilla.org/r/117476/#review119094

> Since we are cleaning up this file, condisder using the helper funtion in manifest.js, `function getMajorMimeType(mimetype) {...}`, here.

We can always create a video element which is able to play both audio and video files.
Comment on attachment 8843882 [details]
Bug 1289742. P8 - always create a video element which is capable of playing both audio and video.

https://reviewboard.mozilla.org/r/117508/#review119132
Attachment #8843882 - Flags: review?(kaku) → review+
Thanks!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9a65dfeb3d70
P1 - remove tests that are not relevant. dynamic_resource.sjs is the only one we need to test media cache. r=kaku
https://hg.mozilla.org/integration/autoland/rev/f5c0df74ee73
P2 - remove the e.canPlayType() test which is already done by MediaTestManager. r=kaku
https://hg.mozilla.org/integration/autoland/rev/34ad60160536
P3 - use the 'once' handler. r=kaku
https://hg.mozilla.org/integration/autoland/rev/f4ce1268e40b
P4 - remove the load() call because the load algirithm will be triggered when .src is assigned. r=kaku
https://hg.mozilla.org/integration/autoland/rev/c0b2e94b0b5e
P5 - test.duration is mandatory to run the test. r=kaku
https://hg.mozilla.org/integration/autoland/rev/c3f28dc126fa
P6 - give the cloned test a different token for better logging messages. r=kaku
https://hg.mozilla.org/integration/autoland/rev/ca56682eb7c1
P7 - print PR logs for debugging r=kaku
https://hg.mozilla.org/integration/autoland/rev/b73a2e4a93d7
P8 - always create a video element which is capable of playing both audio and video. r=kaku
Whiteboard: [stockwell unknown] → [stockwell fixed]
Component: DOM → Audio/Video: Playback
These failures go way back on trunk, but seem to never hit on the release branches. Do you have any idea why that would be and what that says about whether this should be consider for backport or not?
Assignee: nobody → jwwang
Flags: needinfo?(jwwang)
No. This is just a workaround instead of a fix. I will keep monitoring this bug.
Flags: needinfo?(jwwang)
You need to log in before you can comment on or make changes to this bug.