| Index: content/browser/tab_contents/render_view_host_manager.cc
|
| diff --git a/content/browser/tab_contents/render_view_host_manager.cc b/content/browser/tab_contents/render_view_host_manager.cc
|
| index 3386f24dec3eaf9817ba45bf5df3abc16228d32a..003f359d0c772e94d103447fbc3c53e17cce4e60 100644
|
| --- a/content/browser/tab_contents/render_view_host_manager.cc
|
| +++ b/content/browser/tab_contents/render_view_host_manager.cc
|
| @@ -56,14 +56,14 @@ RenderViewHostManager::~RenderViewHostManager() {
|
| }
|
| }
|
|
|
| -void RenderViewHostManager::Init(Profile* profile,
|
| +void RenderViewHostManager::Init(content::BrowserContext* browser_context,
|
| SiteInstance* site_instance,
|
| int routing_id) {
|
| // Create a RenderViewHost, once we have an instance. It is important to
|
| // immediately give this SiteInstance to a RenderViewHost so that it is
|
| // ref counted.
|
| if (!site_instance)
|
| - site_instance = SiteInstance::CreateSiteInstance(profile);
|
| + site_instance = SiteInstance::CreateSiteInstance(browser_context);
|
| render_view_host_ = RenderViewHostFactory::Create(
|
| site_instance, render_view_delegate_, routing_id, delegate_->
|
| GetControllerForRenderManager().session_storage_namespace());
|
| @@ -346,15 +346,17 @@ bool RenderViewHostManager::ShouldSwapProcessesForNavigation(
|
| // site, which might already be committed to a Web UI URL (such as the NTP).
|
| const GURL& current_url = (cur_entry) ? cur_entry->url() :
|
| render_view_host_->site_instance()->site();
|
| - Profile* profile = delegate_->GetControllerForRenderManager().profile();
|
| + content::BrowserContext* browser_context =
|
| + delegate_->GetControllerForRenderManager().browser_context();
|
| const content::WebUIFactory* web_ui_factory = content::WebUIFactory::Get();
|
| - if (web_ui_factory->UseWebUIForURL(profile, current_url)) {
|
| + if (web_ui_factory->UseWebUIForURL(browser_context, current_url)) {
|
| // Force swap if it's not an acceptable URL for Web UI.
|
| - if (!web_ui_factory->IsURLAcceptableForWebUI(profile, new_entry->url()))
|
| + if (!web_ui_factory->IsURLAcceptableForWebUI(browser_context,
|
| + new_entry->url()))
|
| return true;
|
| } else {
|
| // Force swap if it's a Web UI URL.
|
| - if (web_ui_factory->UseWebUIForURL(profile, new_entry->url()))
|
| + if (web_ui_factory->UseWebUIForURL(browser_context, new_entry->url()))
|
| return true;
|
| }
|
|
|
| @@ -393,7 +395,7 @@ SiteInstance* RenderViewHostManager::GetSiteInstanceForEntry(
|
|
|
| const GURL& dest_url = entry.url();
|
| NavigationController& controller = delegate_->GetControllerForRenderManager();
|
| - Profile* profile = controller.profile();
|
| + content::BrowserContext* browser_context = controller.browser_context();
|
|
|
| // If the entry has an instance already we should use it, unless the URL
|
| // is part of an app that has been installed or uninstalled since the last
|
| @@ -437,8 +439,9 @@ SiteInstance* RenderViewHostManager::GetSiteInstanceForEntry(
|
| // want to use the curr_instance if it has no site, since it will have a
|
| // RenderProcessHost of TYPE_NORMAL. Create a new SiteInstance for this
|
| // URL instead (with the correct process type).
|
| - if (content::WebUIFactory::Get()->UseWebUIForURL(profile, dest_url)) {
|
| - return SiteInstance::CreateSiteInstanceForURL(profile, dest_url);
|
| + if (content::WebUIFactory::Get()->UseWebUIForURL(browser_context,
|
| + dest_url)) {
|
| + return SiteInstance::CreateSiteInstanceForURL(browser_context, dest_url);
|
| }
|
|
|
| // Normally the "site" on the SiteInstance is set lazily when the load
|
| @@ -487,7 +490,7 @@ SiteInstance* RenderViewHostManager::GetSiteInstanceForEntry(
|
| // Use the current SiteInstance for same site navigations, as long as the
|
| // process type is correct. (The URL may have been installed as an app since
|
| // the last time we visited it.)
|
| - if (SiteInstance::IsSameWebSite(profile, current_url, dest_url) &&
|
| + if (SiteInstance::IsSameWebSite(browser_context, current_url, dest_url) &&
|
| !curr_instance->HasWrongProcessForURL(dest_url)) {
|
| return curr_instance;
|
| } else if (ShouldSwapProcessesForNavigation(curr_entry, &entry)) {
|
| @@ -497,7 +500,7 @@ SiteInstance* RenderViewHostManager::GetSiteInstanceForEntry(
|
| // Pages), keeping them in the same process. When you navigate away from
|
| // that page, we want to explicity ignore that BrowsingInstance and group
|
| // this page into the appropriate SiteInstance for its URL.
|
| - return SiteInstance::CreateSiteInstanceForURL(profile, dest_url);
|
| + return SiteInstance::CreateSiteInstanceForURL(browser_context, dest_url);
|
| } else {
|
| // Start the new renderer in a new SiteInstance, but in the current
|
| // BrowsingInstance. It is important to immediately give this new
|
|
|