Revert "synthetic gesture: Allow high-frequency dispatch."

This reverts commit 581a728b.

Reason for revert: Causing significant test flakiness in telemetry.internal.actions.scroll_unittest.ScrollActionTest.testScrollDistanceSlowWheel.

Original change's description:
> synthetic gesture: Allow high-frequency dispatch.
> 
> For synthesized scroll/drag/pinch gestures, dispatch the events
> at a high frequency (120Hz) so that there's at least one event in
> each frame. Dispatching events at a lower frequency (60Hz) means
> the timer can go out of sync with begin-frame, and so there can
> be frames where there was to event dispatch. Our telemetry code
> reports this as jank, which is incorrect. Dispatching at a higher
> frequency for the scroll/drag/pinch gestures resolves this issue.
> 
> BUG=783034
> 
> Change-Id: I1102f76ed743231cbb10fdb872e8c66f895fa223
> Reviewed-on: https://chromium-review.googlesource.com/c/1255152
> Reviewed-by: Victor Miura <vmiura@chromium.org>
> Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
> Reviewed-by: Lan Wei <lanwei@chromium.org>
> Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#596507}

TBR=sadrul@chromium.org,vmiura@chromium.org,lanwei@chromium.org,nzolghadr@chromium.org

Change-Id: I1e4d64dfc3b98a547c38f8f82a2dcd45141fd66b
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 783034, 892036
Reviewed-on: https://chromium-review.googlesource.com/c/1261972Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596748}

content/browser/renderer_host/input/synthetic_gesture.cc

@@ -52,8 +52,4 @@ std::unique_ptr<SyntheticGesture> SyntheticGesture::Create(
   }
 }
 
-bool SyntheticGesture::AllowHighFrequencyDispatch() const {
-  return true;
-}
-
 }  // namespace content

content/browser/renderer_host/input/synthetic_gesture.h

@@ -51,12 +51,6 @@ class CONTENT_EXPORT SyntheticGesture {
   virtual Result ForwardInputEvents(
       const base::TimeTicks& timestamp, SyntheticGestureTarget* target) = 0;
 
-  // Returns whether the gesture events can be dispatched at high frequency
-  // (e.g. at 120Hz), instead of the regular frequence (at 60Hz). Some gesture
-  // interact differently depending on how long they take (e.g. the TAP gesture
-  // generates a click only if its duration is longer than a threshold).
-  virtual bool AllowHighFrequencyDispatch() const;
-
  protected:
   DISALLOW_COPY_AND_ASSIGN(SyntheticGesture);
 };

content/browser/renderer_host/input/synthetic_gesture_controller.cc

@@ -40,10 +40,11 @@ void SyntheticGestureController::QueueSyntheticGesture(
     StartGesture(*pending_gesture_queue_.FrontGesture());
 }
 
-void SyntheticGestureController::StartTimer(bool high_frequency) {
+void SyntheticGestureController::StartTimer() {
+  // TODO(sad): Change the interval to allow sending multiple events per begin
+  // frame.
   dispatch_timer_.Start(
-      FROM_HERE,
-      base::TimeDelta::FromMicroseconds(high_frequency ? 8333 : 16666),
+      FROM_HERE, base::TimeDelta::FromMicroseconds(16666),
       base::BindRepeating(
           [](base::WeakPtr<SyntheticGestureController> weak_ptr) {
             if (weak_ptr)
@@ -89,7 +90,7 @@ void SyntheticGestureController::StartGesture(const SyntheticGesture& gesture) {
                            "SyntheticGestureController::running",
                            &gesture);
   if (!dispatch_timer_.IsRunning())
-    StartTimer(gesture.AllowHighFrequencyDispatch());
+    StartTimer();
 }
 
 void SyntheticGestureController::StopGesture(

content/browser/renderer_host/input/synthetic_gesture_controller.h

@@ -52,7 +52,7 @@ class CONTENT_EXPORT SyntheticGestureController {
  private:
   friend class SyntheticGestureControllerTestBase;
 
-  void StartTimer(bool high_frequency);
+  void StartTimer();
   void StartGesture(const SyntheticGesture& gesture);
   void StopGesture(const SyntheticGesture& gesture,
                    OnGestureCompleteCallback completion_callback,

content/browser/renderer_host/input/synthetic_pointer_action.cc

@@ -47,10 +47,6 @@ SyntheticGesture::Result SyntheticPointerAction::ForwardInputEvents(
                           : SyntheticGesture::GESTURE_RUNNING;
 }
 
-bool SyntheticPointerAction::AllowHighFrequencyDispatch() const {
-  return false;
-}
-
 SyntheticPointerAction::GestureState
 SyntheticPointerAction::ForwardTouchOrMouseInputEvents(
     const base::TimeTicks& timestamp,

content/browser/renderer_host/input/synthetic_pointer_action.h

@@ -24,7 +24,6 @@ class CONTENT_EXPORT SyntheticPointerAction : public SyntheticGesture {
   SyntheticGesture::Result ForwardInputEvents(
       const base::TimeTicks& timestamp,
       SyntheticGestureTarget* target) override;
-  bool AllowHighFrequencyDispatch() const override;
 
  private:
   enum GestureState { UNINITIALIZED, RUNNING, INVALID, DONE };

content/browser/renderer_host/input/synthetic_tap_gesture.cc

@@ -46,10 +46,6 @@ SyntheticGesture::Result SyntheticTapGesture::ForwardInputEvents(
                           : SyntheticGesture::GESTURE_RUNNING;
 }
 
-bool SyntheticTapGesture::AllowHighFrequencyDispatch() const {
-  return false;
-}
-
 void SyntheticTapGesture::ForwardTouchOrMouseInputEvents(
     const base::TimeTicks& timestamp, SyntheticGestureTarget* target) {
   switch (state_) {

content/browser/renderer_host/input/synthetic_tap_gesture.h

@@ -22,7 +22,6 @@ class CONTENT_EXPORT SyntheticTapGesture : public SyntheticGesture {
   SyntheticGesture::Result ForwardInputEvents(
       const base::TimeTicks& timestamp,
       SyntheticGestureTarget* target) override;
-  bool AllowHighFrequencyDispatch() const override;
 
  private:
   enum GestureState {