input: Change how synthesized events are dispatched in telemetry tests.

content/browser/renderer_host/render_widget_host_view_base.h

Index: content/browser/renderer_host/render_widget_host_view_base.h
diff --git a/content/browser/renderer_host/render_widget_host_view_base.h b/content/browser/renderer_host/render_widget_host_view_base.h
index 2c2364fc861f41c2dabb04a7dcb3389fca89ad70..49a5831bd241c2194f893dd5df52ca18772d6baf 100644
--- a/content/browser/renderer_host/render_widget_host_view_base.h
+++ b/content/browser/renderer_host/render_widget_host_view_base.h
@@ -17,7 +17,6 @@
 #include "base/observer_list.h"
 #include "base/process/kill.h"
 #include "base/strings/string16.h"
-#include "base/timer/timer.h"
 #include "build/build_config.h"
 #include "cc/ipc/mojo_compositor_frame_sink.mojom.h"
 #include "cc/output/compositor_frame.h"
@@ -190,7 +189,7 @@ class CONTENT_EXPORT RenderWidgetHostViewBase : public RenderWidgetHostView,
 
   // Called by the host when it requires an input flush; the flush call should
   // by synchronized with BeginFrame.
-  virtual void OnSetNeedsFlushInput();
+  virtual void OnSetNeedsFlushInput() {}

[dtapuska, 2017/05/16 14:39:41]

Would it not make sense to make this pure virtual?

[sadrul, 2017/05/16 17:35:06]

I initially did that, but added back the default impl because of
RWHVChildFrame/RWHVGuest. But making this pure virtual makes it easier to follow
the code I think. Done.

 
   virtual void WheelEventAck(const blink::WebMouseWheelEvent& event,
                              InputEventAckState ack_result);
@@ -457,14 +456,10 @@ class CONTENT_EXPORT RenderWidgetHostViewBase : public RenderWidgetHostView,
   TextInputManager* text_input_manager_;
 
  private:
-  void FlushInput();
-
   gfx::Rect current_display_area_;
 
   uint32_t renderer_frame_number_;
 
-  base::OneShotTimer flush_input_timer_;
-
   base::ObserverList<RenderWidgetHostViewBaseObserver> observers_;
 
   base::WeakPtrFactory<RenderWidgetHostViewBase> weak_factory_;