Closed Bug 1574315 Opened 7 years ago Closed 7 years ago

Permafailing fission tier 2 toolkit/content/tests/browser/browser_block_autoplay_media.js | The suspeded state of autoplay audio is correct. - 0 == 2 -

Categories

(Core :: Audio/Video: Playback, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla71
Fission Milestone M4
Tracking Status
firefox-esr60 --- unaffected
firefox-esr68 --- unaffected
firefox69 --- unaffected
firefox70 --- disabled
firefox71 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: whawkins)

References

(Regression)

Details

(Keywords: intermittent-failure, regression)

Attachments

(1 file)

Filed by: ncsoregi [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=261862934&repo=mozilla-central
Full log: https://queue.taskcluster.net/v1/task/O8Tuc5jrRIa_KwmRaGcgQw/runs/0/artifacts/public/logs/live_backing.log


[task 2019-08-15T20:49:43.352Z] 20:49:43 INFO - Console message: [JavaScript Warning: "Autoplay is only allowed when approved by the user, the site is activated by the user, or media is muted." {file: "https://example.com/browser/toolkit/content/tests/browser/file_multipleAudio.html" line: 16}]
[task 2019-08-15T20:49:43.352Z] 20:49:43 INFO - - should block autoplay for non-visited tab4 -
[task 2019-08-15T20:49:43.353Z] 20:49:43 INFO - Buffered messages finished
[task 2019-08-15T20:49:43.353Z] 20:49:43 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/browser/browser_block_autoplay_media.js | The suspeded state of autoplay audio is correct. - 0 == 2 -
[task 2019-08-15T20:49:43.353Z] 20:49:43 INFO - Stack trace:
[task 2019-08-15T20:49:43.353Z] 20:49:43 INFO - resource://testing-common/content-task.js line 62 > eval:null:8
[task 2019-08-15T20:49:43.354Z] 20:49:43 INFO - resource://testing-common/content-task.js:null:63
[task 2019-08-15T20:49:43.354Z] 20:49:43 INFO - Console message: [JavaScript Error: "NotAllowedError: The play method is not allowed by the user agent or the platform in the current context, possibly because the user denied permission."]
[task 2019-08-15T20:49:43.355Z] 20:49:43 INFO - TEST-PASS | toolkit/content/tests/browser/browser_block_autoplay_media.js | autoplay audio should be paused. - true == true -
[task 2019-08-15T20:49:43.355Z] 20:49:43 INFO - - should not block media from tab4 -
[task 2019-08-15T20:49:43.355Z] 20:49:43 INFO - Playing state doens't match, wait for attributes changes.
[task 2019-08-15T20:50:26.437Z] 20:50:26 INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-08-15T20:50:26.439Z] 20:50:26 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/browser/browser_block_autoplay_media.js | Test timed out -
[task 2019-08-15T20:50:26.441Z] 20:50:26 INFO - GECKO(16129) | MEMORY STAT | vsize 3062MB | residentFast 306MB | heapAllocated 101MB
[task 2019-08-15T20:50:26.442Z] 20:50:26 INFO - TEST-OK | toolkit/content/tests/browser/browser_block_autoplay_media.js | took 45015ms
[task 2019-08-15T20:50:26.443Z] 20:50:26 INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-08-15T20:50:26.444Z] 20:50:26 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/browser/browser_block_autoplay_media.js | Found a tab after previous test timed out: https://example.com/browser/toolkit/content/tests/browser/file_multipleAudio.html -
[task 2019-08-15T20:50:26.459Z] 20:50:26 INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-08-15T20:50:26.460Z] 20:50:26 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/browser/browser_block_autoplay_media.js | Found a tab after previous test timed out: about:blank -
[task 2019-08-15T20:50:26.495Z] 20:50:26 INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-08-15T20:50:26.496Z] 20:50:26 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/browser/browser_block_autoplay_media.js | Found a tab after previous test timed out: https://example.com/browser/toolkit/content/tests/browser/file_multipleAudio.html -
[task 2019-08-15T20:50:26.511Z] 20:50:26 INFO - Not taking screenshot here: see the one that was previously logged
[task 2019-08-15T20:50:26.512Z] 20:50:26 INFO - TEST-UNEXPECTED-FAIL | toolkit/content/tests/browser/browser_block_autoplay_media.js | Found a tab after previous test timed out: https://example.com/browser/toolkit/content/tests/browser/file_multipleAudio.html -
[task 2019-08-15T20:50:26.535Z] 20:50:26 INFO - checking window state
[task 2019-08-15T20:50:26.551Z] 20:50:26 INFO - GECKO(16129) | JavaScript error: resource://testing-common/PromiseTestUtils.jsm, line 112: uncaught exception: Object

Regressed by: 1551313

Thanks for flagging me on this! I will absolutely take a look. I hope that I am not the cause, but I will definitely take a look. Thanks for flagging me!

Will

PS: I am going to leave the ni to keep up the pressure.:-)

Flags: needinfo?(whawkins)

I popped up a try job to recreate this error because I cannot recreate it locally. I am testing locally on a 64 bit machine so I don't know if/how my local setup differs from the try server. I'll be interested in seeing the additional logs that the try server generates.

Flags: needinfo?(whawkins)

Hi Will, are there any updates on this?

Yes, I have done extensive testing on this and, I think, have definitively determined that this is a timing-related failure. I have added extensive debugging and verified that the behavior is correct but that the tests sometimes race. I am still working to come up with a proposed fix.

Flags: needinfo?(whawkins)
Attachment #9089963 - Attachment description: Bug 1574315: Add/use a method for letting the browser attempt to start autoplaying the audio before the test checks whether it was blocked. → Bug 1574315: Specify the proper parameters to BrowserTestUtils.browserLoaded to ensure that a page's content is entirely loaded before testing for the results of side effects of operations in that content.
Assignee: nobody → whawkins
Keywords: checkin-needed

Pushed by cbrindusan@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/dd6fd67a75eb
Specify the proper parameters to BrowserTestUtils.browserLoaded to ensure that a page's content is entirely loaded before testing for the results of side effects of operations in that content.

Keywords: checkin-needed
Pushed by cbrindusan@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/3224524e42b0 Backed out changeset dd6fd67a75eb for landing without the reviewer. https://hg.mozilla.org/integration/mozilla-inbound/rev/f266902fe178 Specify the proper parameters to BrowserTestUtils.browserLoaded to ensure that a page's content is entirely loaded before testing for the results of side effects of operations in that content. r=mconley
Attachment #9089963 - Attachment description: Bug 1574315: Specify the proper parameters to BrowserTestUtils.browserLoaded to ensure that a page's content is entirely loaded before testing for the results of side effects of operations in that content. → Bug 1574315: Specify the proper parameters to BrowserTestUtils.browserLoaded to ensure that a page's content is entirely loaded before testing for the results of side effects of operations in that content. r=mconley
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71

I was worried about this. We will take our backup approach and see if that helps. Thanks for flagging this.

Will

Retroactively moving fixed bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to an appropriate Fission Milestone.

This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:

0ee3c76a-bc79-4eb2-8d12-05dc0b68e732

Fission Milestone: --- → M4
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: