Allow out-of-process iframes to update the mouse cursor

content/browser/frame_host/cross_process_frame_connector.cc

Index: content/browser/frame_host/cross_process_frame_connector.cc
diff --git a/content/browser/frame_host/cross_process_frame_connector.cc b/content/browser/frame_host/cross_process_frame_connector.cc
index 7fb24a48369efd96ff93e12856228f36584d5684..be3b39735fbdd480947d93753d5ff6ee79f51e2b 100644
--- a/content/browser/frame_host/cross_process_frame_connector.cc
+++ b/content/browser/frame_host/cross_process_frame_connector.cc
@@ -7,6 +7,7 @@
 #include "cc/surfaces/surface.h"
 #include "cc/surfaces/surface_manager.h"
 #include "content/browser/compositor/surface_utils.h"
+#include "content/browser/frame_host/frame_tree.h"
 #include "content/browser/frame_host/frame_tree_node.h"
 #include "content/browser/frame_host/render_frame_host_manager.h"
 #include "content/browser/frame_host/render_frame_proxy_host.h"
@@ -163,6 +164,12 @@ void CrossProcessFrameConnector::GetScreenInfo(blink::WebScreenInfo* results) {
     static_cast<RenderWidgetHostViewBase*>(rwhv)->GetScreenInfo(results);
 }
 
+void CrossProcessFrameConnector::UpdateCursor(const WebCursor& cursor) {
+  RenderWidgetHostViewBase* root_view = GetRootRenderWidgetHostView();
+  if (root_view)
+    root_view->UpdateCursor(cursor);
+}
+
 void CrossProcessFrameConnector::OnForwardInputEvent(
     const blink::WebInputEvent* event) {
   if (!view_)
@@ -218,4 +225,14 @@ void CrossProcessFrameConnector::SetSize(gfx::Rect frame_rect) {
     view_->SetSize(frame_rect.size());
 }
 
+RenderWidgetHostViewBase*
+CrossProcessFrameConnector::GetRootRenderWidgetHostView() {
+  return static_cast<RenderWidgetHostViewBase*>(
+      frame_proxy_in_parent_renderer_->frame_tree_node()
+          ->frame_tree()
+          ->root()
+          ->current_frame_host()
+          ->GetView());

[nasko, 2015/10/15 17:06:54]

What happens with <webview> where the root RWHV is not at the inner WebContents,
but the outer? I think in that case we have a RWHVChildFrame, which will do
another hop through its own frame connector, right?

[kenrb, 2015/10/15 17:12:54]

Yes.

+}
+
 }  // namespace content