Route touch-events for WebViewGuest directly to guest renderer.

content/browser/web_contents/web_contents_impl.cc

 // 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.
 
 #include "content/browser/web_contents/web_contents_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
@@ skipping 1575 matching lines @@
 #endif
   return false;
 }
 
 bool WebContentsImpl::PreHandleGestureEvent(
     const blink::WebGestureEvent& event) {
   return delegate_ && delegate_->PreHandleGestureEvent(this, event);
 }
 
 RenderWidgetHostInputEventRouter* WebContentsImpl::GetInputEventRouter() {
-  // Currently only supported in site per process mode (--site-per-process).
-  if (!rwh_input_event_router_.get() && !is_being_destroyed_ &&
-      SiteIsolationPolicy::AreCrossProcessFramesPossible())
+  if (!is_being_destroyed_ && GetOuterWebContents())
+    return GetOuterWebContents()->GetInputEventRouter();

[kenrb, 2015/11/05 17:05:26]

If we create an input event router before being attached to the outer
WebContents, does that persist as an unused object containing stale
SurfaceIdNamespace/RWHV entries?

It might not be a problem, as long as we can guarantee that those entries never
get referenced, but I would worry about the potential for bugs down the road if
we are leaving around stale state like that.

[wjmaclean, 2015/11/05 17:46:16]

Yes, at present we do leave these around. I thought about deleting existing
entries before adding the new ones, but we'd had to have a way of either
remembering or retrieving the *old* InputEventROuter, which would make
GetInputEventrouter() much more complicated (though we could have a separate
function like GetLocalInputEventRouter() that wouldn't defer to the
OuterWebContents if it exists, and use it to remove things?)

At present, the input event router doesn't guarantee that its map is empty at
destruction time, so it's conceivable we could have stale state around even
outside of this CL.

Let me know if you'd like me to go the removal route ...

+
+  if (!rwh_input_event_router_.get() && !is_being_destroyed_)
     rwh_input_event_router_.reset(new RenderWidgetHostInputEventRouter);
   return rwh_input_event_router_.get();
 }
 
 void WebContentsImpl::ReplicatePageFocus(bool is_focused) {
   // Focus loss may occur while this WebContents is being destroyed.  Don't
   // send the message in this case, as the main frame's RenderFrameHost and
   // other state has already been cleared.
   if (is_being_destroyed_)
     return;
@@ skipping 3070 matching lines @@
   return NULL;
 }
 
 void WebContentsImpl::SetForceDisableOverscrollContent(bool force_disable) {
   force_disable_overscroll_content_ = force_disable;
   if (view_)
     view_->SetOverscrollControllerEnabled(CanOverscrollContent());
 }
 
 }  // namespace content