Allow MimeHandlerViewGuest be embedded inside OOPIFs

extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc

 // Copyright 2014 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 "extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.h"
 
 #include <utility>
 
 #include "base/strings/stringprintf.h"
 #include "components/guest_view/common/guest_view_constants.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/host_zoom_map.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/stream_handle.h"
 #include "content/public/browser/stream_info.h"
+#include "content/public/common/child_process_host.h"
 #include "content/public/common/url_constants.h"
 #include "extensions/browser/api/extensions_api_client.h"
 #include "extensions/browser/api/mime_handler_private/mime_handler_private.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/guest_view/mime_handler_view/mime_handler_stream_manager.h"
 #include "extensions/browser/guest_view/mime_handler_view/mime_handler_view_constants.h"
 #include "extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest_delegate.h"
 #include "extensions/browser/process_manager.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension_messages.h"
@@ skipping 42 matching lines @@
 // static
 const char MimeHandlerViewGuest::Type[] = "mimehandler";
 
 // static
 GuestViewBase* MimeHandlerViewGuest::Create(WebContents* owner_web_contents) {
   return new MimeHandlerViewGuest(owner_web_contents);
 }
 
 MimeHandlerViewGuest::MimeHandlerViewGuest(WebContents* owner_web_contents)
     : GuestView<MimeHandlerViewGuest>(owner_web_contents),
-      delegate_(ExtensionsAPIClient::Get()->CreateMimeHandlerViewGuestDelegate(
-          this)) {}
+      delegate_(
+          ExtensionsAPIClient::Get()->CreateMimeHandlerViewGuestDelegate(this)),
+      embedder_frame_process_id_(content::ChildProcessHost::kInvalidUniqueID),
+      embedder_frame_routing_id_(MSG_ROUTING_NONE) {}
 
 MimeHandlerViewGuest::~MimeHandlerViewGuest() {
 }
 
 bool MimeHandlerViewGuest::CanUseCrossProcessFrames() {
   return false;
 }
 
+content::RenderFrameHost* MimeHandlerViewGuest::GetEmbedderFrame() {
+  if (embedder_frame_routing_id_ != MSG_ROUTING_NONE)

[lfg, 2016/10/20 22:50:49]

This should be a DCHECK.

[EhsanK, 2016/10/25 19:30:51]

Acknowledged.

+    return content::RenderFrameHost::FromID(embedder_frame_process_id_,
+                                            embedder_frame_routing_id_);
+  return GuestViewBase::GetEmbedderFrame();

[lfg, 2016/10/20 22:50:50]

And this should be NOTREACHED().

[EhsanK, 2016/10/25 19:30:51]

I guess only DCHECK would do then and I removed this line.

+}
+
+void MimeHandlerViewGuest::SetEmbedderFrame(int process_id, int routing_id) {
+  DCHECK_NE(routing_id, MSG_ROUTING_NONE);
+  DCHECK_EQ(embedder_frame_routing_id_, MSG_ROUTING_NONE);
+
+  embedder_frame_process_id_ = process_id;
+  embedder_frame_routing_id_ = routing_id;
+}
+
 const char* MimeHandlerViewGuest::GetAPINamespace() const {
   return "mimeHandlerViewGuestInternal";
 }
 
 int MimeHandlerViewGuest::GetTaskPrefix() const {
   return IDS_EXTENSION_TASK_MANAGER_MIMEHANDLERVIEW_TAG_PREFIX;
 }
 
 void MimeHandlerViewGuest::CreateWebContents(
     const base::DictionaryValue& create_params,
@@ skipping 150 matching lines @@
           element_instance_id()));
 }
 
 base::WeakPtr<StreamContainer> MimeHandlerViewGuest::GetStream() const {
   if (!stream_)
     return base::WeakPtr<StreamContainer>();
   return stream_->GetWeakPtr();
 }
 
 }  // namespace extensions