Don't handle touch events in the OOP PDF plugin.

Don't handle touch events in the OOP PDF plugin.

We don't handle touch events in any special way in the plugin. Viewport
operations all happen in the container page so we can skip handling them
in the plugin.

Committed: https://crrev.com/ffb9bd55ca4328556ca495b9661b42e9d76a42ea
Cr-Commit-Position: refs/heads/master@{#314109}

[raymes, 2015-01-30 05:21:10]

raymes@chromium.org changed reviewers:
+ sammc@chromium.org

[Sam McNally, 2015-01-30 06:04:51]

https://codereview.chromium.org/889583005/diff/1/pdf/out_of_process_instance.cc
File pdf/out_of_process_instance.cc (left):

https://codereview.chromium.org/889583005/diff/1/pdf/out_of_process_instance....
pdf/out_of_process_instance.cc:281:
RequestFilteringInputEvents(PP_INPUTEVENT_CLASS_TOUCH);
Are touch events used for links?

[raymes, 2015-02-02 01:13:26]

They seem to still work, probably because they get turned into mouse events.

On Fri Jan 30 2015 at 5:04:52 PM <sammc@chromium.org> wrote:

>
> https://codereview.chromium.org/889583005/diff/1/pdf/out_of_
> process_instance.cc
> File pdf/out_of_process_instance.cc (left):
>
> https://codereview.chromium.org/889583005/diff/1/pdf/out_of_
> process_instance.cc#oldcode281
> pdf/out_of_process_instance.cc:281
>
<https://codereview.chromium.org/889583005/diff/1/pdf/out_of_process_instance....>
> :
> RequestFilteringInputEvents(PP_INPUTEVENT_CLASS_TOUCH);
> Are touch events used for links?
>
> https://codereview.chromium.org/889583005/
>

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

[Sam McNally, 2015-02-02 03:32:40]

lgtm

[raymes, 2015-02-02 03:32:59]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2015-02-02 03:34:06]

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

[commit-bot: I haz the power, 2015-02-02 08:12:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-02-02 08:13:42]

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