Bogdan Maris, Desktop QA Reporter
	Description • 10 years ago

Latest Nightly 34.0a1 (buildID: 20140722030201)

STR:
1. Open Firefox
2. Set media.getusermedia.screensharing.enabled to true
3. Open http://queze.net/goinfre/ff_gum_test.htm in a new tab
4. Click Window&Audio
5. Open Window or screen to share dropdown and select something that can be closed afterwards (eg: a folder)
6. Click Share Selected Items
7. Have both Firefox and the selected item to share (eg: a folder) in focus.
8. Move the mouse in the selected item (eg: a folder)

Expected result: The pointer shows in the same place on both Firefox and selected item.

Actual result: The pointer is misplaced in Firefox, a few pixels up.

Notes:
1. This is Linux only.

	Maire Reavy [:mreavy]
	Comment 1 • 10 years ago

Gcp, Matt -- can one of you guys take/own this bug?

	Maire Reavy [:mreavy]
	Comment 2 • 10 years ago

I'd like to get this fixed in fx34, but I don't think we need to uplift this

	Gian-Carlo Pascutto [:gcp]
	Comment 3 • 9 years ago

Confirmed that this still reproduces (not necessarily up, but indeed not in the exact same place).

	carlosellery
	Comment 4 • 8 years ago

Hello, any good news about this? The company I work for (10000+ empl) is on FF 45.1 ESR and Ubuntu 16.04. This bug is impeding us to use WebRTC screen sharing on FF. :(

I'm newbie developer. Is there anything I can do to solve this case? I've noted it happens only on window sharing and not on screen sharing.

	Nico Grunbaum [:ng, @chew:mozilla.org]
	Comment 6 • 8 years ago

Comment on attachment 8772530 [details] [diff] [review]
Fix Linux mouse position when sharing a window in WebRTC screensharing

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

+, with fix

::: media/webrtc/trunk/webrtc/modules/desktop_capture/mouse_cursor_monitor_x11.cc
@@ +229,5 @@
>      const DesktopCaptureOptions& options,
>      ScreenId screen) {
>    if (!options.x_display())
>      return NULL;
> +  WindowId window = DefaultRootWindow(options.x_display()->display();

Missing closing )

	Pulsebot
	Comment 7 • 8 years ago

Pushed by rjesup@wgate.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/51c543f66795
Fix Linux mouse position when sharing a window in WebRTC screensharing r=ng

	Randell Jesup [:jesup] (needinfo me) Assignee
	Comment 8 • 8 years ago

FYI, the mouse position capture was using the decoration window (with borders from the WM), not the inner window (whose coordinate system it used to apply the cursor).

Note: ESR45 only gets critical crash/security fixes in general.  This will be in 50, perhaps 49, and will eventually be in ESR 52.  50 goes to release in around 15ish weeks; 49 around 6 weeks less.

	Pulsebot
	Comment 9 • 8 years ago

Pushed by rjesup@wgate.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6765609dcb46
Bustage fix for typo when resolving nits rs=bustage

	Carsten Book [:Tomcat]
	Comment 10 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/51c543f66795
https://hg.mozilla.org/mozilla-central/rev/6765609dcb46

	Maire Reavy [:mreavy]
	Comment 11 • 8 years ago

Randell -- Can you ask for uplift to Fx 49?  Thanks.

	Randell Jesup [:jesup] (needinfo me) Assignee
	Comment 12 • 8 years ago

Comment on attachment 8772530 [details] [diff] [review]
Fix Linux mouse position when sharing a window in WebRTC screensharing

Note: land this patch and the followup bustage-fix that landed to correct an oops when resolving nits before landing.

Approval Request Comment
[Feature/regressing bug #]: N/A

[User impact if declined]: When screensharing a window in Linux, the pointer position is significantly off (too low, and maybe too right).  For some uses this breaks the feature (you can't say "look here")

[Describe test coverage new/current, TreeHerder]: Manual testing of position, basic sharing is tested.

[Risks and why]: very low risk; just use a different windowid as the base for asking the mouse position.

[String/UUID change made/needed]: none

	Gerry Chang [:gchang]
	Comment 13 • 8 years ago

Comment on attachment 8772530 [details] [diff] [review]
Fix Linux mouse position when sharing a window in WebRTC screensharing

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

This patch fixes a UI issue when screensharing a window in Linux. Take it in aurora. This also needs to be verified.

	Ryan VanderMeulen [:RyanVM]
	Comment 14 • 8 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-aurora/rev/957bd28fba3e

	Bogdan Maris, Desktop QA Reporter
	Comment 15 • 8 years ago

Using Ubuntu 14.04 x64 I can see that the issue is fixed using Firefox 49 beta 8. Though using Ubuntu 16.04 x64 and x86 the pointer is invisible on both latest Nightly 51.0a1 and Firefox 49 beta 8. 
I did test this using https://mozilla.github.io/webrtc-landing/gum_test.html on two different machines. Any ideas here?

	Randell Jesup [:jesup] (needinfo me) Assignee
	Comment 16 • 8 years ago

No - please file a separate bug on that (also, confirm if it worked on 16.04 in older releases/ever).

	Bogdan Maris, Desktop QA Reporter
	Comment 17 • 8 years ago

Logged bug 1299751 on that, mozregression pointed to this bug as a possible culprit. 
I will go ahead and close this bug as verified fixed since I did verify on 14.04 and 12.04 as well and continue the discussions for the issue from 16.04 to bug 1299751.