[Chromoting] Log mouse events in remoting_key_tester

[Chromoting] Log mouse events in remoting_key_tester

We have received a bug to complain scrolling moves in only one direction when
zooming with CAD tools in chromoting from Mac to Windows. This issue won't
reproduce on Linux client, so a potential Mac PNACL implementation defect could
be the root cause. So this change logs mouse related events in
remoting_key_tester to help debugging on Mac.

BUG=

Committed: https://crrev.com/55492620e9b09c593cd353b2a9f77317a4837d7f
Cr-Commit-Position: refs/heads/master@{#399525}

[Hzj_jie, 2016-06-12 19:31:21]

Description was changed from

==========
[Chromoting] Log mouse events in remoting_key_tester

We have received a bug to complain scrolling moves in only one direction when
zooming with CAD tools in chromoting from Mac to Windows. This issue won't
reproduce on Linux client, so a potential Mac PNACL implementation defect could
be the root cause. So this change logs mouse related events in
remoting_key_tester to help debugging on Mac.

BUG=
==========

to

==========
[Chromoting] Log mouse events in remoting_key_tester

We have received a bug to complain scrolling moves in only one direction when
zooming with CAD tools in chromoting from Mac to Windows. This issue won't
reproduce on Linux client, so a potential Mac PNACL implementation defect could
be the root cause. So this change logs mouse related events in
remoting_key_tester to help debugging on Mac.

BUG=
==========

[Hzj_jie, 2016-06-12 19:31:21]

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

[Jamie, 2016-06-13 17:25:42]

lgtm

https://codereview.chromium.org/2063623002/diff/1/remoting/tools/javascript_k...
File remoting/tools/javascript_key_tester/pnacl/remoting_key_tester.cc (right):

https://codereview.chromium.org/2063623002/diff/1/remoting/tools/javascript_k...
remoting/tools/javascript_key_tester/pnacl/remoting_key_tester.cc:23:
LogToConsole(PP_LOGLEVEL_LOG,
I don't think you want LogToConsole here. If you do want to log something, it
should be more descriptive than just an int.

[Hzj_jie, 2016-06-13 20:23:09]

https://codereview.chromium.org/2063623002/diff/1/remoting/tools/javascript_k...
File remoting/tools/javascript_key_tester/pnacl/remoting_key_tester.cc (right):

https://codereview.chromium.org/2063623002/diff/1/remoting/tools/javascript_k...
remoting/tools/javascript_key_tester/pnacl/remoting_key_tester.cc:23:
LogToConsole(PP_LOGLEVEL_LOG,
On 2016/06/13 17:25:42, Jamie wrote:
> I don't think you want LogToConsole here. If you do want to log something, it
> should be more descriptive than just an int.

Oh, these are for debugging purpose, I should remove them. Also the one in line
30.

[Hzj_jie, 2016-06-13 20:23:29]

The CQ bit was checked by zijiehe@chromium.org

[Hzj_jie, 2016-06-13 20:23:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
jamiewalch@chromium.org
Link to the patchset: https://codereview.chromium.org/2063623002/#ps20001
(title: "Resolve review comments -- Remove LogToConsole(s).")

[commit-bot: I haz the power, 2016-06-13 20:24:27]

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

[commit-bot: I haz the power, 2016-06-13 20:30:48]

Description was changed from

==========
[Chromoting] Log mouse events in remoting_key_tester

We have received a bug to complain scrolling moves in only one direction when
zooming with CAD tools in chromoting from Mac to Windows. This issue won't
reproduce on Linux client, so a potential Mac PNACL implementation defect could
be the root cause. So this change logs mouse related events in
remoting_key_tester to help debugging on Mac.

BUG=
==========

to

==========
[Chromoting] Log mouse events in remoting_key_tester

We have received a bug to complain scrolling moves in only one direction when
zooming with CAD tools in chromoting from Mac to Windows. This issue won't
reproduce on Linux client, so a potential Mac PNACL implementation defect could
be the root cause. So this change logs mouse related events in
remoting_key_tester to help debugging on Mac.

BUG=
==========

[commit-bot: I haz the power, 2016-06-13 20:30:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-13 20:31:03]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-13 20:33:00]

Description was changed from

==========
[Chromoting] Log mouse events in remoting_key_tester

We have received a bug to complain scrolling moves in only one direction when
zooming with CAD tools in chromoting from Mac to Windows. This issue won't
reproduce on Linux client, so a potential Mac PNACL implementation defect could
be the root cause. So this change logs mouse related events in
remoting_key_tester to help debugging on Mac.

BUG=
==========

to

==========
[Chromoting] Log mouse events in remoting_key_tester

We have received a bug to complain scrolling moves in only one direction when
zooming with CAD tools in chromoting from Mac to Windows. This issue won't
reproduce on Linux client, so a potential Mac PNACL implementation defect could
be the root cause. So this change logs mouse related events in
remoting_key_tester to help debugging on Mac.

BUG=

Committed: https://crrev.com/55492620e9b09c593cd353b2a9f77317a4837d7f
Cr-Commit-Position: refs/heads/master@{#399525}
==========

[commit-bot: I haz the power, 2016-06-13 20:33:01]

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