Allow MimeHandlerViewGuest be embedded inside OOPIFs

content/public/browser/browser_plugin_guest_delegate.cc

Index: content/public/browser/browser_plugin_guest_delegate.cc
diff --git a/content/public/browser/browser_plugin_guest_delegate.cc b/content/public/browser/browser_plugin_guest_delegate.cc
index 732df238f91431509e6f998d56cc1508e9a0bdf3..affcdfb858bc3d8f53e3c7ed58d2757b23509354 100644
--- a/content/public/browser/browser_plugin_guest_delegate.cc
+++ b/content/public/browser/browser_plugin_guest_delegate.cc
@@ -36,4 +36,10 @@ bool BrowserPluginGuestDelegate::CanUseCrossProcessFrames() {
   return true;
 }
 
+RenderFrameHost* BrowserPluginGuestDelegate::GetEmbedderFrame() {
+  if (GetOwnerWebContents())
+    return GetOwnerWebContents()->GetMainFrame();

[Charlie Reis, 2016/11/02 20:47:07]

This code should probably be in GuestViewBase or an appropriate subclass.  This
class only has empty implementations of methods, not any real logic.

[EhsanK, 2016/11/08 23:10:16]

We make references to this method in BrowserPluginGuest which is in content/. We
do not depend on components/ as I understand so adding this to GuestViewBase
might be a layering violation (since GuestViewBase is a
BrowserPluginGuestDelegate I believe the right way is perhaps to add to BPGD and
implement in GVB).

I, however, moved the logic to GuestViewBase and return nullptr here.

+  return nullptr;
+}
+
 }  // namespace content