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

Unified Diff: chrome/browser/guest_view/extension_options/extension_options_guest.cc

Issue 546073003: Embedded Extension Options: Open all external links in a new tab (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add comments 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
« no previous file with comments | « chrome/browser/guest_view/extension_options/extension_options_guest.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/guest_view/extension_options/extension_options_guest.cc
diff --git a/chrome/browser/guest_view/extension_options/extension_options_guest.cc b/chrome/browser/guest_view/extension_options/extension_options_guest.cc
index 117f562d8bdb58388a8e6559f8331f2fb9f1a899..f3d5f6ef0887d3048646e3b2733afcd0660459b1 100644
--- a/chrome/browser/guest_view/extension_options/extension_options_guest.cc
+++ b/chrome/browser/guest_view/extension_options/extension_options_guest.cc
@@ -24,6 +24,7 @@
#include "extensions/browser/extension_function_dispatcher.h"
#include "extensions/browser/extension_registry.h"
#include "extensions/browser/guest_view/guest_view_manager.h"
+#include "extensions/common/constants.h"
#include "extensions/common/extension.h"
#include "extensions/common/extension_messages.h"
#include "extensions/common/feature_switch.h"
@@ -155,6 +156,28 @@ content::WebContents* ExtensionOptionsGuest::GetAssociatedWebContents() const {
return web_contents();
}
+content::WebContents* ExtensionOptionsGuest::OpenURLFromTab(
+ content::WebContents* source,
+ const content::OpenURLParams& params) {
+ Browser* browser =
+ chrome::FindBrowserWithWebContents(embedder_web_contents());
+
+ // Don't allow external URLs with the CURRENT_TAB disposition be opened in
+ // this guest view, change the disposition to NEW_FOREGROUND_TAB.
+ if ((!params.url.SchemeIs(extensions::kExtensionScheme) ||
+ params.url.host() != options_page_.host()) &&
+ params.disposition == CURRENT_TAB) {
+ return browser->OpenURL(
+ content::OpenURLParams(params.url,
+ params.referrer,
+ params.frame_tree_node_id,
+ NEW_FOREGROUND_TAB,
+ params.transition,
+ params.is_renderer_initiated));
+ }
+ return browser->OpenURL(params);
+}
+
void ExtensionOptionsGuest::CloseContents(content::WebContents* source) {
DispatchEventToEmbedder(new extensions::GuestViewBase::Event(
extension_options_internal::OnClose::kEventName,
« no previous file with comments | « chrome/browser/guest_view/extension_options/extension_options_guest.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698