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

Unified Diff: extensions/browser/extension_web_contents_observer.cc

Issue 1225413002: [Extensions] Fix improper extension tab ownership bug (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 5 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/extensions/content_script_apitest.cc ('k') | extensions/common/extension_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/extension_web_contents_observer.cc
diff --git a/extensions/browser/extension_web_contents_observer.cc b/extensions/browser/extension_web_contents_observer.cc
index a36661b2c26e60aa434ac9aec093821734e41c19..187cfb22a95ead9a7d997d18aefe30ccd39cb71c 100644
--- a/extensions/browser/extension_web_contents_observer.cc
+++ b/extensions/browser/extension_web_contents_observer.cc
@@ -61,17 +61,6 @@ void ExtensionWebContentsObserver::InitializeRenderFrame(
->RegisterRenderFrameHost(web_contents(), render_frame_host,
frame_extension);
}
-
- // This can be different from |frame_extension| above in the case of, e.g.,
- // a non-extension iframe hosted in a chrome-extension:// page.
- const Extension* main_frame_extension =
- GetExtensionFromFrame(web_contents()->GetMainFrame());
- // We notify the render frame that it's in an extension's tab, but not if this
- // is a hosted app (we don't mind scripting on hosted apps' pages).
- if (main_frame_extension && !main_frame_extension->is_hosted_app()) {
- render_frame_host->Send(new ExtensionMsg_SetMainFrameExtensionOwner(
- render_frame_host->GetRoutingID(), main_frame_extension->id()));
- }
}
content::WebContents* ExtensionWebContentsObserver::GetAssociatedWebContents()
« no previous file with comments | « chrome/browser/extensions/content_script_apitest.cc ('k') | extensions/common/extension_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698