OLD | NEW |
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "content/browser/browsing_instance.h" | 5 #include "content/browser/browsing_instance.h" |
6 | 6 |
7 #include "base/command_line.h" | 7 #include "base/command_line.h" |
8 #include "base/logging.h" | 8 #include "base/logging.h" |
9 #include "content/browser/browser_context.h" | 9 #include "content/browser/browser_context.h" |
10 #include "content/browser/site_instance.h" | 10 #include "content/browser/site_instance.h" |
11 #include "content/browser/webui/web_ui_factory.h" | 11 #include "content/browser/webui/web_ui_factory.h" |
12 #include "content/public/browser/content_browser_client.h" | 12 #include "content/public/browser/content_browser_client.h" |
13 #include "content/public/common/content_switches.h" | 13 #include "content/public/common/content_switches.h" |
14 #include "content/public/common/url_constants.h" | 14 #include "content/public/common/url_constants.h" |
15 | 15 |
16 // static | 16 // static |
17 BrowsingInstance::ContextSiteInstanceMap | 17 base::LazyInstance< |
18 BrowsingInstance::context_site_instance_map_; | 18 BrowsingInstance::ContextSiteInstanceMap, |
| 19 base::LeakyLazyInstanceTraits<BrowsingInstance::ContextSiteInstanceMap> > |
| 20 BrowsingInstance::context_site_instance_map_(base::LINKER_INITIALIZED); |
19 | 21 |
20 BrowsingInstance::BrowsingInstance(content::BrowserContext* browser_context) | 22 BrowsingInstance::BrowsingInstance(content::BrowserContext* browser_context) |
21 : browser_context_(browser_context) { | 23 : browser_context_(browser_context) { |
22 } | 24 } |
23 | 25 |
24 bool BrowsingInstance::ShouldUseProcessPerSite(const GURL& url) { | 26 bool BrowsingInstance::ShouldUseProcessPerSite(const GURL& url) { |
25 // Returns true if we should use the process-per-site model. This will be | 27 // Returns true if we should use the process-per-site model. This will be |
26 // the case if the --process-per-site switch is specified, or in | 28 // the case if the --process-per-site switch is specified, or in |
27 // process-per-site-instance for particular sites (e.g., the new tab page). | 29 // process-per-site-instance for particular sites (e.g., the new tab page). |
28 | 30 |
(...skipping 24 matching lines...) Expand all Loading... |
53 BrowsingInstance::SiteInstanceMap* BrowsingInstance::GetSiteInstanceMap( | 55 BrowsingInstance::SiteInstanceMap* BrowsingInstance::GetSiteInstanceMap( |
54 content::BrowserContext* browser_context, const GURL& url) { | 56 content::BrowserContext* browser_context, const GURL& url) { |
55 if (!ShouldUseProcessPerSite(SiteInstance::GetEffectiveURL(browser_context, | 57 if (!ShouldUseProcessPerSite(SiteInstance::GetEffectiveURL(browser_context, |
56 url))) { | 58 url))) { |
57 // Not using process-per-site, so use a map specific to this instance. | 59 // Not using process-per-site, so use a map specific to this instance. |
58 return &site_instance_map_; | 60 return &site_instance_map_; |
59 } | 61 } |
60 | 62 |
61 // Otherwise, process-per-site is in use, at least for this URL. Look up the | 63 // Otherwise, process-per-site is in use, at least for this URL. Look up the |
62 // global map for this context, creating an entry if necessary. | 64 // global map for this context, creating an entry if necessary. |
63 return &context_site_instance_map_[browser_context]; | 65 return &context_site_instance_map_.Get()[browser_context]; |
64 } | 66 } |
65 | 67 |
66 bool BrowsingInstance::HasSiteInstance(const GURL& url) { | 68 bool BrowsingInstance::HasSiteInstance(const GURL& url) { |
67 std::string site = | 69 std::string site = |
68 SiteInstance::GetSiteForURL(browser_context_, url) | 70 SiteInstance::GetSiteForURL(browser_context_, url) |
69 .possibly_invalid_spec(); | 71 .possibly_invalid_spec(); |
70 | 72 |
71 SiteInstanceMap* map = GetSiteInstanceMap(browser_context_, url); | 73 SiteInstanceMap* map = GetSiteInstanceMap(browser_context_, url); |
72 SiteInstanceMap::iterator i = map->find(site); | 74 SiteInstanceMap::iterator i = map->find(site); |
73 return (i != map->end()); | 75 return (i != map->end()); |
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
122 | 124 |
123 // We look for the site instance in both the local site_instance_map_ and also | 125 // We look for the site instance in both the local site_instance_map_ and also |
124 // the static context_site_instance_map_ - this is because the logic in | 126 // the static context_site_instance_map_ - this is because the logic in |
125 // ShouldUseProcessPerSite() can produce different results over the lifetime | 127 // ShouldUseProcessPerSite() can produce different results over the lifetime |
126 // of Chrome (e.g. installation of apps with web extents can change our | 128 // of Chrome (e.g. installation of apps with web extents can change our |
127 // process-per-site policy for a given domain), so we don't know which map | 129 // process-per-site policy for a given domain), so we don't know which map |
128 // the site was put into when it was originally registered. | 130 // the site was put into when it was originally registered. |
129 if (!RemoveSiteInstanceFromMap(&site_instance_map_, site, site_instance)) { | 131 if (!RemoveSiteInstanceFromMap(&site_instance_map_, site, site_instance)) { |
130 // Wasn't in our local map, so look in the static per-browser context map. | 132 // Wasn't in our local map, so look in the static per-browser context map. |
131 RemoveSiteInstanceFromMap( | 133 RemoveSiteInstanceFromMap( |
132 &context_site_instance_map_[browser_context_], site, site_instance); | 134 &context_site_instance_map_.Get()[browser_context_], |
| 135 site, |
| 136 site_instance); |
133 } | 137 } |
134 } | 138 } |
135 | 139 |
136 bool BrowsingInstance::RemoveSiteInstanceFromMap(SiteInstanceMap* map, | 140 bool BrowsingInstance::RemoveSiteInstanceFromMap(SiteInstanceMap* map, |
137 const std::string& site, | 141 const std::string& site, |
138 SiteInstance* site_instance) { | 142 SiteInstance* site_instance) { |
139 SiteInstanceMap::iterator i = map->find(site); | 143 SiteInstanceMap::iterator i = map->find(site); |
140 if (i != map->end() && i->second == site_instance) { | 144 if (i != map->end() && i->second == site_instance) { |
141 // Matches, so erase it. | 145 // Matches, so erase it. |
142 map->erase(i); | 146 map->erase(i); |
143 return true; | 147 return true; |
144 } | 148 } |
145 return false; | 149 return false; |
146 } | 150 } |
147 | 151 |
148 BrowsingInstance::~BrowsingInstance() { | 152 BrowsingInstance::~BrowsingInstance() { |
149 // We should only be deleted when all of the SiteInstances that refer to | 153 // We should only be deleted when all of the SiteInstances that refer to |
150 // us are gone. | 154 // us are gone. |
151 DCHECK(site_instance_map_.empty()); | 155 DCHECK(site_instance_map_.empty()); |
152 } | 156 } |
OLD | NEW |