Move early return out of RootView::DispatchGestureEvent()

Move early return out of RootView::DispatchGestureEvent()

In preparation for the eventual removal of
RootView::DispatchGestureEvent(), move an early return
out of this function and into RootView::OnEventFromSource().
This condition ignores any gesture-scroll events which were
not preceded by a ui::ET_GESTURE_SCROLL_BEGIN event.

BUG=353641
TEST=Tested by WidgetTest.GestureScrollEventDispatching

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

[tdanderson, 2014-07-18 18:55:47]

Sadrul, can you please take a look?

[sadrul, 2014-07-21 06:40:30]

lgtm

https://codereview.chromium.org/406683002/diff/1/ui/views/widget/root_view.cc
File ui/views/widget/root_view.cc (right):

https://codereview.chromium.org/406683002/diff/1/ui/views/widget/root_view.cc...
ui/views/widget/root_view.cc:279: }
Let's get rid of the elses in this block:
  if (key-event)
    return EventProcessor::OnEventFromSource(event);
  if (scroll-event)
    ..

[tdanderson, 2014-07-21 14:27:40]

https://codereview.chromium.org/406683002/diff/1/ui/views/widget/root_view.cc
File ui/views/widget/root_view.cc (right):

https://codereview.chromium.org/406683002/diff/1/ui/views/widget/root_view.cc...
ui/views/widget/root_view.cc:279: }
On 2014/07/21 06:40:30, sadrul wrote:
> Let's get rid of the elses in this block:
>   if (key-event)
>     return EventProcessor::OnEventFromSource(event);
>   if (scroll-event)
>     ..

Done.

[tdanderson, 2014-07-21 14:27:46]

The CQ bit was checked by tdanderson@chromium.org

[commit-bot: I haz the power, 2014-07-21 14:28:44]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tdanderson@chromium.org/406683002/20001

[commit-bot: I haz the power, 2014-07-21 16:21:53]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  ios_rel_device on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device/builds...)

[commit-bot: I haz the power, 2014-07-21 16:24:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-21 16:24:43]

Try jobs failed on following builders:
  ios_rel_device on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device/builds...)

[tdanderson, 2014-07-21 16:43:39]

The CQ bit was checked by tdanderson@chromium.org

[commit-bot: I haz the power, 2014-07-21 16:44:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tdanderson@chromium.org/406683002/20001

[commit-bot: I haz the power, 2014-07-21 16:51:13]

Change committed as 284446