webkit/ppapi: Notify the container when a plugin is accepting touch events.

webkit/ppapi: Notify the container when a plugin is accepting touch events.

BUG=128236
TEST=manually

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=143871

[sadrul, 2012-06-22 17:50:18]

The corresponding webkit patch is https://bugs.webkit.org/show_bug.cgi?id=89769

[brettw, 2012-06-22 18:03:53]

http://codereview.chromium.org/10628020/diff/5/webkit/plugins/ppapi/ppapi_web...
File webkit/plugins/ppapi/ppapi_webplugin_impl.cc (right):

http://codereview.chromium.org/10628020/diff/5/webkit/plugins/ppapi/ppapi_web...
webkit/plugins/ppapi/ppapi_webplugin_impl.cc:111:
container_->setAcceptingTouchEvents(instance_->IsAcceptingTouchEvents());
Should the instance do this in Initialize() instead? It seems like then it would
keep all the calls to this in one file.

[sadrul, 2012-06-22 21:37:09]

http://codereview.chromium.org/10628020/diff/5/webkit/plugins/ppapi/ppapi_web...
File webkit/plugins/ppapi/ppapi_webplugin_impl.cc (right):

http://codereview.chromium.org/10628020/diff/5/webkit/plugins/ppapi/ppapi_web...
webkit/plugins/ppapi/ppapi_webplugin_impl.cc:111:
container_->setAcceptingTouchEvents(instance_->IsAcceptingTouchEvents());
On 2012/06/22 18:03:54, brettw wrote:
> Should the instance do this in Initialize() instead? It seems like then it
would
> keep all the calls to this in one file.

Good point. Done.

[brettw, 2012-06-22 22:35:55]

lgtm