touch: Gesture manager receives the touch-sequence status.

touch: Gesture manager receives the touch-sequence status.

OnTouchEvent now returns the status of the touch sequence, instead of a simple
bool. The gesture manager can presumably make a better decision if this
information is available to it. For more details: http://codereview.chromium.org/6347002/

BUG=none
TEST=ViewTest.TouchEvent

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

[sadrul, 2011-01-18 19:33:30]

http://codereview.chromium.org/6253005/diff/1/views/widget/root_view.cc
File views/widget/root_view.cc (right):

http://codereview.chromium.org/6253005/diff/1/views/widget/root_view.cc#newco...
views/widget/root_view.cc:318: gesture_manager_->ProcessTouchEventForGesture(e,
this, status);
Should the status fall through, or do we always want to send
TOUCH_STATUS_CONTINUE instead (seeing as how it always sent 'true' before)?

[rjkroege, 2011-01-18 20:01:27]

was there any test updating needed for this?

http://codereview.chromium.org/6253005/diff/1/views/widget/root_view.cc
File views/widget/root_view.cc (right):

http://codereview.chromium.org/6253005/diff/1/views/widget/root_view.cc#newco...
views/widget/root_view.cc:318: gesture_manager_->ProcessTouchEventForGesture(e,
this, status);
On 2011/01/18 19:33:30, sadrul wrote:
> Should the status fall through, or do we always want to send
> TOUCH_STATUS_CONTINUE instead (seeing as how it always sent 'true' before)?

the true was a bug. This is correct. (famous last words)

[rjkroege, 2011-01-18 20:01:45]

LGTM

[sadrul, 2011-01-18 20:39:19]

On 2011/01/18 20:01:27, rjkroege wrote:
> was there any test updating needed for this?
> 

Indeed there was. Updated the unittest so ViewTest.TouchEvent passes.

[sky, 2011-01-18 22:59:40]

LGTM

[rjkroege, 2011-01-19 19:28:29]

SLGTM