Use offsetX and offsetY to position the fake mouse cursor.

Use offsetX and offsetY to position the fake mouse cursor.

The mouse cursor element is positioned relative to the document body (ie, ignoring
the title bar for apps v2). However, its position was being set using the x and y
coordinates of the mouse event, which are relative to the client viewport. This CL
fixes this, using offsetX/Y, which are relative to the target element.

BUG=410473
Committed: https://crrev.com/a19b19882cd9eff4696d0a081e6fc2f9b95f9c18
Cr-Commit-Position: refs/heads/master@{#317447}

[Jamie, 2015-02-20 19:36:14]

jamiewalch@chromium.org changed reviewers:
+ kelvinp@chromium.org

[Jamie, 2015-02-20 19:36:14]

ptal

[kelvinp, 2015-02-20 21:46:22]

lgtm

https://codereview.chromium.org/923013004/diff/1/remoting/webapp/crd/js/clien...
File remoting/webapp/crd/js/client_session.js (right):

https://codereview.chromium.org/923013004/diff/1/remoting/webapp/crd/js/clien...
remoting/webapp/crd/js/client_session.js:179: img.style.top = event.offsetY +
'px';
Is this ToT?  This function has been moved to desktop_connected_view.js

[Jamie, 2015-02-20 22:44:41]

New patchsets have been uploaded after l-g-t-m from kelvinp@chromium.org

[Jamie, 2015-02-20 22:46:03]

https://codereview.chromium.org/923013004/diff/1/remoting/webapp/crd/js/clien...
File remoting/webapp/crd/js/client_session.js (right):

https://codereview.chromium.org/923013004/diff/1/remoting/webapp/crd/js/clien...
remoting/webapp/crd/js/client_session.js:179: img.style.top = event.offsetY +
'px';
On 2015/02/20 21:46:22, kelvinp wrote:
> Is this ToT?  This function has been moved to desktop_connected_view.js

No, I was behind. Thanks for the heads-up.

[Jamie, 2015-02-20 22:46:08]

The CQ bit was checked by jamiewalch@chromium.org

[commit-bot: I haz the power, 2015-02-20 22:46:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/923013004/20001

[commit-bot: I haz the power, 2015-02-21 00:42:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-21 00:42:47]

Patchset 2 (id:??) landed as
https://crrev.com/a19b19882cd9eff4696d0a081e6fc2f9b95f9c18
Cr-Commit-Position: refs/heads/master@{#317447}