If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Rewrite test_mediaElementAudioSourceNodeCrossOrigin.html to be less machine-loading dependent

RESOLVED FIXED in Firefox 41

Status

()

Core
Web Audio
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

unspecified
mozilla41
Points:
---

Firefox Tracking Flags

(firefox41 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
See https://bugzilla.mozilla.org/show_bug.cgi?id=1169356#c70.

Since bug 1174077 is landed, we are now able to play a captured media element till the end even if it is playing a CORS resource.

We can now re-write test_mediaElementAudioSourceNodeCrossOrigin.html by checking if we get silence as expected before the end of playback without depending on the heuristic loop count.
(Assignee)

Comment 1

2 years ago
Created attachment 8624008 [details] [diff] [review]
1175786_rewrite_testcase-v1.patch

Try looks green. https://treeherder.mozilla.org/#/jobs?repo=try&revision=38839aa04595

No failures in test_mediaElementAudioSourceNodeCrossOrigin.html in about 150 runs.
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
Attachment #8624008 - Flags: review?(padenot)
Comment on attachment 8624008 [details] [diff] [review]
1175786_rewrite_testcase-v1.patch

Review of attachment 8624008 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks !
Attachment #8624008 - Flags: review?(padenot) → review+
(Assignee)

Comment 3

2 years ago
Thanks for the review!
(Assignee)

Updated

2 years ago
Keywords: checkin-needed

Comment 4

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/39e7b3bc8ff7
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/39e7b3bc8ff7
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox41: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.