Refactoring: Extract gesture event filtering into helper class.

Refactoring: Extract gesture event filtering into helper class.

In the course of writing
http://src.chromium.org/viewvc/chrome?view=rev&revision=150463
I noticed that there was getting to be quite a bit of gesture
event related code in RenderWidgetHostImpl. This change gathers
this code together in an external helper class so that some
restructuring needed prior to completing 137555 is easier. This
CL should not change any functionality -- only moves code.

BUG=137555
TBR=jam@chromium.org


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=151063

[rjkroege, 2012-08-09 18:05:59]

nits? suggestions?

[sadrul, 2012-08-09 18:27:43]

Very nice! LGTM

http://codereview.chromium.org/10823262/diff/2001/content/browser/renderer_ho...
File content/browser/renderer_host/gesture_event_filter.h (right):

http://codereview.chromium.org/10823262/diff/2001/content/browser/renderer_ho...
content/browser/renderer_host/gesture_event_filter.h:5: #ifndef
CONTENT_BROWSER_RENDERER_HOST_GESTURE_EVENT_FILTER_
.._FILTER_H_ :)

[rjkroege, 2012-08-09 19:59:12]

https://chromiumcodereview.appspot.com/10823262/diff/2001/content/browser/ren...
File content/browser/renderer_host/gesture_event_filter.h (right):

https://chromiumcodereview.appspot.com/10823262/diff/2001/content/browser/ren...
content/browser/renderer_host/gesture_event_filter.h:5: #ifndef
CONTENT_BROWSER_RENDERER_HOST_GESTURE_EVENT_FILTER_
On 2012/08/09 18:27:43, sadrul wrote:
> .._FILTER_H_ :)

but it was long enough already. :-)

fixed.

[rjkroege, 2012-08-09 21:00:26]

sky@ for OWNERS please?

[sky, 2012-08-09 21:42:52]

LGTM

https://chromiumcodereview.appspot.com/10823262/diff/8/content/browser/render...
File content/browser/renderer_host/gesture_event_filter.h (right):

https://chromiumcodereview.appspot.com/10823262/diff/8/content/browser/render...
content/browser/renderer_host/gesture_event_filter.h:16: using
WebKit::WebGestureEvent;
No using in headers.

[rjkroege, 2012-08-10 15:12:19]

https://chromiumcodereview.appspot.com/10823262/diff/8/content/browser/render...
File content/browser/renderer_host/gesture_event_filter.h (right):

https://chromiumcodereview.appspot.com/10823262/diff/8/content/browser/render...
content/browser/renderer_host/gesture_event_filter.h:16: using
WebKit::WebGestureEvent;
On 2012/08/09 21:42:52, sky wrote:
> No using in headers.

Done.

[commit-bot: I haz the power, 2012-08-10 15:12:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rjkroege@chromium.org/10823262/4003

[commit-bot: I haz the power, 2012-08-10 15:13:07]

Presubmit check for 10823262-4003 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content

Presubmit checks took 1.1s to calculate.

[rjkroege, 2012-08-10 15:34:57]

+ jam for OWNERS on TBR'd change to gypi file.

[commit-bot: I haz the power, 2012-08-10 15:35:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rjkroege@chromium.org/10823262/4003

[commit-bot: I haz the power, 2012-08-10 17:09:00]

Change committed as 151063