Intermittent test_playback.html | split.webm checking readyState - got 4, expected 2

RESOLVED FIXED in Firefox 45

Status

()

Core
Audio/Video: Playback
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

({intermittent-failure})

unspecified
mozilla45
intermittent-failure
Points:
---

Firefox Tracking Flags

(firefox45 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=16236257

 01:47:00     INFO -  44279 INFO [finished redirect.sjs?domain=mochi.test:8888&file=320x240.ogv-33] remaining= seek.webm-34
 49131 
 01:47:00     INFO -  44280 INFO TEST-PASS | dom/media/test/test_playback.html | [finished redirect.sjs?domain=mochi.test:8888&file=320x240.ogv-33] Length of array should match number of running tests
 49132 
 01:47:00     INFO -  44281 INFO TEST-PASS | dom/media/test/test_playback.html | [started split.webm-35] Length of array should match number of running tests
 49133 
 01:47:00     INFO -  44282 INFO TEST-PASS | dom/media/test/test_playback.html | split.webm: Name should match #1
 49134 
 01:47:00     INFO -  44283 INFO TEST-PASS | dom/media/test/test_playback.html | split.webm duration (1.967) should be around 1.967
 49135 
 01:47:00     INFO -  44284 INFO TEST-PASS | dom/media/test/test_playback.html | split.webm isEncrypted should be true if we have decryption keys
 49136 
 01:47:00     INFO -  44285 INFO TEST-PASS | dom/media/test/test_playback.html | split.webm: Name should match #3
 49137 
 01:47:00     INFO -  44286 INFO TEST-PASS | dom/media/test/test_playback.html | split.webm: Name should match #2
 49138 
 01:47:00     INFO -  44287 INFO TEST-PASS | dom/media/test/test_playback.html | split.webm duration (2.001) should be around 1.967
 49139 
 01:47:00     INFO -  44288 INFO TEST-PASS | dom/media/test/test_playback.html | split.webm isEncrypted should be true if we have decryption keys
 49140 
 01:47:00     INFO -  44289 INFO TEST-UNEXPECTED-FAIL | dom/media/test/test_playback.html | split.webm checking readyState - got 4, expected 2
(Assignee)

Updated

2 years ago
Keywords: intermittent-failure
(Assignee)

Comment 1

2 years ago
https://hg.mozilla.org/mozilla-central/file/96377bdbcdf3/dom/media/MediaDecoderStateMachine.cpp#l731

For a video-only file, next frame status is updated in OnVideoPopped() which sometimes comes after MediaDecoder::PlaybackEnded() is notified. Therefore, we get an outdated readyState in the 'ended' event handler.

The fix is simple by call UpdateNextFrameStatus() before notifying MediaDecoder::PlaybackEnded() to ensure readyState is updated again before 'ended' fired.
(Assignee)

Updated

2 years ago
Duplicate of this bug: 1218633
(Assignee)

Updated

2 years ago
Assignee: nobody → jwwang
(Assignee)

Comment 3

2 years ago
Created attachment 8681755 [details]
MozReview Request: Bug 1220512 - ensure next frame status is updated before notifying MediaDecoder::PlaybackEnded. See bug 1220512 comment 1 for the detail. r=cpearce.

Bug 1220512 - ensure next frame status is updated before notifying MediaDecoder::PlaybackEnded. See bug 1220512 comment 1 for the detail. r=cpearce.
Attachment #8681755 - Flags: review?(cpearce)
(Assignee)

Updated

2 years ago
Duplicate of this bug: 1218299
Attachment #8681755 - Flags: review?(cpearce) → review+
Comment on attachment 8681755 [details]
MozReview Request: Bug 1220512 - ensure next frame status is updated before notifying MediaDecoder::PlaybackEnded. See bug 1220512 comment 1 for the detail. r=cpearce.

https://reviewboard.mozilla.org/r/23877/#review21433
(Assignee)

Comment 6

2 years ago
Thanks!

Comment 7

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/a9ba9c5fee9a

Comment 8

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a9ba9c5fee9a
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox45: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45

Comment 9

2 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/a9ba9c5fee9a
status-b2g-v2.5: --- → fixed
removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion
status-b2g-v2.5: fixed → ---

Comment 11

2 years ago
6 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-aurora: 3
* mozilla-inbound: 2
* fx-team: 1

Platform breakdown:
* b2g-emu-ics: 3
* linux64: 2
* windows8-64: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1220512&startday=2015-11-09&endday=2015-11-15&tree=all
You need to log in before you can comment on or make changes to this bug.