Clamp events before they are sent to EventExecutors.

Clamp events before they are sent to EventExecutors.

* MouseInputFilter is moved from remoting/client/ to remoting/protocol/.
* ClientSession now implements clamping using MouseInputFilter.
* EventExecutors no longer drop out-of-bounds events.

BUG=118511
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=134594

[Wez, 2012-04-27 17:39:50]

PTAL.

http://codereview.chromium.org/10255007/diff/1/remoting/host/client_session.cc
File remoting/host/client_session.cc (right):

http://codereview.chromium.org/10255007/diff/1/remoting/host/client_session.c...
remoting/host/client_session.cc:65:
mouse_input_filter_.set_input_size(capturer_->size_most_recent());
I'd like to get rid of the InputStub overrides on ClientSession, which would
currently require deriving a MouseInputClamper that accepts a Capturer to clamp
to.

http://codereview.chromium.org/10255007/diff/1/remoting/host/event_executor_l...
File remoting/host/event_executor_linux.cc (left):

http://codereview.chromium.org/10255007/diff/1/remoting/host/event_executor_l...
remoting/host/event_executor_linux.cc:400: event.x() > width_ || event.y() >
height_) {
In two minds as to whether to CHECK() the coordinates against what the capturer
is reporting.

http://codereview.chromium.org/10255007/diff/1/remoting/host/event_executor_m...
File remoting/host/event_executor_mac.cc (left):

http://codereview.chromium.org/10255007/diff/1/remoting/host/event_executor_m...
remoting/host/event_executor_mac.cc:284: event.x() < size.width() || event.y() <
size.height()) {
Ditto re CHECK()ing here, and in the Windows implementation.

[Jamie, 2012-04-27 18:39:39]

lgtm

[commit-bot: I haz the power, 2012-04-27 22:53:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/10255007/1

[commit-bot: I haz the power, 2012-04-27 23:24:17]

Try job failure for 10255007-1 (retry) on mac_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-04-28 05:21:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/10255007/6002

[commit-bot: I haz the power, 2012-04-28 05:49:12]

Try job failure for 10255007-6002 (retry) on mac_rel for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-04-30 18:52:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/10255007/12003

[commit-bot: I haz the power, 2012-04-30 20:54:46]

Change committed as 134594