Adding MimeHandlerView.

chrome/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc

Index: chrome/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
diff --git a/chrome/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc b/chrome/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..39057c3380c59eb956ae70659f34a5423cf8e9a4
--- /dev/null
+++ b/chrome/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
@@ -0,0 +1,106 @@
+// 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 "chrome/browser/guest_view/mime_handler_view/mime_handler_view_guest.h"
+
+#include "base/strings/stringprintf.h"
+#include "chrome/browser/extensions/chrome_extension_web_contents_observer.h"
+#include "chrome/browser/guest_view/guest_view_manager.h"
+#include "chrome/browser/guest_view/web_view/web_view_renderer_state.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/render_process_host.h"
+#include "content/public/common/url_constants.h"
+
+using content::WebContents;
+
+// static
+const char MimeHandlerViewGuest::Type[] = "mimehandler";
+
+MimeHandlerViewGuest::MimeHandlerViewGuest(
+    content::BrowserContext* browser_context,
+    int guest_instance_id)
+    : GuestView<MimeHandlerViewGuest>(
+        browser_context, guest_instance_id),
+      has_navigated_(false) {

[Fady Samuel, 2014/07/21 14:47:35]

Get rid of this flag.

[lazyboy, 2014/07/21 17:28:21]

Done.

+}
+
+MimeHandlerViewGuest::~MimeHandlerViewGuest() {
+}
+
+bool MimeHandlerViewGuest::CanEmbedderUseGuestView(
+    const std::string& embedder_extension_id) {
+  return true;
+}
+void MimeHandlerViewGuest::CreateWebContents(
+    const std::string& embedder_extension_id,
+    int embedder_render_process_id,
+    const base::DictionaryValue& create_params,
+    const WebContentsCreatedCallback& callback) {
+  content::RenderProcessHost* embedder_render_process_host =

[Fady Samuel, 2014/07/21 14:47:35]

This doesn't seem necessary, you can simply use browser_context().

[lazyboy, 2014/07/21 17:28:21]

Done.

+      content::RenderProcessHost::FromID(embedder_render_process_id);
+
+  GURL guest_site(base::StringPrintf("%s://%s/",
+                                     content::kGuestScheme,
+                                     embedder_extension_id.c_str()));
+
+  // If we already have a webview tag in the same app using the same storage
+  // partition, we should use the same SiteInstance so the existing tag and
+  // the new tag can script each other.
+  GuestViewManager* guest_view_manager =
+      GuestViewManager::FromBrowserContext(
+          embedder_render_process_host->GetBrowserContext());
+  content::SiteInstance* guest_site_instance =
+      guest_view_manager->GetGuestSiteInstance(guest_site);
+  if (!guest_site_instance) {
+    // Create the SiteInstance in a new BrowsingInstance, which will ensure
+    // that webview tags are also not allowed to send messages across

[Fady Samuel, 2014/07/21 14:47:35]

This comment doesn't make sense for MimeHandlerViewGuest.

[lazyboy, 2014/07/21 17:28:21]

I've reworded the comments.

+    // different partitions.
+    guest_site_instance = content::SiteInstance::CreateForURL(
+        embedder_render_process_host->GetBrowserContext(), guest_site);
+  }
+  WebContents::CreateParams params(
+      embedder_render_process_host->GetBrowserContext(),
+      guest_site_instance);
+  params.guest_delegate = this;
+  callback.Run(WebContents::Create(params));
+}
+
+void MimeHandlerViewGuest::DidAttachToEmbedder() {
+  has_navigated_ = false;

[Fady Samuel, 2014/07/21 14:47:35]

This flag seems unnecessary.

[lazyboy, 2014/07/21 17:28:21]

Done.

+  std::string src;
+  if (extra_params()->GetString("src", &src) && !src.empty())

[Fady Samuel, 2014/07/21 14:47:35]

Make src a constant.

[lazyboy, 2014/07/21 17:28:20]

Done.

+    NavigateGuest(src);
+
+  DCHECK(!GetOpener());

[Fady Samuel, 2014/07/21 14:47:35]

Why is this DCHECK important?

[lazyboy, 2014/07/21 17:28:20]

It's not, copy paste stuff, removed.

+}
+
+void MimeHandlerViewGuest::DidInitialize() {
+  AttachWebContentsHelpers(guest_web_contents());
+}
+
+void MimeHandlerViewGuest::AttachWebContentsHelpers(
+    content::WebContents* contents) {
+  // Required for ExtensionHostMsg_PostMessage.
+  extensions::ChromeExtensionWebContentsObserver::CreateForWebContents(
+      contents);
+}
+
+void MimeHandlerViewGuest::NavigateGuest(const std::string& src) {
+  if (has_navigated_)
+    return;
+
+  has_navigated_ = true;
+  GURL validated_url(src);
+  guest_web_contents()->GetRenderProcessHost()->
+      FilterURL(false, &validated_url);
+  // As guests do not swap processes on navigation, only navigations to
+  // normal web URLs are supported.  No protocol handlers are installed for
+  // other schemes (e.g., WebUI or extensions), and no permissions or bindings
+  // can be granted to the guest process.
+  content::NavigationController::LoadURLParams load_url_params(validated_url);
+  load_url_params.referrer = content::Referrer();
+  load_url_params.transition_type = content::PAGE_TRANSITION_AUTO_TOPLEVEL;
+  load_url_params.extra_headers = std::string();
+  guest_web_contents()->GetController().LoadURLWithParams(load_url_params);

[Fady Samuel, 2014/07/21 14:47:35]

This seems unnecessary. Why can't it be something as simple as this:

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/gue...

[lazyboy, 2014/07/21 17:28:21]

That's simpler, thanks.
Done.

+}