Generate tap gesture with tap_count = 3 for triple-tap

Generate tap gesture with tap_count = 3 for triple-tap

This is to make triple-tap select a paragraph.

BUG=236402
TEST=triple-tap on a paragraph and make sure the whole paragraph is
     selected.

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

[sadrul, 2013-05-28 17:48:46]

https://codereview.chromium.org/16018006/diff/1/ui/aura/gestures/gesture_reco...
File ui/aura/gestures/gesture_recognizer_unittest.cc (right):

https://codereview.chromium.org/16018006/diff/1/ui/aura/gestures/gesture_reco...
ui/aura/gestures/gesture_recognizer_unittest.cc:2969:
root_window()->AsRootWindowHostDelegate()->OnHostTouchEvent(&release2);
check tap_count here and after the first tap

https://codereview.chromium.org/16018006/diff/1/ui/base/gestures/gesture_poin...
File ui/base/gestures/gesture_point.cc (right):

https://codereview.chromium.org/16018006/diff/1/ui/base/gestures/gesture_poin...
ui/base/gestures/gesture_point.cc:183: 
Refactor IsInSecondClickTimeWindow and IsInThirdClickTimeWindow into one
function:

  IsInClickAggregateTimeWindow(double before, double after)

https://codereview.chromium.org/16018006/diff/1/ui/base/gestures/gesture_poin...
ui/base/gestures/gesture_point.cc:211: 
Similarly,

 IsLocationsInsideManhattanSquare(point 1, point 2)

[Yufeng Shen (Slow to review), 2013-05-28 19:31:44]

https://codereview.chromium.org/16018006/diff/1/ui/aura/gestures/gesture_reco...
File ui/aura/gestures/gesture_recognizer_unittest.cc (right):

https://codereview.chromium.org/16018006/diff/1/ui/aura/gestures/gesture_reco...
ui/aura/gestures/gesture_recognizer_unittest.cc:2969:
root_window()->AsRootWindowHostDelegate()->OnHostTouchEvent(&release2);
On 2013/05/28 17:48:46, sadrul wrote:
> check tap_count here and after the first tap

Done.

https://codereview.chromium.org/16018006/diff/1/ui/base/gestures/gesture_poin...
File ui/base/gestures/gesture_point.cc (right):

https://codereview.chromium.org/16018006/diff/1/ui/base/gestures/gesture_poin...
ui/base/gestures/gesture_point.cc:183: 
On 2013/05/28 17:48:46, sadrul wrote:
> Refactor IsInSecondClickTimeWindow and IsInThirdClickTimeWindow into one
> function:
> 
>   IsInClickAggregateTimeWindow(double before, double after)

Done.

https://codereview.chromium.org/16018006/diff/1/ui/base/gestures/gesture_poin...
ui/base/gestures/gesture_point.cc:211: 
On 2013/05/28 17:48:46, sadrul wrote:
> Similarly,
> 
>  IsLocationsInsideManhattanSquare(point 1, point 2)
> 

Done.

[sadrul, 2013-05-28 19:53:14]

LGTM

[Yufeng Shen (Slow to review), 2013-05-28 20:01:57]

cc sky@ for OWNER of  ash/touch/touch_uma.h

[sky, 2013-05-28 21:08:37]

LGTM

https://codereview.chromium.org/16018006/diff/6001/ash/touch/touch_uma.cc
File ash/touch/touch_uma.cc (right):

https://codereview.chromium.org/16018006/diff/6001/ash/touch/touch_uma.cc#new...
ash/touch/touch_uma.cc:205: else if (tap_count == 2)
nit: style guide says no else for ifs that return, eg:

if (tab_count == 1)
  return ..
if (tab_count == 2)
  return ..

[commit-bot: I haz the power, 2013-05-28 21:19:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/miletus@chromium.org/16018006/7006

[commit-bot: I haz the power, 2013-05-29 01:22:47]

Change committed as 202735