Modern.ie MSE demo stalls

VERIFIED FIXED in Firefox 43

Status

()

P2
normal
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: cpearce, Assigned: jya)

Tracking

(Blocks: 1 bug)

unspecified
mozilla45
x86_64
Windows 10
Points:
---

Firefox Tracking Flags

(firefox43 verified, firefox44 verified, firefox45 verified, b2g-v2.5 fixed)

Details

(Whiteboard: [bugday-20151111], URL)

Attachments

(2 attachments)

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)

Updated

3 years ago
Assignee: nobody → jyavenard
(Assignee)

Updated

3 years ago
Blocks: 778617
Attachment #8682895 - Flags: review?(gsquelart) → review+
Priority: -- → P2

Comment 4

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/849e4c4784ca
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox45: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
(Assignee)

Comment 6

3 years ago
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+

Comment 8

3 years ago
Posted 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)

Updated

3 years ago
Status: RESOLVED → VERIFIED
Whiteboard: [bugday-20151111]
(Assignee)

Comment 10

3 years ago
(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)

Comment 11

3 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/fcbbc5261056
status-firefox44: --- → fixed

Comment 12

3 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/84b0b59ed1d2
status-firefox43: --- → fixed
Updating Fx45 flag per Comment 8. Setting [qe-verify+] for verification on 43b3.
status-firefox45: fixed → verified
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)
(Assignee)

Comment 16

3 years ago
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
(Assignee)

Comment 17

3 years ago
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)
(Assignee)

Updated

3 years ago
status-firefox43: fixed → affected
status-firefox44: fixed → affected
This already landed on aurora in http://hg.mozilla.org/releases/mozilla-aurora/rev/d05eb25b0baa as far as I can tell.
status-firefox44: affected → fixed

Comment 19

3 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/f6fa2e5fb632
status-firefox43: affected → fixed
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.
status-firefox43: fixed → verified
status-firefox44: fixed → verified
Flags: needinfo?(cbook)
You need to log in before you can comment on or make changes to this bug.