Closed Bug 1106776 Opened 5 years ago Closed 5 years ago

Intermittent mediasource-config-change-mp4-a-bitrate.html,mediasource-config-change-webm-a-bitrate.html | Tests {mp4,webm} audio-only bitrate changes. - assert_greater_than: duration expected a number greater than 2 but got NaN

Categories

(Core :: Audio/Video, defect)

All
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla38
Tracking Status
firefox36 --- wontfix
firefox37 --- fixed
firefox38 --- fixed
firefox-esr31 --- unaffected

People

(Reporter: cbook, Assigned: jya)

References

()

Details

(Keywords: intermittent-failure)

Attachments

(2 files)

Windows 7 32-bit mozilla-central pgo test web-platform-tests-4

https://treeherder.mozilla.org/ui/logviewer.html#?job_id=715920&repo=mozilla-central

06:21:46 INFO - TEST-UNEXPECTED-FAIL | /media-source/mediasource-config-change-webm-a-bitrate.html | Tests webm audio-only bitrate changes. - assert_greater_than: duration expected a number greater than 2 but got NaN
Hardware: x86 → All
Summary: Intermittent mediasource-config-change-webm-a-bitrate.html | Tests webm audio-only bitrate changes. - assert_greater_than: duration expected a number greater than 2 but got NaN → Intermittent mediasource-config-change-mp4-a-bitrate.html,mediasource-config-change-webm-a-bitrate.html | Tests {mp4,webm} audio-only bitrate changes. - assert_greater_than: duration expected a number greater than 2 but got NaN
this is due to bug 1125776
Depends on: 1125776
https://hg.mozilla.org/mozilla-central/rev/fc58ed477ccb
Assignee: nobody → cpearce
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
The webm test case was left enabled, so this is only sorta "fixed".

https://hg.mozilla.org/releases/mozilla-aurora/rev/8d168e340cd5
https://hg.mozilla.org/releases/mozilla-beta/rev/4c698f648403
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: mozilla38 → ---
Disable offending test, since it tests behaviour that we don't implement. It's a wonder that it passed in the first place.

r=jya
Attachment #8556779 - Flags: review+
Comment on attachment 8556779 [details] [diff] [review]
Patch: Disable test.

Approval Request Comment
[Feature/regressing bug #]: Media Source Extensions
[User impact if declined]: Test failures; can't land other required fixes.
[Describe test coverage new/current, TreeHerder]: Disables test.
[Risks and why]: Low risk; this disables a test that exercises functionality that is not used by YouTube, and that we don't implement anyway. It's a wonder the test ever passed.
[String/UUID change made/needed]: None.
Attachment #8556779 - Flags: approval-mozilla-beta?
Attachment #8556779 - Flags: approval-mozilla-aurora?
Comment on attachment 8556779 [details] [diff] [review]
Patch: Disable test.

Test-only changes don't need approval. Besides, this was already uplifted. That said, should we kill the webm version as well?
Flags: needinfo?(cpearce)
Attachment #8556779 - Flags: approval-mozilla-beta?
Attachment #8556779 - Flags: approval-mozilla-aurora?
Bah! I should stick to writing code.

Yes, I think the webm version should be disabled too.
Flags: needinfo?(cpearce)
Assignee: cpearce → jyavenard
Status: REOPENED → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/74d1c70ac2cb
Status: ASSIGNED → RESOLVED
Closed: 5 years ago5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in before you can comment on or make changes to this bug.