Open
Bug 1334752
Opened 7 years ago
Updated 2 months ago
Intermittent browser/base/content/test/webrtc/browser_devices_get_user_media_unprompted_access_tear_off_tab.js | Test timed out -
Categories
(Core :: WebRTC, defect, P4)
Core
WebRTC
Tracking
()
REOPENED
Tracking | Status | |
---|---|---|
firefox52 | --- | unaffected |
firefox53 | --- | disabled |
firefox54 | --- | disabled |
backlog | tech-debt |
People
(Reporter: intermittent-bug-filer, Unassigned)
References
(Depends on 1 open bug, Regression)
Details
(Keywords: intermittent-failure, regression, Whiteboard: [stockwell disabled])
Attachments
(2 files, 1 obsolete file)
Filed by: philringnalda [at] gmail.com https://treeherder.mozilla.org/logviewer.html#?job_id=72791038&repo=autoland https://archive.mozilla.org/pub/firefox/tinderbox-builds/autoland-win64-pgo/1485571207/autoland_win8_64_test_pgo-mochitest-e10s-browser-chrome-2-bm126-tests1-windows-build153.txt.gz
Comment hidden (Intermittent Failures Robot) |
Updated•7 years ago
|
Blocks: 1332516
status-firefox52:
--- → unaffected
status-firefox53:
--- → affected
status-firefox54:
--- → affected
Flags: needinfo?(mchiang)
Updated•7 years ago
|
backlog: --- → tech-debt
Rank: 30
Priority: -- → P3
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee: nobody → mchiang
Flags: needinfo?(mchiang)
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Florian, From the log[1], I found the callback eventCallback[2] is never called after we call chromeDoc.defaultView.PopupNotifications.show[3]. Then mochitest timed out while waiting for the event getUserMedia:response:allow. Any idea? [1] https://treeherder.mozilla.org/logviewer.html#?job_id=74669822&repo=try&lineNumber=9200 [2] http://searchfox.org/mozilla-central/rev/b1aadb3572eaf7d2c70e19a2ba5413809d9ac698/browser/modules/webrtcUI.jsm#460 [3] http://searchfox.org/mozilla-central/rev/b1aadb3572eaf7d2c70e19a2ba5413809d9ac698/browser/modules/webrtcUI.jsm#818-823
Flags: needinfo?(florian)
Comment hidden (Intermittent Failures Robot) |
Comment 10•7 years ago
|
||
(In reply to Munro Mengjue Chiang [:mchiang] from comment #8) > Florian, > > From the log[1], I found the callback eventCallback[2] is never called after > we call chromeDoc.defaultView.PopupNotifications.show[3]. I'm not sure that's relevant. > Then mochitest > timed out while waiting for the event getUserMedia:response:allow. Any idea? How do you know if we timeout waiting for "getUserMedia:response:allow", or "recording-device-events", or both? (But I would also guess it's getUserMedia:response:allow) I wonder if it would be possible that in the case where we don't show the prompt, the "getUserMedia:response:allow" notification would sometime be sent before we return from 'yield promiseObserverCalled("getUserMedia:request")' (which requires an exchange of asynchronous messages between the parent and child processes). Seeing that this failure started happening after we resolved bug 1332516, I'm tempted to say there was an issue with that patch. I would revert the patch from bug 1332516 and try this: promise = promiseObserverCalled("getUserMedia:response:allow"); yield promiseRequestDevice(true, true, null, null, win.gBrowser.selectedBrowser); yield promiseObserverCalled("getUserMedia:request"); yield promiseNoPopupNotification("webRTC-shareDevices"); yield promise; yield expectObserverCalled("recording-device-events"); For your next try run, please limit the tests to browser/base/content/test/webrtc using: --try-test-paths browser-chrome:browser/base/content/test/webrtc
Flags: needinfo?(florian)
Comment hidden (Intermittent Failures Robot) |
(In reply to Florian Quèze [:florian] [:flo] from comment #10) > How do you know if we timeout waiting for "getUserMedia:response:allow", or > "recording-device-events", or both? (But I would also guess it's > getUserMedia:response:allow) We are waiting for both. I put some logs [1] to debug this issue. In normal case, we got these logs 20:02:40 INFO - Munro: MediaManager call getUserMedia:request 20:02:40 INFO - console.log: Munro: prompt 1 20:02:40 INFO - console.log: Munro: eventCallback: aTopic: showing 20:02:40 INFO - console.log: Munro: aRequest.secure: true 20:02:40 INFO - console.log: Munro: there is an active camera 20:02:40 INFO - console.log: Munro: there is an active mic 20:02:40 INFO - console.log: Munro: send webrtc:Allow 20:02:40 INFO - console.log: Munro: eventCallback: aTopic: removed In the abnormal case, we only got 20:02:49 INFO - Munro: MediaManager call getUserMedia:request 20:02:49 INFO - console.log: Munro: prompt 1 Without sending webrtc:Allow, the test definitely would fail. > I wonder if it would be possible that in the case where we don't show the > prompt, the "getUserMedia:response:allow" notification would sometime be > sent before we return from 'yield > promiseObserverCalled("getUserMedia:request")' (which requires an exchange > of asynchronous messages between the parent and child processes). > > Seeing that this failure started happening after we resolved bug 1332516, > I'm tempted to say there was an issue with that patch. > > I would revert the patch from bug 1332516 and try this: > > promise = promiseObserverCalled("getUserMedia:response:allow"); > yield promiseRequestDevice(true, true, null, null, > win.gBrowser.selectedBrowser); > yield promiseObserverCalled("getUserMedia:request"); > yield promiseNoPopupNotification("webRTC-shareDevices"); > yield promise; > yield expectObserverCalled("recording-device-events"); After modifying the mochitest, the test result [2] still shows the same symptom. [1] https://hg.mozilla.org/try/rev/4987c35ea2abc9651cf47d4188c8caac3a319de9 [2] https://treeherder.mozilla.org/#/jobs?repo=try&revision=4987c35ea2abc9651cf47d4188c8caac3a319de9&selectedJob=76197470
Comment 13•7 years ago
|
||
(In reply to Munro Mengjue Chiang [:mchiang] from comment #12) > After modifying the mochitest, the test result [2] still shows the same > symptom. The results seem very different to me. On your previous try push, you had 7 failures out of 20 runs without e10s, and 0 failure with e10s (not completely sure my count is right, because there's the browser_verify_content_about_newtab.js failure that adds noise to these results). On the new push, you have 0 failure out of 107 runs without e10s, and 1 failure out of 147 runs with e10s. So that's 1 failure out of 250+ runs; probably below the threshold where we would start investing time in debugging.
Comment 14•7 years ago
|
||
Your logs are difficult to read because you used console.log instead of info(), so your logging wasn't buffered, while the rest of the test output was buffered. You may want to use SimpleTest.requestCompleteLog() But indeed, the popupnotification doesn't seem to show up in the one failure that remains. One possible reason could be that the window is not focused (yet?). When looking at other tests that call replaceTabWithWindow, I found: http://searchfox.org/mozilla-central/rev/afcf40f3eafd895611a839017730debb58a342a6/browser/base/content/test/plugins/browser_CTP_drag_drop.js#62 doing a promiseWaitForFocus call before continuing. This function is defined at http://searchfox.org/mozilla-central/rev/afcf40f3eafd895611a839017730debb58a342a6/browser/base/content/test/plugins/head.js#310 Some popupnotification tests do a waitForWindowReadyForPopupNotifications call: http://searchfox.org/mozilla-central/source/browser/base/content/test/popupNotifications/browser_popupNotification_4.js#143 That's defined at http://searchfox.org/mozilla-central/rev/afcf40f3eafd895611a839017730debb58a342a6/browser/base/content/test/popupNotifications/head.js#34 and also waits for the window to be selected.
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (mozreview-request) |
Comment 19•7 years ago
|
||
mozreview-review |
Comment on attachment 8836586 [details] Bug 1334752 - Fix Intermittent browser_devices_get_user_media_unprompted_access_tear_off_tab.js; https://reviewboard.mozilla.org/r/111982/#review113262 Straight forward, and try is green, so let's land this and see what happens :-), r=me.
Attachment #8836586 -
Flags: review?(florian) → review+
Keywords: checkin-needed
Comment 20•7 years ago
|
||
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3a166ded7d98 Fix Intermittent browser_devices_get_user_media_unprompted_access_tear_off_tab.js; r=florian
Keywords: checkin-needed
I had to back this out for turning the test permafail on OSX debug like https://treeherder.mozilla.org/logviewer.html#?job_id=76891577&repo=autoland https://hg.mozilla.org/integration/autoland/rev/24a43cd8a5684e613d641eae424ca03365e7faec https://treeherder.mozilla.org/#/jobs?repo=autoland&fromchange=8e075d90bf3133576a275dd2f28a4a83ce375979&filter-searchStr=10.%20m(bc3%20debug&selectedJob=76891577&tochange=24a43cd8a5684e613d641eae424ca03365e7faec
Flags: needinfo?(mchiang)
Comment hidden (Intermittent Failures Robot) |
Flags: needinfo?(mchiang)
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 27•7 years ago
|
||
to reduce the impact on developers lets disable this on win8 for now until pto is over and we can do another try here.
Attachment #8840424 -
Flags: review?(gbrown)
Updated•7 years ago
|
Keywords: leave-open
Comment 28•7 years ago
|
||
Comment on attachment 8840424 [details] [diff] [review] temporarily disable test win8 This is disabling an unrelated test.
Attachment #8840424 -
Flags: review?(gbrown) → review-
Comment 29•7 years ago
|
||
oh, I see the flaws of my error- similar names but I missed the unprompted_access part of the test :(
Comment 30•7 years ago
|
||
ok, this is for the real test.
Attachment #8840424 -
Attachment is obsolete: true
Attachment #8840559 -
Flags: review?(florian)
Updated•7 years ago
|
Attachment #8840559 -
Flags: review?(florian) → review+
Comment hidden (Intermittent Failures Robot) |
Comment 32•7 years ago
|
||
Pushed by jmaher@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/e4f60215a489 Intermittent browser/base/content/test/webrtc/browser_devices_get_user_media_unprompted_access_tear_off_tab.js. r=gbrown
Comment 33•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e4f60215a489
Updated•7 years ago
|
Whiteboard: [stockwell disabled]
Comment hidden (Intermittent Failures Robot) |
Comment 35•7 years ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/136a88ba8ecb
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 43•6 years ago
|
||
Mass change P3->P4 to align with new Mozilla triage process.
Priority: P3 → P4
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 99•4 years ago
|
||
Mike, please take a look.
Flags: needinfo?(mconley)
Regressed by: 1641546
Whiteboard: [stockwell disabled] → [stockwell disabled][retriggered][stockwell needswork:owner]
Updated•4 years ago
|
Keywords: regression
Comment hidden (Intermittent Failures Robot) |
Comment 101•4 years ago
|
||
The failures were fixed by this backout: https://hg.mozilla.org/integration/autoland/rev/c4dc5740defdf2888d7aadd20ceaf8b6dcb80ace
Flags: needinfo?(mconley)
Whiteboard: [stockwell disabled][retriggered][stockwell needswork:owner] → [stockwell disabled]
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Updated•2 years ago
|
Has Regression Range: --- → yes
Comment 104•2 years ago
|
||
The bug assignee didn't login in Bugzilla in the last 7 months.
:mjf, could you have a look please?
For more information, please visit auto_nag documentation.
Assignee: bonchiang → nobody
Flags: needinfo?(mfroman)
Comment 105•2 years ago
|
||
It appears this hasn't happened in 2 years and was low priority. Comment 101 says this was fixed by a backout. I'm going to close this until we see it happen again.
Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(mfroman)
Resolution: --- → INACTIVE
Updated•2 years ago
|
Keywords: leave-open
Comment 106•9 months ago
|
||
Still disabled.
https://searchfox.org/mozilla-central/rev/a3a9112d4d73d1323eabbc7faa9937cd9aae6465/browser/base/content/test/webrtc/browser.ini#94,96
Depends on: 1691625
Updated•9 months ago
|
Status: RESOLVED → REOPENED
Resolution: INACTIVE → ---
Updated•2 months ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•