Pepper: Fix comments for PPB and PPP InputEvent.

Pepper: Fix comments for PPB and PPP InputEvent.

The comments provided in PPB_InputEvent and PPP_InputEvent incorrectly describe
how handled events are delivered to handlers in web pages.

When a plugin handles an event, the event is not "bubbled" to any default
handlers. However, any listeners in a web page run before the event reaches the
plugin, so the plugin can't prevent any listeners from getting the event.

An attempt to make the implementation match the specification broke existing
plugins, so the only reasonable approach is to make the comments match the
behavior.

BUG=324351

[dmichael (off chromium), 2014-04-08 19:37:59]

lgtm

https://codereview.chromium.org/228653006/diff/1/ppapi/api/ppp_input_event.idl
File ppapi/api/ppp_input_event.idl (right):

https://codereview.chromium.org/228653006/diff/1/ppapi/api/ppp_input_event.id...
ppapi/api/ppp_input_event.idl:24: * in the web page.  If it was not handled, it
will bubble according to the
"bubble" might give the wrong impression. What about:
"If it was not handled, it may be dispatched to a default handler."

https://codereview.chromium.org/228653006/diff/1/ppapi/cpp/instance.h
File ppapi/cpp/instance.h (right):

https://codereview.chromium.org/228653006/diff/1/ppapi/cpp/instance.h#newcode178
ppapi/cpp/instance.h:178: /// handlers. If it was not handled, it will bubble
according to the normal
Same comment as for the C

[teravest, 2014-04-08 21:21:40]

On Tue, Apr 8, 2014 at 1:37 PM,  <dmichael@chromium.org> wrote:
> lgtm
>
>
>
>
>
https://codereview.chromium.org/228653006/diff/1/ppapi/api/ppp_input_event.idl
> File ppapi/api/ppp_input_event.idl (right):
>
>
https://codereview.chromium.org/228653006/diff/1/ppapi/api/ppp_input_event.id...
> ppapi/api/ppp_input_event.idl:24: * in the web page.  If it was not
>
> handled, it will bubble according to the
> "bubble" might give the wrong impression. What about:
> "If it was not handled, it may be dispatched to a default handler."

Good call, done.

>
> https://codereview.chromium.org/228653006/diff/1/ppapi/cpp/instance.h
> File ppapi/cpp/instance.h (right):
>
>
https://codereview.chromium.org/228653006/diff/1/ppapi/cpp/instance.h#newcode178
> ppapi/cpp/instance.h:178: /// handlers. If it was not handled, it will
>
> bubble according to the normal
> Same comment as for the C

Done.

>
> https://codereview.chromium.org/228653006/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[teravest, 2014-04-10 19:13:02]

Submitted.

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@263034