Allow MimeHandlerViewGuest be embedded inside OOPIFs

content/browser/renderer_host/render_widget_host_input_event_router.cc

 // Copyright 2015 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/renderer_host/render_widget_host_input_event_router.h"
 
 #include <vector>
 
 #include "base/metrics/histogram_macros.h"
 
@@ skipping 179 matching lines @@
   } else {
     target = FindEventTarget(root_view, gfx::Point(event->x, event->y),
                              &transformed_point);
   }
 
   // RenderWidgetHostViewGuest does not properly handle direct routing of mouse
   // events, so they have to go by the double-hop forwarding path through
   // the embedding renderer and then BrowserPluginGuest.
   if (target && target->IsRenderWidgetHostViewGuest()) {
     ui::LatencyInfo latency_info;
-    root_view->ProcessMouseEvent(*event, latency_info);
+    RenderWidgetHostViewBase* embedder = target->GetOuterView();
+    if (embedder != root_view) {
+      // This happens when the view is embedded inside a cross process frame.

[nasko, 2016/10/27 21:03:40]

nit: cross-process

[EhsanK, 2016/10/27 22:06:48]

Acknowledged.

+      gfx::Point embedder_point;
+      if (!root_view->TransformPointToCoordSpaceForView(
+              gfx::Point(event->x, event->y), embedder, &embedder_point))

[nasko, 2016/10/27 21:03:40]

Two line if statement should have {}.

[EhsanK, 2016/10/27 22:06:48]

Done. But is it too lines in the condition or two lines in the "then" part?
Should I fix line 185 as well?

+        return;
+      event->x = embedder_point.x();
+      event->y = embedder_point.y();
+    }
+    embedder->ProcessMouseEvent(*event, latency_info);
     return;
   }
 
   if (event->type == blink::WebInputEvent::MouseDown)
     mouse_capture_target_.target = target;
 
   if (!target)
     return;
 
   // SendMouseEnterOrLeaveEvents is called with the original event
@@ skipping 537 matching lines @@
   if (!touchpad_gesture_target_.target)
     return;
 
   // TODO(mohsen): Add tests to check event location.
   event->x += touchpad_gesture_target_.delta.x();
   event->y += touchpad_gesture_target_.delta.y();
   touchpad_gesture_target_.target->ProcessGestureEvent(*event, latency);
 }
 
 }  // namespace content