1324260 - Intermittent browser/base/content/test/webrtc/browser_devices_get_user_media_screen.js | NotFoundError, likely because there's no other application running. - Got error: OverconstrainedError: Constraints could be not satisfied....

Status: RESOLVED INCOMPLETE

(Core :: WebRTC, defect, P2)

Description

[Treeherder Bug Filer]

Filed by: philringnalda [at] gmail.com

https://treeherder.mozilla.org/logviewer.html#?job_id=8067922&repo=autoland

https://archive.mozilla.org/pub/firefox/tinderbox-builds/autoland-macosx64/1482012170/autoland_yosemite_r7_test-mochitest-browser-chrome-6-bm132-tests1-macosx-build292.txt.gz

Comment 1

[Florian Quèze [:florian]]

The failure here is:

15:16:40 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/webrtc/browser_devices_get_user_media_screen.js | NotFoundError, likely because there's no other application running. - Got error: OverconstrainedError: Constraints could be not satisfied., expected error: NotFoundError: The object can not be found here.


Which of these 2 messages should the test expect when there's no non-Firefox application running?
  NotFoundError: The object can not be found here.
or
  OverconstrainedError: Constraints could be not satisfied.

Is it currently inconsistent across platforms, or did I misinterpret these errors while writing the test?
If needed I could make the test accept either message, but I don't want to just hide a platform bug.

Comment 2

[Randell Jesup [:jesup] (needinfo me)]

Forwarding to jib

Comment 3

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

NotFoundError is the correct error when no apps are available (I simulated it locally to confirm). I have no idea where OverconstrainedError is coming from here. Do you have a reproducible case?

If not, instrumenting the test to include error.constraint in the log output may help shed light on it.

Comment 4

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Wild guess: Is this test testing the permission prompt? If so, may the error be coming from the preview code in https://dxr.mozilla.org/mozilla-central/rev/567894f026558e6dada617a3998f29aed06ac7d8/browser/modules/webrtcUI.jsm#548 :

    let constraints = { video: { mediaSource: type, deviceId: {exact: deviceId } } };

? It is using an exact constraint, is why I suspect it.

I could see if an app window disappeared between the time it was enumerated (and given a deviceId) to when the preview code tries to obtain it with the deviceId it cached, gUM would fail with OverconstrainedError (the error.constraint would be 'deviceId' in that case). Are preview errors suppressed properly?

Comment 5

[Florian Quèze [:florian]]

(In reply to Jan-Ivar Bruaroey [:jib] from comment #4)
> Wild guess: Is this test testing the permission prompt? If so, may the error
> be coming from the preview code in
> https://dxr.mozilla.org/mozilla-central/rev/
> 567894f026558e6dada617a3998f29aed06ac7d8/browser/modules/webrtcUI.jsm#548 :
> 
>     let constraints = { video: { mediaSource: type, deviceId: {exact:
> deviceId } } };
> 
> ? It is using an exact constraint, is why I suspect it.
> 
> I could see if an app window disappeared between the time it was enumerated
> (and given a deviceId) to when the preview code tries to obtain it with the
> deviceId it cached, gUM would fail with OverconstrainedError (the
> error.constraint would be 'deviceId' in that case). Are preview errors
> suppressed properly?

So, preview errors aren't currently handled, which would likely result in a message added in the browser console, but not rejecting the gUM request.

The failing test here is checking the message obtained from promiseMessageReceived defined at http://searchfox.org/mozilla-central/rev/cc2a84852bd4e6f6d8d4d5b17b8382bb5d005749/browser/base/content/test/webrtc/head.js#246 (chrome process) and http://searchfox.org/mozilla-central/rev/cc2a84852bd4e6f6d8d4d5b17b8382bb5d005749/browser/base/content/test/webrtc/get_user_media_content_script.js#104 (content process).
The test code sends the message at http://searchfox.org/mozilla-central/rev/cc2a84852bd4e6f6d8d4d5b17b8382bb5d005749/browser/base/content/test/webrtc/get_user_media.html#42 so I see no way for the preview code to pollute that.

Comment 6

[Jan-Ivar Bruaroey [:jib] (needinfo? me)]

Happened only once AFAIK. No luck reproducing it (w/extra error log output). Nothing actionable.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=0f4f123a77884c9d6338dd67b7d856490b232198

Comment 7

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Hit again: https://treeherder.mozilla.org/logviewer.html#?job_id=84752025&repo=autoland
Please set the bug to WONTFIX if nothing should be done here.

Comment 8

[Intermittent Failures Robot]

1 failures in 718 pushes (0.001 failures/push) were associated with this bug in the last 7 days.   

Repository breakdown:
* mozilla-release: 1

Platform breakdown:
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1324260&startday=2017-06-26&endday=2017-07-02&tree=all

Comment 9

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://treeherder.mozilla.org/logviewer.html#?job_id=141933024&repo=mozilla-inbound

Comment 10

[Intermittent Failures Robot]

1 failures in 857 pushes (0.001 failures/push) were associated with this bug in the last 7 days.    

Repository breakdown:
* mozilla-inbound: 1

Platform breakdown:
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1324260&startday=2017-10-30&endday=2017-11-05&tree=all

Comment 11

[Firefox Bug Husbandry Bot]

https://wiki.mozilla.org/Bugmasters#Intermittent_Test_Failure_Cleanup