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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 6880089: Don't hold the installed app in BrowserProcessRenderHost, since that's in content layer now. Ext... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix browser tests Created 9 years, 8 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/automation/automation_util.cc ('k') | chrome/browser/extensions/extension_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chrome_content_browser_client.cc
===================================================================
--- chrome/browser/chrome_content_browser_client.cc (revision 82593)
+++ chrome/browser/chrome_content_browser_client.cc (working copy)
@@ -37,8 +37,10 @@
render_view_host->set_is_extension_process(is_extension_process);
const Extension* installed_app = service->GetInstalledApp(url);
- static_cast<BrowserRenderProcessHost*>(render_view_host->process())->
- set_installed_app(installed_app);
+ if (installed_app) {
+ service->SetInstalledAppForRenderer(
+ render_view_host->process()->id(), installed_app);
+ }
}
}
@@ -47,8 +49,7 @@
host->channel()->AddFilter(new ChromeRenderMessageFilter(
host->id(),
host->profile(),
- host->profile()->GetRequestContextForPossibleApp(
- host->installed_app())));
+ host->profile()->GetRequestContextForRenderProcess(host->id())));
host->channel()->AddFilter(new PrintingMessageFilter());
host->channel()->AddFilter(
new SearchProviderInstallStateMessageFilter(host->id(), host->profile()));
« no previous file with comments | « chrome/browser/automation/automation_util.cc ('k') | chrome/browser/extensions/extension_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698