Reland http://codereview.chromium.org/7452002/, update chromoting

Reland http://codereview.chromium.org/7452002/

Update chromoting input events. (Clang caught this. Thanks, Clang!)

Note I'm leaving in temporary backwards-compatibility.

brettw, just need a rubber-stamp. This is the same as the last CL. awong/garykac/sergeyu, looking for someone on chromoting team to glance at my changes and make sure I didn't do anything you don't like. This is a high priority for 14, so please look if you can.

BUG=None
TEST=ppapi tests

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

[brettw, 2011-07-20 17:28:24]

LGTM

[Sergey Ulanov, 2011-07-20 17:41:50]

remoting - LGTM

http://codereview.chromium.org/7466002/diff/2030/remoting/client/plugin/peppe...
File remoting/client/plugin/pepper_input_handler.h (right):

http://codereview.chromium.org/7466002/diff/2030/remoting/client/plugin/peppe...
remoting/client/plugin/pepper_input_handler.h:15: 
nit: don't need this empty line

http://codereview.chromium.org/7466002/diff/2030/remoting/client/plugin/peppe...
remoting/client/plugin/pepper_input_handler.h:18: 
nit: don't need this empty line

[wez1, 2011-07-20 22:36:53]

The changes under remoting/ look to duplicate
http://codereview.chromium.org/7453003/?

On 20 July 2011 10:41, <sergeyu@chromium.org> wrote:

> remoting - LGTM
>
>
> http://codereview.chromium.**org/7466002/diff/2030/**
>
remoting/client/plugin/pepper_**input_handler.h<http://codereview.chromium.org/7466002/diff/2030/remoting/client/plugin/pepper_input_handler.h>
> File remoting/client/plugin/pepper_**input_handler.h (right):
>
> http://codereview.chromium.**org/7466002/diff/2030/**
>
remoting/client/plugin/pepper_**input_handler.h#newcode15<http://codereview.chromium.org/7466002/diff/2030/remoting/client/plugin/pepper_input_handler.h#newcode15>
> remoting/client/plugin/pepper_**input_handler.h:15:
> nit: don't need this empty line
>
> http://codereview.chromium.**org/7466002/diff/2030/**
>
remoting/client/plugin/pepper_**input_handler.h#newcode18<http://codereview.chromium.org/7466002/diff/2030/remoting/client/plugin/pepper_input_handler.h#newcode18>
> remoting/client/plugin/pepper_**input_handler.h:18:
> nit: don't need this empty line
>
>
>
http://codereview.chromium.**org/7466002/<http://codereview.chromium.org/7466...
>

[dmichael(do not use this one), 2011-07-20 22:40:48]

I'm sorry, I wasn't aware of that one. I thought we were ready to go so
pulled the old interface. Then saw that remoting wasn't done, so I did it
quickly. I can circle back and fix anything I did differently in a future
CL, but I really want my CL to stick if that's okay.

On Wed, Jul 20, 2011 at 4:36 PM, Wez <wez@google.com> wrote:

> The changes under remoting/ look to duplicate
> http://codereview.chromium.org/7453003/?
>
>
> On 20 July 2011 10:41, <sergeyu@chromium.org> wrote:
>
>> remoting - LGTM
>>
>>
>> http://codereview.chromium.**org/7466002/diff/2030/**
>>
remoting/client/plugin/pepper_**input_handler.h<http://codereview.chromium.org/7466002/diff/2030/remoting/client/plugin/pepper_input_handler.h>
>> File remoting/client/plugin/pepper_**input_handler.h (right):
>>
>> http://codereview.chromium.**org/7466002/diff/2030/**
>>
remoting/client/plugin/pepper_**input_handler.h#newcode15<http://codereview.chromium.org/7466002/diff/2030/remoting/client/plugin/pepper_input_handler.h#newcode15>
>> remoting/client/plugin/pepper_**input_handler.h:15:
>> nit: don't need this empty line
>>
>> http://codereview.chromium.**org/7466002/diff/2030/**
>>
remoting/client/plugin/pepper_**input_handler.h#newcode18<http://codereview.chromium.org/7466002/diff/2030/remoting/client/plugin/pepper_input_handler.h#newcode18>
>> remoting/client/plugin/pepper_**input_handler.h:18:
>> nit: don't need this empty line
>>
>>
>>
http://codereview.chromium.**org/7466002/<http://codereview.chromium.org/7466...
>>
>
>