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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 9146028: Define the public interface for content browser SiteInstance. This interface is implemented by th... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 8 years, 11 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/chrome_content_browser_client.h ('k') | chrome/browser/chromeos/login/html_page_screen.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 118952)
+++ chrome/browser/chrome_content_browser_client.cc (working copy)
@@ -72,12 +72,12 @@
#include "content/browser/renderer_host/render_view_host.h"
#include "content/browser/renderer_host/resource_dispatcher_host.h"
#include "content/browser/resource_context.h"
-#include "content/browser/site_instance.h"
#include "content/browser/ssl/ssl_cert_error_handler.h"
#include "content/browser/ssl/ssl_client_auth_handler.h"
#include "content/browser/worker_host/worker_process_host.h"
#include "content/public/browser/browser_main_parts.h"
#include "content/public/browser/render_process_host.h"
+#include "content/public/browser/site_instance.h"
#include "content/public/browser/web_contents.h"
#include "content/public/browser/web_contents_view.h"
#include "grit/generated_resources.h"
@@ -134,6 +134,7 @@
using content::AccessTokenStore;
using content::BrowserThread;
+using content::SiteInstance;
using content::WebContents;
namespace {
@@ -486,20 +487,20 @@
const Extension* extension =
service->extensions()->GetExtensionOrAppByURL(ExtensionURLInfo(
- site_instance->site()));
+ site_instance->GetSite()));
if (!extension)
return;
service->process_map()->Insert(extension->id(),
site_instance->GetProcess()->GetID(),
- site_instance->id());
+ site_instance->GetId());
BrowserThread::PostTask(
BrowserThread::IO, FROM_HERE,
base::Bind(&ExtensionInfoMap::RegisterExtensionProcess,
profile->GetExtensionInfoMap(),
extension->id(),
site_instance->GetProcess()->GetID(),
- site_instance->id()));
+ site_instance->GetId()));
}
void ChromeContentBrowserClient::SiteInstanceDeleting(
@@ -515,20 +516,20 @@
const Extension* extension =
service->extensions()->GetExtensionOrAppByURL(
- ExtensionURLInfo(site_instance->site()));
+ ExtensionURLInfo(site_instance->GetSite()));
if (!extension)
return;
service->process_map()->Remove(extension->id(),
site_instance->GetProcess()->GetID(),
- site_instance->id());
+ site_instance->GetId());
BrowserThread::PostTask(
BrowserThread::IO, FROM_HERE,
base::Bind(&ExtensionInfoMap::UnregisterExtensionProcess,
profile->GetExtensionInfoMap(),
extension->id(),
site_instance->GetProcess()->GetID(),
- site_instance->id()));
+ site_instance->GetId()));
}
bool ChromeContentBrowserClient::ShouldSwapProcessesForNavigation(
@@ -1347,7 +1348,7 @@
ExtensionService* service = profile->GetExtensionService();
if (service) {
const Extension* extension = service->extensions()->GetByID(
- rvh->site_instance()->site().host());
+ rvh->site_instance()->GetSite().host());
extension_webkit_preferences::SetPreferences(
extension, rvh->delegate()->GetRenderViewType(), &web_prefs);
}
« no previous file with comments | « chrome/browser/chrome_content_browser_client.h ('k') | chrome/browser/chromeos/login/html_page_screen.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698