Resend unconsumed scroll update from guest back to embedder (WebView Scroll Bubble).

content/browser/renderer_host/render_widget_host_impl.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
 
 #include <list>
 #include <map>
 #include <string>
@@ skipping 441 matching lines @@
   void RejectMouseLockOrUnlockIfNecessary();
 
 #if defined(OS_WIN)
   gfx::NativeViewAccessible GetParentNativeViewAccessible();
 #endif
 
   void set_renderer_initialized(bool renderer_initialized) {
     renderer_initialized_ = renderer_initialized;
   }
 
+  bool is_in_gesture_scroll() { return is_in_gesture_scroll_; }

[kenrb, 2015/09/14 15:48:51]

Is there a way to avoid exposing this and keeping the logic entirely in
RenderWidgetHostImpl? It would be cleaner if that was possible, in particular
because then BrowserPluginGuest could call Process*Event() methods on
RenderWidgetHostView instead of Forward*Event() methods on RenderWidgetHostImpl
(which isn't possible now because ProcessGestureEvent doesn't exist yet, but I'm
thinking forward a bit).

[wjmaclean, 2015/09/14 20:47:21]

Yes, but it involves detecting |resendSource| (now renamed |resendingPluginId|)
in the Forward*Event()s, and if necessary creating the synthetic
GestureScrollBegin/End within RenderWidgetHostImpl.

My original idea had been to keep as much of the machinery as possible in
BrowserPluginGuest to make it easier removing this later on (once OOPIF
scrolling works, this code will be unnecessary).

If you're OK with moving the extra baggage into RWHI then I'll go ahead and do
that.

+
  protected:
   RenderWidgetHostImpl* AsRenderWidgetHostImpl() override;
 
   // Called when we receive a notification indicating that the renderer
   // process has gone. This will reset our state so that our state will be
   // consistent if a new renderer is created.
   void RendererExited(base::TerminationStatus status, int exit_code);
 
   // Retrieves an id the renderer can use to refer to its view.
   // This is used for various IPC messages, including plugins.
@@ skipping 302 matching lines @@
   bool suppress_next_char_events_;
 
   bool pending_mouse_lock_request_;
   bool allow_privileged_mouse_lock_;
 
   // Keeps track of whether the webpage has any touch event handler. If it does,
   // then touch events are sent to the renderer. Otherwise, the touch events are
   // not sent to the renderer.
   bool has_touch_handler_;
 
+  bool is_in_gesture_scroll_;
+
   scoped_ptr<SyntheticGestureController> synthetic_gesture_controller_;
 
   scoped_ptr<TouchEmulator> touch_emulator_;
 
   // Receives and handles all input events.
   scoped_ptr<InputRouter> input_router_;
 
   scoped_ptr<TimeoutMonitor> hang_monitor_timeout_;
 
 #if defined(OS_WIN)
@@ skipping 17 matching lines @@
   bool is_focused_;
 
   base::WeakPtrFactory<RenderWidgetHostImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_