Closed
Bug 1116006
Opened 9 years ago
Closed 9 years ago
Intermittent test_bug448534.html | Video should not be paused while playing
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla37
Tracking | Status | |
---|---|---|
firefox35 | --- | unaffected |
firefox36 | --- | fixed |
firefox37 | --- | fixed |
firefox-esr31 | --- | unaffected |
People
(Reporter: philor, Assigned: jwwang)
Details
(Keywords: intermittent-failure)
Attachments
(1 file, 1 obsolete file)
912 bytes,
patch
|
cpearce
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 2•9 years ago
|
||
Add some debugging logs. For now it is not clear which file emits the fail logs.
Updated•9 years ago
|
Attachment #8542013 -
Flags: review?(cpearce) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed,
leave-open
Comment 3•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1869a533edc4
Keywords: checkin-needed
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 17•9 years ago
|
||
For a short file (320x240.ogv is 0.266s long), it could reach the end before 'play' received. We should skip the test for 'paused' would be true when ended.
Attachment #8542013 -
Attachment is obsolete: true
Attachment #8544475 -
Flags: review?(cpearce)
Assignee | ||
Comment 18•9 years ago
|
||
Try: https://tbpl.mozilla.org/?tree=Try&rev=353da258fec7
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•9 years ago
|
Attachment #8544475 -
Flags: review?(cpearce) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: leave-open → checkin-needed
Comment 20•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ee4eb01e2b14
Keywords: checkin-needed
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 22•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ee4eb01e2b14
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Comment 23•9 years ago
|
||
Comment on attachment 8544475 [details] [diff] [review] 1116006_fix_ended_before_play_event-v1.patch Approval Request Comment [Feature/regressing bug #]: MSE [User impact if declined]: Less consistent testing. [Describe test coverage new/current, TBPL]: Landed on m-c. [Risks and why]: Minimal. Mochitest is made more permissive. [String/UUID change made/needed]: None.
Attachment #8544475 -
Flags: approval-mozilla-aurora?
Comment 24•9 years ago
|
||
Comment on attachment 8544475 [details] [diff] [review] 1116006_fix_ended_before_play_event-v1.patch Test-only changes don't need approval.
Attachment #8544475 -
Flags: approval-mozilla-aurora?
Comment 25•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/6e5c3475184b
status-firefox35:
--- → unaffected
status-firefox36:
--- → fixed
status-firefox37:
--- → fixed
status-firefox-esr31:
--- → unaffected
Flags: in-testsuite+
Comment 26•9 years ago
|
||
Thanks, Ryan!
You need to log in
before you can comment on or make changes to this bug.
Description
•