Closed Bug 1220033 Opened 5 years ago Closed 5 years ago

Modern.ie MSE demo stalls

Categories

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

x86_64
Windows 10
defect

Tracking

()

VERIFIED FIXED
mozilla45
Tracking Status
firefox43 --- verified
firefox44 --- verified
firefox45 --- verified
b2g-v2.5 --- fixed

People

(Reporter: cpearce, Assigned: jya)

References

(Blocks 1 open bug, )

Details

(Whiteboard: [bugday-20151111])

Attachments

(2 files)

Microsoft's demo at https://dev.modern.ie/testdrive/demos/eme/ stalls on the unencrypted Big Buck Bunny demo. I tested in Beta 42, Windows 7.

STR:
1. Open https://dev.modern.ie/testdrive/demos/eme/
2. Select the unencrypted Big Buck Bunny video.
3. Start it playing, and play around with the bitrate slider at the bottom, eventually playback still stall.
Duplicate of this bug: 1219473
Assignee: nobody → jyavenard
Blocks: MSE
Attachment #8682895 - Flags: review?(gsquelart) → review+
Priority: -- → P2
https://hg.mozilla.org/mozilla-central/rev/849e4c4784ca
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Comment on attachment 8682895 [details] [diff] [review]
Don't use fuzz arithmetic for calculating internal buffered ranges

Approval Request Comment
[Feature/regressing bug #]: 1220033
[User impact if declined]: Site using MSE may stall
[Describe test coverage new/current, TreeHerder]: In central for a week
[Risks and why]: Low. It's the proper thing to do.
[String/UUID change made/needed]: None
Attachment #8682895 - Flags: approval-mozilla-beta?
Attachment #8682895 - Flags: approval-mozilla-aurora?
Comment on attachment 8682895 [details] [diff] [review]
Don't use fuzz arithmetic for calculating internal buffered ranges

Prevents a video hang, let's take this for aurora and beta.
Attachment #8682895 - Flags: approval-mozilla-beta?
Attachment #8682895 - Flags: approval-mozilla-beta+
Attachment #8682895 - Flags: approval-mozilla-aurora?
Attachment #8682895 - Flags: approval-mozilla-aurora+
Attached image bug-1219473.PNG
¡Hola Jean-Yves!

Ended up here from https://bugzilla.mozilla.org/show_bug.cgi?id=1219473

The steps on that bug no longer cause the playback to stop on Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:45.0) Gecko/20100101 Firefox/45.0 ID:20151110030205 CSet: cc473fe5dc512c450634506f68cbacfb40a06a23 so I'm marking this verified.

There's still the error "TypeError: video.error is null" being shown at the top the page and a bit of weird artifacting on the bottom part of the video (please see bug-1219473.PNG attached).

Shall I file separate bugs for these or are these known already?

¡Gracias!
Alex
Flags: needinfo?(jyavenard)
Status: RESOLVED → VERIFIED
Whiteboard: [bugday-20151111]
(In reply to alex_mayorga from comment #8)

> Shall I file separate bugs for these or are these known already?

This is likely the same as bug 1222201, and there's another.

I'll look into those shortly
Flags: needinfo?(jyavenard)
Updating Fx45 flag per Comment 8. Setting [qe-verify+] for verification on 43b3.
Flags: qe-verify+
Reproduced with 43.0b2 under Windows 7 x64. 
Still reproducible with 43.0b3 build 2 (Build ID: 20151112144305), under Windows 7 64-bit and Windows 10x32
Browser console output:
- ‘setting 'encrypted' event listener’ thrown by demo.js:451:13
- ‘Setting src: [object MediaSource]’ thrown by demo.js:412:13

Jean-Yves, any ideas why? Should I reopen this bug or file a different one?
Flags: qe-verify+ → needinfo?(jyavenard)
Only the first stream can work. The others are EME using CDM we don't support.

But you're right I can reproduce it on beta and aurora too....

need to check what was uplifted and it's missing anyting
oh I see... it's the wrong patch that was uplifted. Well, right path but wrong bug number.

Tomcat, it's this patch that need uplifting
Flags: needinfo?(jyavenard) → needinfo?(cbook)
Flags: qe-verify+
I reproduced the initial issue using an old build (Firefox 43 beta 2), verified that the issue does not reproduce anymore using Firefox 43 beta 4 and latest Firefox Developer Edition 44.0a2 on Windows 7 64-bit and Windows 10 64-bit.
Flags: needinfo?(cbook)
You need to log in before you can comment on or make changes to this bug.