Removes the |event| parameter from ActivationDelegate::ShouldActivate and makes the method const.

Removes the |event| parameter from ActivationDelegate::ShouldActivate and makes the method const.

It is mostly unused, except for one place in RenderWidgetHostViewAura, where depending on the event type etc it will have a consequence. I think it's bad for a method with a const-sounding name like "ShouldActivate" to have side-effects.

Instead, I made the RWHVA store the data that would indicate a pointer activate and check that state during OnActivated(). It'd be a little cleaner if OnActivated had some way of knowing what the currently processed input event was, hence the TODO.

http://crbug.com/162100
R=sadrul@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170477

[sadrul, 2012-11-30 17:17:50]

LGTM