Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(270)

Unified Diff: extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc

Issue 376033002: Adding MimeHandlerView. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@pending-zork-patch2
Patch Set: Address comment from kenrb@ Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
diff --git a/extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc b/extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..41e076000f5b74660cef82ba4fa8b7c978f51ee4
--- /dev/null
+++ b/extensions/browser/guest_view/mime_handler_view/mime_handler_view_guest.cc
@@ -0,0 +1,97 @@
+// 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 "base/strings/stringprintf.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/render_process_host.h"
+#include "content/public/common/url_constants.h"
+#include "extensions/browser/guest_view/guest_view_constants.h"
+#include "extensions/browser/guest_view/guest_view_manager.h"
+#include "extensions/browser/guest_view/mime_handler_view/mime_handler_view_constants.h"
+#include "extensions/common/feature_switch.h"
+#include "extensions/strings/grit/extensions_strings.h"
+#include "net/base/url_util.h"
+
+using content::WebContents;
+
+namespace extensions {
+
+// static
+const char MimeHandlerViewGuest::Type[] = "mimehandler";
+
+// static
+GuestViewBase* MimeHandlerViewGuest::Create(
+ content::BrowserContext* browser_context,
+ int guest_instance_id) {
+ if (!extensions::FeatureSwitch::mime_handler_view()->IsEnabled())
+ return NULL;
+
+ return new MimeHandlerViewGuest(browser_context, guest_instance_id);
+}
+
+MimeHandlerViewGuest::MimeHandlerViewGuest(
+ content::BrowserContext* browser_context,
+ int guest_instance_id)
+ : GuestView<MimeHandlerViewGuest>(browser_context, guest_instance_id) {
+}
+
+MimeHandlerViewGuest::~MimeHandlerViewGuest() {
+}
+
+const char* MimeHandlerViewGuest::GetAPINamespace() const {
+ return "mimeHandlerViewGuestInternal";
+}
+
+int MimeHandlerViewGuest::GetTaskPrefix() const {
+ return IDS_EXTENSION_TASK_MANAGER_MIMEHANDLERVIEW_TAG_PREFIX;
+}
+
+void MimeHandlerViewGuest::CreateWebContents(
+ const std::string& embedder_extension_id,
+ int embedder_render_process_id,
+ const base::DictionaryValue& create_params,
+ const WebContentsCreatedCallback& callback) {
+ std::string orig_mime_type;
+ DCHECK(
+ create_params.GetString(mime_handler_view::kMimeType, &orig_mime_type) &&
+ !orig_mime_type.empty());
+ std::string guest_site_str;
+ // Note that we put a prefix "mime-" before the mime type so that this
+ // can never collide with an extension ID.
+ guest_site_str = base::StringPrintf(
+ "%s://mime-%s", content::kGuestScheme, orig_mime_type.c_str());
+ GURL guest_site(guest_site_str);
+
+ // If we already have a mime handler view for the same mime type, we should
+ // use the same SiteInstance so they go under same process.
+ GuestViewManager* guest_view_manager =
+ GuestViewManager::FromBrowserContext(browser_context());
+ 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 guests from different render process are not allowed to send
+ // messages to each other.
+ guest_site_instance =
+ content::SiteInstance::CreateForURL(browser_context(), guest_site);
+ }
+ WebContents::CreateParams params(browser_context(), guest_site_instance);
+ params.guest_delegate = this;
+ callback.Run(WebContents::Create(params));
+}
+
+void MimeHandlerViewGuest::DidAttachToEmbedder() {
+ std::string src;
+ DCHECK(attach_params()->GetString(mime_handler_view::kSrc, &src) &&
+ !src.empty());
+ web_contents()->GetController().LoadURL(
+ GURL(src),
+ content::Referrer(),
+ content::PAGE_TRANSITION_AUTO_TOPLEVEL,
+ std::string());
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698