webkit/ppapi: Notify the container when a plugin is accepting wheel events just like r143871.

webkit/ppapi: Notify the container when a plugin is accepting wheel events just like r143871.

Otherwise, the container might not be able to receive wheel events in some cases
(e.g. when threaded compositing is in use).

BUG=158506
TEST=manually

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

[Yusuke Sato, 2012-11-20 17:52:41]

James,
Could you have a look at the Chrome part as well?

[jamesr, 2012-11-26 06:23:07]

lgtm but could be a bit cleaner, IMO

https://codereview.chromium.org/11312195/diff/2001/webkit/plugins/ppapi/ppapi...
File webkit/plugins/ppapi/ppapi_plugin_instance.cc (right):

https://codereview.chromium.org/11312195/diff/2001/webkit/plugins/ppapi/ppapi...
webkit/plugins/ppapi/ppapi_plugin_instance.cc:2150:
container_->setWantsWheelEvents(IsAcceptingWheelEvents());
this little 4-line block of checking for PP_INPUTEVENT_CLASS_... is repeated 3
times now. Could you fold these into a helper function?

[Yusuke Sato, 2012-11-26 23:22:18]

https://codereview.chromium.org/11312195/diff/2001/webkit/plugins/ppapi/ppapi...
File webkit/plugins/ppapi/ppapi_plugin_instance.cc (right):

https://codereview.chromium.org/11312195/diff/2001/webkit/plugins/ppapi/ppapi...
webkit/plugins/ppapi/ppapi_plugin_instance.cc:2150:
container_->setWantsWheelEvents(IsAcceptingWheelEvents());
On 2012/11/26 06:23:07, jamesr wrote:
> this little 4-line block of checking for PP_INPUTEVENT_CLASS_... is repeated 3
> times now. Could you fold these into a helper function?

Done.

[commit-bot: I haz the power, 2012-11-26 23:27:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yusukes@chromium.org/11312195/10006