Closed Bug 1106776 Opened 10 years ago Closed 10 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
normal

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
Assignee: nobody → cpearce
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
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
Status: ASSIGNED → RESOLVED
Closed: 10 years ago10 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.

Attachment

General

Created:
Updated:
Size: