[email protected] | 144a810 | 2012-01-14 01:05:31 | [diff] [blame] | 1 | // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
license.bot | bf09a50 | 2008-08-24 00:55:55 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 4 | |
[email protected] | b658359 | 2012-01-25 19:52:33 | [diff] [blame] | 5 | #include "content/browser/site_instance_impl.h" |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 6 | |
Lukasz Anforowicz | 48097c4 | 2017-12-15 00:23:38 | [diff] [blame] | 7 | #include <string> |
| 8 | |
creis | 0b49fa1 | 2017-07-10 16:31:11 | [diff] [blame] | 9 | #include "base/command_line.h" |
Alex Moshchuk | 4479b97c | 2017-10-17 23:20:32 | [diff] [blame] | 10 | #include "base/debug/crash_logging.h" |
alexmos | 3b9ad10 | 2017-05-26 23:41:08 | [diff] [blame] | 11 | #include "base/macros.h" |
[email protected] | 3936521 | 2011-02-24 01:01:00 | [diff] [blame] | 12 | #include "content/browser/browsing_instance.h" |
[email protected] | b953542 | 2012-02-09 01:47:59 | [diff] [blame] | 13 | #include "content/browser/child_process_security_policy_impl.h" |
[email protected] | c02f1ba | 2014-02-03 06:53:53 | [diff] [blame] | 14 | #include "content/browser/frame_host/debug_urls.h" |
nick | 9f34e289 | 2016-01-12 21:01:07 | [diff] [blame] | 15 | #include "content/browser/frame_host/frame_tree_node.h" |
[email protected] | f3b1a08 | 2011-11-18 00:34:30 | [diff] [blame] | 16 | #include "content/browser/renderer_host/render_process_host_impl.h" |
[email protected] | 4c3a2358 | 2012-08-18 08:54:34 | [diff] [blame] | 17 | #include "content/browser/storage_partition_impl.h" |
[email protected] | 87f3c08 | 2011-10-19 18:07:44 | [diff] [blame] | 18 | #include "content/public/browser/content_browser_client.h" |
[email protected] | f3b1a08 | 2011-11-18 00:34:30 | [diff] [blame] | 19 | #include "content/public/browser/render_process_host_factory.h" |
Nick Carter | bf6264a5 | 2018-04-06 02:39:33 | [diff] [blame] | 20 | #include "content/public/browser/site_isolation_policy.h" |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 21 | #include "content/public/browser/web_ui_controller_factory.h" |
creis | 0b49fa1 | 2017-07-10 16:31:11 | [diff] [blame] | 22 | #include "content/public/common/content_switches.h" |
[email protected] | a1d2916 | 2011-10-14 17:14:03 | [diff] [blame] | 23 | #include "content/public/common/url_constants.h" |
clamy | 7fced7b | 2017-11-16 19:52:43 | [diff] [blame] | 24 | #include "content/public/common/url_utils.h" |
[email protected] | be28b5f4 | 2012-07-20 11:31:25 | [diff] [blame] | 25 | #include "net/base/registry_controlled_domains/registry_controlled_domain.h" |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 26 | |
[email protected] | 4648832 | 2012-10-30 03:22:20 | [diff] [blame] | 27 | namespace content { |
[email protected] | b658359 | 2012-01-25 19:52:33 | [diff] [blame] | 28 | |
avi | b734894 | 2015-12-25 20:57:10 | [diff] [blame] | 29 | int32_t SiteInstanceImpl::next_site_instance_id_ = 1; |
[email protected] | 992db4c | 2011-05-12 15:37:15 | [diff] [blame] | 30 | |
Alex Moshchuk | 1656c67 | 2018-04-28 01:48:34 | [diff] [blame^] | 31 | using CheckOriginLockResult = |
| 32 | ChildProcessSecurityPolicyImpl::CheckOriginLockResult; |
| 33 | |
[email protected] | b658359 | 2012-01-25 19:52:33 | [diff] [blame] | 34 | SiteInstanceImpl::SiteInstanceImpl(BrowsingInstance* browsing_instance) |
[email protected] | 992db4c | 2011-05-12 15:37:15 | [diff] [blame] | 35 | : id_(next_site_instance_id_++), |
creis | cce56cd | 2014-09-29 22:45:22 | [diff] [blame] | 36 | active_frame_count_(0), |
[email protected] | 992db4c | 2011-05-12 15:37:15 | [diff] [blame] | 37 | browsing_instance_(browsing_instance), |
nick | d5bbd0b | 2016-03-31 19:52:44 | [diff] [blame] | 38 | process_(nullptr), |
| 39 | has_site_(false), |
Tsuyoshi Horo | eb576e6 | 2017-06-28 06:00:41 | [diff] [blame] | 40 | process_reuse_policy_(ProcessReusePolicy::DEFAULT), |
| 41 | is_for_service_worker_(false) { |
[email protected] | 4566f13 | 2009-03-12 01:55:13 | [diff] [blame] | 42 | DCHECK(browsing_instance); |
[email protected] | 4566f13 | 2009-03-12 01:55:13 | [diff] [blame] | 43 | } |
| 44 | |
[email protected] | b658359 | 2012-01-25 19:52:33 | [diff] [blame] | 45 | SiteInstanceImpl::~SiteInstanceImpl() { |
[email protected] | 4648832 | 2012-10-30 03:22:20 | [diff] [blame] | 46 | GetContentClient()->browser()->SiteInstanceDeleting(this); |
[email protected] | 056ad2a | 2011-07-12 02:13:55 | [diff] [blame] | 47 | |
[email protected] | 40bfaf3 | 2013-11-19 01:35:43 | [diff] [blame] | 48 | if (process_) |
| 49 | process_->RemoveObserver(this); |
| 50 | |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 51 | // Now that no one is referencing us, we can safely remove ourselves from |
| 52 | // the BrowsingInstance. Any future visits to a page from this site |
| 53 | // (within the same BrowsingInstance) can safely create a new SiteInstance. |
| 54 | if (has_site_) |
dcheng | bccd6b8 | 2016-03-30 16:24:19 | [diff] [blame] | 55 | browsing_instance_->UnregisterSiteInstance(this); |
| 56 | } |
| 57 | |
Nasko Oskov | deb6c7ea | 2017-11-30 18:18:11 | [diff] [blame] | 58 | // static |
dcheng | bccd6b8 | 2016-03-30 16:24:19 | [diff] [blame] | 59 | scoped_refptr<SiteInstanceImpl> SiteInstanceImpl::Create( |
| 60 | BrowserContext* browser_context) { |
kylechar | da69d88 | 2017-10-04 05:49:52 | [diff] [blame] | 61 | return base::WrapRefCounted( |
dcheng | bccd6b8 | 2016-03-30 16:24:19 | [diff] [blame] | 62 | new SiteInstanceImpl(new BrowsingInstance(browser_context))); |
| 63 | } |
| 64 | |
Nasko Oskov | deb6c7ea | 2017-11-30 18:18:11 | [diff] [blame] | 65 | // static |
dcheng | bccd6b8 | 2016-03-30 16:24:19 | [diff] [blame] | 66 | scoped_refptr<SiteInstanceImpl> SiteInstanceImpl::CreateForURL( |
| 67 | BrowserContext* browser_context, |
| 68 | const GURL& url) { |
| 69 | // This will create a new SiteInstance and BrowsingInstance. |
| 70 | scoped_refptr<BrowsingInstance> instance( |
| 71 | new BrowsingInstance(browser_context)); |
| 72 | return instance->GetSiteInstanceForURL(url); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 73 | } |
| 74 | |
Nasko Oskov | deb6c7ea | 2017-11-30 18:18:11 | [diff] [blame] | 75 | // static |
| 76 | bool SiteInstanceImpl::ShouldAssignSiteForURL(const GURL& url) { |
| 77 | // about:blank should not "use up" a new SiteInstance. The SiteInstance can |
| 78 | // still be used for a normal web site. |
| 79 | if (url == url::kAboutBlankURL) |
| 80 | return false; |
| 81 | |
| 82 | // The embedder will then have the opportunity to determine if the URL |
| 83 | // should "use up" the SiteInstance. |
| 84 | return GetContentClient()->browser()->ShouldAssignSiteForURL(url); |
| 85 | } |
| 86 | |
avi | b734894 | 2015-12-25 20:57:10 | [diff] [blame] | 87 | int32_t SiteInstanceImpl::GetId() { |
[email protected] | b658359 | 2012-01-25 19:52:33 | [diff] [blame] | 88 | return id_; |
| 89 | } |
| 90 | |
| 91 | bool SiteInstanceImpl::HasProcess() const { |
Ivan Kotenkov | 2c0d2bb3 | 2017-11-01 15:41:28 | [diff] [blame] | 92 | if (process_ != nullptr) |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 93 | return true; |
| 94 | |
| 95 | // If we would use process-per-site for this site, also check if there is an |
| 96 | // existing process that we would use if GetProcess() were called. |
[email protected] | 4648832 | 2012-10-30 03:22:20 | [diff] [blame] | 97 | BrowserContext* browser_context = |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 98 | browsing_instance_->browser_context(); |
| 99 | if (has_site_ && |
[email protected] | 1ae93fb1 | 2013-06-14 03:38:56 | [diff] [blame] | 100 | RenderProcessHost::ShouldUseProcessPerSite(browser_context, site_) && |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 101 | RenderProcessHostImpl::GetProcessHostForSite(browser_context, site_)) { |
| 102 | return true; |
| 103 | } |
| 104 | |
| 105 | return false; |
[email protected] | 5ab79b0 | 2010-04-26 16:47:11 | [diff] [blame] | 106 | } |
| 107 | |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 108 | RenderProcessHost* SiteInstanceImpl::GetProcess() { |
[email protected] | 08c8ec7e | 2010-07-11 17:14:48 | [diff] [blame] | 109 | // TODO(erikkay) It would be nice to ensure that the renderer type had been |
| 110 | // properly set before we get here. The default tab creation case winds up |
| 111 | // with no site set at this point, so it will default to TYPE_NORMAL. This |
| 112 | // may not be correct, so we'll wind up potentially creating a process that |
| 113 | // we then throw away, or worse sharing a process with the wrong process type. |
| 114 | // See crbug.com/43448. |
| 115 | |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 116 | // Create a new process if ours went away or was reused. |
[email protected] | 4566f13 | 2009-03-12 01:55:13 | [diff] [blame] | 117 | if (!process_) { |
[email protected] | 4c3a2358 | 2012-08-18 08:54:34 | [diff] [blame] | 118 | BrowserContext* browser_context = browsing_instance_->browser_context(); |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 119 | |
clamy | 63960c3 | 2017-05-10 22:57:07 | [diff] [blame] | 120 | // Check if the ProcessReusePolicy should be updated. |
| 121 | bool should_use_process_per_site = |
creis | a9f04ba | 2017-05-19 22:27:42 | [diff] [blame] | 122 | has_site_ && |
| 123 | RenderProcessHost::ShouldUseProcessPerSite(browser_context, site_); |
clamy | 63960c3 | 2017-05-10 22:57:07 | [diff] [blame] | 124 | if (should_use_process_per_site) { |
| 125 | process_reuse_policy_ = ProcessReusePolicy::PROCESS_PER_SITE; |
| 126 | } else if (process_reuse_policy_ == ProcessReusePolicy::PROCESS_PER_SITE) { |
| 127 | process_reuse_policy_ = ProcessReusePolicy::DEFAULT; |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 128 | } |
| 129 | |
clamy | 63960c3 | 2017-05-10 22:57:07 | [diff] [blame] | 130 | process_ = RenderProcessHostImpl::GetProcessHostForSiteInstance( |
| 131 | browser_context, this); |
avi | 6d686db | 2017-01-06 02:28:57 | [diff] [blame] | 132 | |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 133 | CHECK(process_); |
[email protected] | 40bfaf3 | 2013-11-19 01:35:43 | [diff] [blame] | 134 | process_->AddObserver(this); |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 135 | |
| 136 | // If we are using process-per-site, we need to register this process |
| 137 | // for the current site so that we can find it again. (If no site is set |
| 138 | // at this time, we will register it in SetSite().) |
clamy | 63960c3 | 2017-05-10 22:57:07 | [diff] [blame] | 139 | if (process_reuse_policy_ == ProcessReusePolicy::PROCESS_PER_SITE && |
| 140 | has_site_) { |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 141 | RenderProcessHostImpl::RegisterProcessHostForSite(browser_context, |
| 142 | process_, site_); |
| 143 | } |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 144 | |
nasko | b8744d2 | 2014-08-28 17:07:43 | [diff] [blame] | 145 | TRACE_EVENT2("navigation", "SiteInstanceImpl::GetProcess", |
| 146 | "site id", id_, "process id", process_->GetID()); |
[email protected] | 4648832 | 2012-10-30 03:22:20 | [diff] [blame] | 147 | GetContentClient()->browser()->SiteInstanceGotProcess(this); |
[email protected] | 6f37144 | 2011-11-09 06:45:46 | [diff] [blame] | 148 | |
[email protected] | 313b80bd | 2011-11-23 03:49:10 | [diff] [blame] | 149 | if (has_site_) |
alexmos | 13fe196 | 2017-06-28 04:25:12 | [diff] [blame] | 150 | LockToOriginIfNeeded(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 151 | } |
[email protected] | 4566f13 | 2009-03-12 01:55:13 | [diff] [blame] | 152 | DCHECK(process_); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 153 | |
[email protected] | 4566f13 | 2009-03-12 01:55:13 | [diff] [blame] | 154 | return process_; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 155 | } |
| 156 | |
[email protected] | b658359 | 2012-01-25 19:52:33 | [diff] [blame] | 157 | void SiteInstanceImpl::SetSite(const GURL& url) { |
nasko | b8744d2 | 2014-08-28 17:07:43 | [diff] [blame] | 158 | TRACE_EVENT2("navigation", "SiteInstanceImpl::SetSite", |
| 159 | "site id", id_, "url", url.possibly_invalid_spec()); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 160 | // A SiteInstance's site should not change. |
| 161 | // TODO(creis): When following links or script navigations, we can currently |
| 162 | // render pages from other sites in this SiteInstance. This will eventually |
| 163 | // be fixed, but until then, we should still not set the site of a |
| 164 | // SiteInstance more than once. |
| 165 | DCHECK(!has_site_); |
| 166 | |
| 167 | // Remember that this SiteInstance has been used to load a URL, even if the |
| 168 | // URL is invalid. |
| 169 | has_site_ = true; |
[email protected] | 4c3a2358 | 2012-08-18 08:54:34 | [diff] [blame] | 170 | BrowserContext* browser_context = browsing_instance_->browser_context(); |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 171 | site_ = GetSiteForURL(browser_context, url); |
Alex Moshchuk | 25c64bb | 2017-12-02 02:50:11 | [diff] [blame] | 172 | original_url_ = url; |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 173 | |
| 174 | // Now that we have a site, register it with the BrowsingInstance. This |
| 175 | // ensures that we won't create another SiteInstance for this site within |
| 176 | // the same BrowsingInstance, because all same-site pages within a |
| 177 | // BrowsingInstance can script each other. |
| 178 | browsing_instance_->RegisterSiteInstance(this); |
[email protected] | 313b80bd | 2011-11-23 03:49:10 | [diff] [blame] | 179 | |
clamy | 63960c3 | 2017-05-10 22:57:07 | [diff] [blame] | 180 | // Update the process reuse policy based on the site. |
| 181 | bool should_use_process_per_site = |
| 182 | RenderProcessHost::ShouldUseProcessPerSite(browser_context, site_); |
| 183 | if (should_use_process_per_site) { |
| 184 | process_reuse_policy_ = ProcessReusePolicy::PROCESS_PER_SITE; |
| 185 | } |
| 186 | |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 187 | if (process_) { |
alexmos | 13fe196 | 2017-06-28 04:25:12 | [diff] [blame] | 188 | LockToOriginIfNeeded(); |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 189 | |
| 190 | // Ensure the process is registered for this site if necessary. |
clamy | 63960c3 | 2017-05-10 22:57:07 | [diff] [blame] | 191 | if (should_use_process_per_site) { |
[email protected] | 41fb79a5 | 2012-06-29 16:34:33 | [diff] [blame] | 192 | RenderProcessHostImpl::RegisterProcessHostForSite( |
| 193 | browser_context, process_, site_); |
| 194 | } |
| 195 | } |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 196 | } |
| 197 | |
[email protected] | 77ab1731 | 2012-09-28 15:34:59 | [diff] [blame] | 198 | const GURL& SiteInstanceImpl::GetSiteURL() const { |
[email protected] | b658359 | 2012-01-25 19:52:33 | [diff] [blame] | 199 | return site_; |
| 200 | } |
| 201 | |
| 202 | bool SiteInstanceImpl::HasSite() const { |
| 203 | return has_site_; |
| 204 | } |
| 205 | |
| 206 | bool SiteInstanceImpl::HasRelatedSiteInstance(const GURL& url) { |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 207 | return browsing_instance_->HasSiteInstance(url); |
| 208 | } |
| 209 | |
dcheng | bccd6b8 | 2016-03-30 16:24:19 | [diff] [blame] | 210 | scoped_refptr<SiteInstance> SiteInstanceImpl::GetRelatedSiteInstance( |
| 211 | const GURL& url) { |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 212 | return browsing_instance_->GetSiteInstanceForURL(url); |
| 213 | } |
| 214 | |
[email protected] | 14392a5 | 2012-05-02 20:28:44 | [diff] [blame] | 215 | bool SiteInstanceImpl::IsRelatedSiteInstance(const SiteInstance* instance) { |
[email protected] | fc72bb1 | 2013-06-02 21:13:46 | [diff] [blame] | 216 | return browsing_instance_.get() == static_cast<const SiteInstanceImpl*>( |
| 217 | instance)->browsing_instance_.get(); |
[email protected] | 14392a5 | 2012-05-02 20:28:44 | [diff] [blame] | 218 | } |
| 219 | |
[email protected] | d5072a8 | 2014-05-15 05:50:18 | [diff] [blame] | 220 | size_t SiteInstanceImpl::GetRelatedActiveContentsCount() { |
| 221 | return browsing_instance_->active_contents_count(); |
| 222 | } |
| 223 | |
[email protected] | f88628d0 | 2012-11-11 17:58:59 | [diff] [blame] | 224 | bool SiteInstanceImpl::HasWrongProcessForURL(const GURL& url) { |
[email protected] | d292d8a | 2011-05-25 03:47:11 | [diff] [blame] | 225 | // Having no process isn't a problem, since we'll assign it correctly. |
[email protected] | f88628d0 | 2012-11-11 17:58:59 | [diff] [blame] | 226 | // Note that HasProcess() may return true if process_ is null, in |
| 227 | // process-per-site cases where there's an existing process available. |
| 228 | // We want to use such a process in the IsSuitableHost check, so we |
| 229 | // may end up assigning process_ in the GetProcess() call below. |
| 230 | if (!HasProcess()) |
[email protected] | d292d8a | 2011-05-25 03:47:11 | [diff] [blame] | 231 | return false; |
| 232 | |
[email protected] | 144a810 | 2012-01-14 01:05:31 | [diff] [blame] | 233 | // If the URL to navigate to can be associated with any site instance, |
| 234 | // we want to keep it in the same process. |
[email protected] | c02f1ba | 2014-02-03 06:53:53 | [diff] [blame] | 235 | if (IsRendererDebugURL(url)) |
[email protected] | 144a810 | 2012-01-14 01:05:31 | [diff] [blame] | 236 | return false; |
| 237 | |
alexmos | 13fe196 | 2017-06-28 04:25:12 | [diff] [blame] | 238 | // Any process can host an about:blank URL. This check avoids a process |
| 239 | // transfer for browser-initiated navigations to about:blank in a dedicated |
| 240 | // process; without it, IsSuitableHost would consider this process unsuitable |
| 241 | // for about:blank when it compares origin locks. Renderer-initiated |
| 242 | // navigations will handle about:blank navigations elsewhere and leave them |
| 243 | // in the source SiteInstance, along with about:srcdoc and data:. |
| 244 | if (url == url::kAboutBlankURL) |
| 245 | return false; |
| 246 | |
[email protected] | 88aae97 | 2011-12-16 01:14:18 | [diff] [blame] | 247 | // If the site URL is an extension (e.g., for hosted apps or WebUI) but the |
[email protected] | d292d8a | 2011-05-25 03:47:11 | [diff] [blame] | 248 | // process is not (or vice versa), make sure we notice and fix it. |
[email protected] | 2a5221b | 2011-09-27 23:07:31 | [diff] [blame] | 249 | GURL site_url = GetSiteForURL(browsing_instance_->browser_context(), url); |
[email protected] | 88aae97 | 2011-12-16 01:14:18 | [diff] [blame] | 250 | return !RenderProcessHostImpl::IsSuitableHost( |
[email protected] | f88628d0 | 2012-11-11 17:58:59 | [diff] [blame] | 251 | GetProcess(), browsing_instance_->browser_context(), site_url); |
[email protected] | d292d8a | 2011-05-25 03:47:11 | [diff] [blame] | 252 | } |
| 253 | |
nick | d5bbd0b | 2016-03-31 19:52:44 | [diff] [blame] | 254 | scoped_refptr<SiteInstanceImpl> |
| 255 | SiteInstanceImpl::GetDefaultSubframeSiteInstance() { |
| 256 | return browsing_instance_->GetDefaultSubframeSiteInstance(); |
| 257 | } |
| 258 | |
nick | d30fd96 | 2015-07-27 21:51:08 | [diff] [blame] | 259 | bool SiteInstanceImpl::RequiresDedicatedProcess() { |
| 260 | if (!has_site_) |
| 261 | return false; |
nick | d5bbd0b | 2016-03-31 19:52:44 | [diff] [blame] | 262 | |
| 263 | return DoesSiteRequireDedicatedProcess(GetBrowserContext(), site_); |
nick | d30fd96 | 2015-07-27 21:51:08 | [diff] [blame] | 264 | } |
| 265 | |
avi | 85cacb7 | 2016-10-26 22:39:33 | [diff] [blame] | 266 | bool SiteInstanceImpl::IsDefaultSubframeSiteInstance() const { |
clamy | 63960c3 | 2017-05-10 22:57:07 | [diff] [blame] | 267 | return process_reuse_policy_ == |
| 268 | ProcessReusePolicy::USE_DEFAULT_SUBFRAME_PROCESS; |
avi | 85cacb7 | 2016-10-26 22:39:33 | [diff] [blame] | 269 | } |
| 270 | |
nick | 9f34e289 | 2016-01-12 21:01:07 | [diff] [blame] | 271 | void SiteInstanceImpl::IncrementActiveFrameCount() { |
| 272 | active_frame_count_++; |
| 273 | } |
| 274 | |
| 275 | void SiteInstanceImpl::DecrementActiveFrameCount() { |
ericwilligers | 254597b | 2016-10-17 10:32:31 | [diff] [blame] | 276 | if (--active_frame_count_ == 0) { |
| 277 | for (auto& observer : observers_) |
| 278 | observer.ActiveFrameCountIsZero(this); |
| 279 | } |
nick | 9f34e289 | 2016-01-12 21:01:07 | [diff] [blame] | 280 | } |
| 281 | |
[email protected] | d5072a8 | 2014-05-15 05:50:18 | [diff] [blame] | 282 | void SiteInstanceImpl::IncrementRelatedActiveContentsCount() { |
| 283 | browsing_instance_->increment_active_contents_count(); |
| 284 | } |
| 285 | |
| 286 | void SiteInstanceImpl::DecrementRelatedActiveContentsCount() { |
| 287 | browsing_instance_->decrement_active_contents_count(); |
| 288 | } |
| 289 | |
nick | 9f34e289 | 2016-01-12 21:01:07 | [diff] [blame] | 290 | void SiteInstanceImpl::AddObserver(Observer* observer) { |
| 291 | observers_.AddObserver(observer); |
| 292 | } |
| 293 | |
| 294 | void SiteInstanceImpl::RemoveObserver(Observer* observer) { |
| 295 | observers_.RemoveObserver(observer); |
| 296 | } |
| 297 | |
[email protected] | 4c3a2358 | 2012-08-18 08:54:34 | [diff] [blame] | 298 | BrowserContext* SiteInstanceImpl::GetBrowserContext() const { |
[email protected] | 72daaa9 | 2012-01-18 13:39:02 | [diff] [blame] | 299 | return browsing_instance_->browser_context(); |
| 300 | } |
| 301 | |
nick | cc0d914 | 2015-10-14 16:27:10 | [diff] [blame] | 302 | // static |
dcheng | bccd6b8 | 2016-03-30 16:24:19 | [diff] [blame] | 303 | scoped_refptr<SiteInstance> SiteInstance::Create( |
| 304 | BrowserContext* browser_context) { |
| 305 | return SiteInstanceImpl::Create(browser_context); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 306 | } |
| 307 | |
nick | cc0d914 | 2015-10-14 16:27:10 | [diff] [blame] | 308 | // static |
dcheng | bccd6b8 | 2016-03-30 16:24:19 | [diff] [blame] | 309 | scoped_refptr<SiteInstance> SiteInstance::CreateForURL( |
| 310 | BrowserContext* browser_context, |
| 311 | const GURL& url) { |
| 312 | return SiteInstanceImpl::CreateForURL(browser_context, url); |
[email protected] | d8a9662 | 2009-05-07 19:21:16 | [diff] [blame] | 313 | } |
| 314 | |
nick | cc0d914 | 2015-10-14 16:27:10 | [diff] [blame] | 315 | // static |
Nasko Oskov | deb6c7ea | 2017-11-30 18:18:11 | [diff] [blame] | 316 | bool SiteInstance::ShouldAssignSiteForURL(const GURL& url) { |
| 317 | return SiteInstanceImpl::ShouldAssignSiteForURL(url); |
| 318 | } |
| 319 | |
| 320 | // static |
[email protected] | 399583b | 2012-12-11 09:33:42 | [diff] [blame] | 321 | bool SiteInstance::IsSameWebSite(BrowserContext* browser_context, |
[email protected] | 855d7d57 | 2014-08-02 11:18:17 | [diff] [blame] | 322 | const GURL& real_src_url, |
| 323 | const GURL& real_dest_url) { |
Alex Moshchuk | 25c64bb | 2017-12-02 02:50:11 | [diff] [blame] | 324 | return SiteInstanceImpl::IsSameWebSite(browser_context, real_src_url, |
| 325 | real_dest_url, true); |
| 326 | } |
| 327 | |
| 328 | bool SiteInstanceImpl::IsSameWebSite(BrowserContext* browser_context, |
| 329 | const GURL& real_src_url, |
| 330 | const GURL& real_dest_url, |
| 331 | bool should_compare_effective_urls) { |
| 332 | GURL src_url = |
| 333 | should_compare_effective_urls |
| 334 | ? SiteInstanceImpl::GetEffectiveURL(browser_context, real_src_url) |
| 335 | : real_src_url; |
| 336 | GURL dest_url = |
| 337 | should_compare_effective_urls |
| 338 | ? SiteInstanceImpl::GetEffectiveURL(browser_context, real_dest_url) |
| 339 | : real_dest_url; |
[email protected] | 399583b | 2012-12-11 09:33:42 | [diff] [blame] | 340 | |
| 341 | // We infer web site boundaries based on the registered domain name of the |
| 342 | // top-level page and the scheme. We do not pay attention to the port if |
| 343 | // one is present, because pages served from different ports can still |
| 344 | // access each other if they change their document.domain variable. |
| 345 | |
| 346 | // Some special URLs will match the site instance of any other URL. This is |
| 347 | // done before checking both of them for validity, since we want these URLs |
| 348 | // to have the same site instance as even an invalid one. |
[email protected] | 855d7d57 | 2014-08-02 11:18:17 | [diff] [blame] | 349 | if (IsRendererDebugURL(src_url) || IsRendererDebugURL(dest_url)) |
[email protected] | 399583b | 2012-12-11 09:33:42 | [diff] [blame] | 350 | return true; |
| 351 | |
| 352 | // If either URL is invalid, they aren't part of the same site. |
[email protected] | 855d7d57 | 2014-08-02 11:18:17 | [diff] [blame] | 353 | if (!src_url.is_valid() || !dest_url.is_valid()) |
[email protected] | 399583b | 2012-12-11 09:33:42 | [diff] [blame] | 354 | return false; |
| 355 | |
[email protected] | 855d7d57 | 2014-08-02 11:18:17 | [diff] [blame] | 356 | // If the destination url is just a blank page, we treat them as part of the |
| 357 | // same site. |
| 358 | GURL blank_page(url::kAboutBlankURL); |
| 359 | if (dest_url == blank_page) |
| 360 | return true; |
| 361 | |
Alex Moshchuk | a308c9b | 2018-02-08 20:58:14 | [diff] [blame] | 362 | // If the source and destination URLs are equal excluding the hash, they have |
| 363 | // the same site. This matters for file URLs, where SameDomainOrHost() would |
| 364 | // otherwise return false below. |
| 365 | if (src_url.EqualsIgnoringRef(dest_url)) |
| 366 | return true; |
| 367 | |
Daniel Cheng | 88186bd5 | 2017-10-20 08:14:46 | [diff] [blame] | 368 | url::Origin src_origin = url::Origin::Create(src_url); |
| 369 | url::Origin dest_origin = url::Origin::Create(dest_url); |
alexmos | 3b9ad10 | 2017-05-26 23:41:08 | [diff] [blame] | 370 | |
[email protected] | 399583b | 2012-12-11 09:33:42 | [diff] [blame] | 371 | // If the schemes differ, they aren't part of the same site. |
alexmos | cbf99578 | 2017-06-01 03:13:13 | [diff] [blame] | 372 | if (src_origin.scheme() != dest_origin.scheme()) |
[email protected] | 399583b | 2012-12-11 09:33:42 | [diff] [blame] | 373 | return false; |
| 374 | |
alexmos | 4bc2632 | 2017-07-01 00:57:14 | [diff] [blame] | 375 | if (!net::registry_controlled_domains::SameDomainOrHost( |
| 376 | src_origin, dest_origin, |
| 377 | net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES)) { |
| 378 | return false; |
| 379 | } |
| 380 | |
| 381 | // If the sites are the same, check isolated origins. If either URL matches |
| 382 | // an isolated origin, compare origins rather than sites. |
| 383 | auto* policy = ChildProcessSecurityPolicyImpl::GetInstance(); |
| 384 | url::Origin src_isolated_origin; |
| 385 | url::Origin dest_isolated_origin; |
| 386 | bool src_origin_is_isolated = |
| 387 | policy->GetMatchingIsolatedOrigin(src_origin, &src_isolated_origin); |
| 388 | bool dest_origin_is_isolated = |
| 389 | policy->GetMatchingIsolatedOrigin(dest_origin, &dest_isolated_origin); |
| 390 | if (src_origin_is_isolated || dest_origin_is_isolated) { |
| 391 | // Compare most specific matching origins to ensure that a subdomain of an |
| 392 | // isolated origin (e.g., https://subdomain.isolated.foo.com) also matches |
| 393 | // the isolated origin's site URL (e.g., https://isolated.foo.com). |
| 394 | return src_isolated_origin == dest_isolated_origin; |
| 395 | } |
| 396 | |
| 397 | return true; |
[email protected] | 399583b | 2012-12-11 09:33:42 | [diff] [blame] | 398 | } |
| 399 | |
nick | cc0d914 | 2015-10-14 16:27:10 | [diff] [blame] | 400 | // static |
[email protected] | 399583b | 2012-12-11 09:33:42 | [diff] [blame] | 401 | GURL SiteInstance::GetSiteForURL(BrowserContext* browser_context, |
| 402 | const GURL& real_url) { |
[email protected] | 25a9e8e53 | 2012-06-22 06:16:28 | [diff] [blame] | 403 | // TODO(fsamuel, creis): For some reason appID is not recognized as a host. |
[email protected] | 6eb1a11e | 2013-10-09 00:54:37 | [diff] [blame] | 404 | if (real_url.SchemeIs(kGuestScheme)) |
[email protected] | 25a9e8e53 | 2012-06-22 06:16:28 | [diff] [blame] | 405 | return real_url; |
| 406 | |
[email protected] | b658359 | 2012-01-25 19:52:33 | [diff] [blame] | 407 | GURL url = SiteInstanceImpl::GetEffectiveURL(browser_context, real_url); |
Daniel Cheng | 88186bd5 | 2017-10-20 08:14:46 | [diff] [blame] | 408 | url::Origin origin = url::Origin::Create(url); |
[email protected] | 3a8eecb | 2010-04-22 23:56:30 | [diff] [blame] | 409 | |
alexmos | 4bc2632 | 2017-07-01 00:57:14 | [diff] [blame] | 410 | // Isolated origins should use the full origin as their site URL. A subdomain |
Alex Moshchuk | bb99a33 | 2017-11-22 04:49:57 | [diff] [blame] | 411 | // of an isolated origin should also use that isolated origin's site URL. It |
| 412 | // is important to check |url| rather than |real_url| here, since some |
| 413 | // effective URLs (such as for NTP) need to be resolved prior to the isolated |
| 414 | // origin lookup. |
alexmos | 3b9ad10 | 2017-05-26 23:41:08 | [diff] [blame] | 415 | auto* policy = ChildProcessSecurityPolicyImpl::GetInstance(); |
alexmos | 4bc2632 | 2017-07-01 00:57:14 | [diff] [blame] | 416 | url::Origin isolated_origin; |
Lukasz Anforowicz | 217fd27 | 2018-03-07 21:41:43 | [diff] [blame] | 417 | if (policy->GetMatchingIsolatedOrigin(origin, &isolated_origin)) |
alexmos | 4bc2632 | 2017-07-01 00:57:14 | [diff] [blame] | 418 | return isolated_origin.GetURL(); |
alexmos | 3b9ad10 | 2017-05-26 23:41:08 | [diff] [blame] | 419 | |
Lukasz Anforowicz | 48097c4 | 2017-12-15 00:23:38 | [diff] [blame] | 420 | // If the url has a host, then determine the site. Skip file URLs to avoid a |
| 421 | // situation where site URL of file://localhost/ would mismatch Blink's origin |
| 422 | // (which ignores the hostname in this case - see https://crbug.com/776160). |
| 423 | if (!origin.host().empty() && origin.scheme() != url::kFileScheme) { |
nick | 1dd4792 | 2016-04-29 16:44:48 | [diff] [blame] | 424 | // Only keep the scheme and registered domain of |origin|. |
| 425 | std::string domain = net::registry_controlled_domains::GetDomainAndRegistry( |
| 426 | origin.host(), |
| 427 | net::registry_controlled_domains::INCLUDE_PRIVATE_REGISTRIES); |
| 428 | std::string site = origin.scheme(); |
| 429 | site += url::kStandardSchemeSeparator; |
| 430 | site += domain.empty() ? origin.host() : domain; |
| 431 | return GURL(site); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 432 | } |
creis | f60c2cd | 2014-12-18 00:41:02 | [diff] [blame] | 433 | |
| 434 | // If there is no host but there is a scheme, return the scheme. |
| 435 | // This is useful for cases like file URLs. |
Lukasz Anforowicz | 217fd27 | 2018-03-07 21:41:43 | [diff] [blame] | 436 | if (!origin.unique()) { |
| 437 | // Prefer to use the scheme of |origin| rather than |url|, to correctly |
| 438 | // cover blob: and filesystem: URIs (see also https://crbug.com/697111). |
Lukasz Anforowicz | d926a84 | 2018-03-09 01:50:45 | [diff] [blame] | 439 | DCHECK(!origin.scheme().empty()); |
| 440 | return GURL(origin.scheme() + ":"); |
| 441 | } else if (url.has_scheme()) { |
| 442 | DCHECK(!url.scheme().empty()); |
creis | f60c2cd | 2014-12-18 00:41:02 | [diff] [blame] | 443 | return GURL(url.scheme() + ":"); |
Lukasz Anforowicz | d926a84 | 2018-03-09 01:50:45 | [diff] [blame] | 444 | } |
creis | f60c2cd | 2014-12-18 00:41:02 | [diff] [blame] | 445 | |
| 446 | // Otherwise the URL should be invalid; return an empty site. |
Lukasz Anforowicz | 217fd27 | 2018-03-07 21:41:43 | [diff] [blame] | 447 | DCHECK(!url.is_valid()) << url; |
creis | f60c2cd | 2014-12-18 00:41:02 | [diff] [blame] | 448 | return GURL(); |
initial.commit | 09911bf | 2008-07-26 23:55:29 | [diff] [blame] | 449 | } |
| 450 | |
nick | cc0d914 | 2015-10-14 16:27:10 | [diff] [blame] | 451 | // static |
[email protected] | 4c3a2358 | 2012-08-18 08:54:34 | [diff] [blame] | 452 | GURL SiteInstanceImpl::GetEffectiveURL(BrowserContext* browser_context, |
| 453 | const GURL& url) { |
Alex Moshchuk | a31c788 | 2018-01-17 00:57:30 | [diff] [blame] | 454 | return GetContentClient()->browser()->GetEffectiveURL(browser_context, url); |
[email protected] | 3a8eecb | 2010-04-22 23:56:30 | [diff] [blame] | 455 | } |
| 456 | |
nick | cc0d914 | 2015-10-14 16:27:10 | [diff] [blame] | 457 | // static |
Alex Moshchuk | 25c64bb | 2017-12-02 02:50:11 | [diff] [blame] | 458 | bool SiteInstanceImpl::HasEffectiveURL(BrowserContext* browser_context, |
| 459 | const GURL& url) { |
| 460 | return GetEffectiveURL(browser_context, url) != url; |
| 461 | } |
| 462 | |
| 463 | // static |
nick | cc0d914 | 2015-10-14 16:27:10 | [diff] [blame] | 464 | bool SiteInstanceImpl::DoesSiteRequireDedicatedProcess( |
| 465 | BrowserContext* browser_context, |
nick | db193a1 | 2016-09-09 23:09:23 | [diff] [blame] | 466 | const GURL& url) { |
nick | cc0d914 | 2015-10-14 16:27:10 | [diff] [blame] | 467 | // If --site-per-process is enabled, site isolation is enabled everywhere. |
| 468 | if (SiteIsolationPolicy::UseDedicatedProcessesForAllSites()) |
| 469 | return true; |
| 470 | |
alexmos | 3b9ad10 | 2017-05-26 23:41:08 | [diff] [blame] | 471 | // Always require a dedicated process for isolated origins. |
| 472 | GURL site_url = GetSiteForURL(browser_context, url); |
| 473 | auto* policy = ChildProcessSecurityPolicyImpl::GetInstance(); |
Daniel Cheng | 88186bd5 | 2017-10-20 08:14:46 | [diff] [blame] | 474 | if (policy->IsIsolatedOrigin(url::Origin::Create(site_url))) |
alexmos | 3b9ad10 | 2017-05-26 23:41:08 | [diff] [blame] | 475 | return true; |
| 476 | |
nick | db193a1 | 2016-09-09 23:09:23 | [diff] [blame] | 477 | // Let the content embedder enable site isolation for specific URLs. Use the |
| 478 | // canonical site url for this check, so that schemes with nested origins |
| 479 | // (blob and filesystem) work properly. |
Avi Drissman | 8c36df7 | 2017-12-11 21:19:20 | [diff] [blame] | 480 | if (GetContentClient()->browser()->DoesSiteRequireDedicatedProcess( |
nick | db193a1 | 2016-09-09 23:09:23 | [diff] [blame] | 481 | browser_context, site_url)) { |
nick | cc0d914 | 2015-10-14 16:27:10 | [diff] [blame] | 482 | return true; |
| 483 | } |
| 484 | |
| 485 | return false; |
| 486 | } |
| 487 | |
alexmos | 13fe196 | 2017-06-28 04:25:12 | [diff] [blame] | 488 | // static |
| 489 | bool SiteInstanceImpl::ShouldLockToOrigin(BrowserContext* browser_context, |
Alex Moshchuk | 9cf5a45d | 2017-08-14 18:50:57 | [diff] [blame] | 490 | RenderProcessHost* host, |
alexmos | 13fe196 | 2017-06-28 04:25:12 | [diff] [blame] | 491 | GURL site_url) { |
creis | 0b49fa1 | 2017-07-10 16:31:11 | [diff] [blame] | 492 | // Don't lock to origin in --single-process mode, since this mode puts |
| 493 | // cross-site pages into the same process. |
Alex Moshchuk | 9cf5a45d | 2017-08-14 18:50:57 | [diff] [blame] | 494 | if (host->run_renderer_in_process()) |
creis | 0b49fa1 | 2017-07-10 16:31:11 | [diff] [blame] | 495 | return false; |
| 496 | |
alexmos | 13fe196 | 2017-06-28 04:25:12 | [diff] [blame] | 497 | if (!DoesSiteRequireDedicatedProcess(browser_context, site_url)) |
| 498 | return false; |
| 499 | |
| 500 | // Guest processes cannot be locked to their site because guests always have |
| 501 | // a fixed SiteInstance. The site of GURLs a guest loads doesn't match that |
| 502 | // SiteInstance. So we skip locking the guest process to the site. |
| 503 | // TODO(ncarter): Remove this exclusion once we can make origin lock per |
| 504 | // RenderFrame routing id. |
| 505 | if (site_url.SchemeIs(content::kGuestScheme)) |
| 506 | return false; |
| 507 | |
| 508 | // TODO(creis, nick) https://crbug.com/510588 Chrome UI pages use the same |
| 509 | // site (chrome://chrome), so they can't be locked because the site being |
| 510 | // loaded doesn't match the SiteInstance. |
| 511 | if (site_url.SchemeIs(content::kChromeUIScheme)) |
| 512 | return false; |
| 513 | |
| 514 | // TODO(creis, nick): Until we can handle sites with effective URLs at the |
| 515 | // call sites of ChildProcessSecurityPolicy::CanAccessDataForOrigin, we |
| 516 | // must give the embedder a chance to exempt some sites to avoid process |
| 517 | // kills. |
| 518 | if (!GetContentClient()->browser()->ShouldLockToOrigin(browser_context, |
| 519 | site_url)) { |
| 520 | return false; |
| 521 | } |
| 522 | |
| 523 | return true; |
| 524 | } |
| 525 | |
[email protected] | 40bfaf3 | 2013-11-19 01:35:43 | [diff] [blame] | 526 | void SiteInstanceImpl::RenderProcessHostDestroyed(RenderProcessHost* host) { |
| 527 | DCHECK_EQ(process_, host); |
| 528 | process_->RemoveObserver(this); |
nick | d5bbd0b | 2016-03-31 19:52:44 | [diff] [blame] | 529 | process_ = nullptr; |
[email protected] | 4566f13 | 2009-03-12 01:55:13 | [diff] [blame] | 530 | } |
[email protected] | 313b80bd | 2011-11-23 03:49:10 | [diff] [blame] | 531 | |
nick | 9f34e289 | 2016-01-12 21:01:07 | [diff] [blame] | 532 | void SiteInstanceImpl::RenderProcessWillExit(RenderProcessHost* host) { |
| 533 | // TODO(nick): http://crbug.com/575400 - RenderProcessWillExit might not serve |
| 534 | // any purpose here. |
ericwilligers | 254597b | 2016-10-17 10:32:31 | [diff] [blame] | 535 | for (auto& observer : observers_) |
| 536 | observer.RenderProcessGone(this); |
nick | 9f34e289 | 2016-01-12 21:01:07 | [diff] [blame] | 537 | } |
| 538 | |
Bo Liu | 2a48940 | 2018-04-24 23:41:27 | [diff] [blame] | 539 | void SiteInstanceImpl::RenderProcessExited( |
| 540 | RenderProcessHost* host, |
| 541 | const ChildProcessTerminationInfo& info) { |
ericwilligers | 254597b | 2016-10-17 10:32:31 | [diff] [blame] | 542 | for (auto& observer : observers_) |
| 543 | observer.RenderProcessGone(this); |
nick | 9f34e289 | 2016-01-12 21:01:07 | [diff] [blame] | 544 | } |
| 545 | |
alexmos | 13fe196 | 2017-06-28 04:25:12 | [diff] [blame] | 546 | void SiteInstanceImpl::LockToOriginIfNeeded() { |
| 547 | DCHECK(HasSite()); |
| 548 | |
| 549 | // From now on, this process should be considered "tainted" for future |
| 550 | // process reuse decisions: |
| 551 | // (1) If |site_| required a dedicated process, this SiteInstance's process |
| 552 | // can only host URLs for the same site. |
| 553 | // (2) Even if |site_| does not require a dedicated process, this |
| 554 | // SiteInstance's process still cannot be reused to host other sites |
| 555 | // requiring dedicated sites in the future. |
| 556 | // We can get here either when we commit a URL into a SiteInstance that does |
| 557 | // not yet have a site, or when we create a process for a SiteInstance with a |
| 558 | // preassigned site. |
| 559 | process_->SetIsUsed(); |
| 560 | |
Alex Moshchuk | 47dd00f | 2017-10-10 17:15:23 | [diff] [blame] | 561 | ChildProcessSecurityPolicyImpl* policy = |
| 562 | ChildProcessSecurityPolicyImpl::GetInstance(); |
| 563 | auto lock_state = policy->CheckOriginLock(process_->GetID(), site_); |
Alex Moshchuk | 9cf5a45d | 2017-08-14 18:50:57 | [diff] [blame] | 564 | if (ShouldLockToOrigin(GetBrowserContext(), process_, site_)) { |
Alex Moshchuk | 9cf5a45d | 2017-08-14 18:50:57 | [diff] [blame] | 565 | // Sanity check that this won't try to assign an origin lock to a <webview> |
| 566 | // process, which can't be locked. |
| 567 | CHECK(!process_->IsForGuestsOnly()); |
| 568 | |
Alex Moshchuk | 9cf5a45d | 2017-08-14 18:50:57 | [diff] [blame] | 569 | switch (lock_state) { |
Alex Moshchuk | 1656c67 | 2018-04-28 01:48:34 | [diff] [blame^] | 570 | case CheckOriginLockResult::NO_LOCK: { |
Alex Moshchuk | 47dd00f | 2017-10-10 17:15:23 | [diff] [blame] | 571 | // TODO(nick): When all sites are isolated, this operation provides |
| 572 | // strong protection. If only some sites are isolated, we need |
| 573 | // additional logic to prevent the non-isolated sites from requesting |
| 574 | // resources for isolated sites. https://crbug.com/509125 |
Alex Moshchuk | 9cf5a45d | 2017-08-14 18:50:57 | [diff] [blame] | 575 | policy->LockToOrigin(process_->GetID(), site_); |
| 576 | break; |
| 577 | } |
Alex Moshchuk | 1656c67 | 2018-04-28 01:48:34 | [diff] [blame^] | 578 | case CheckOriginLockResult::HAS_WRONG_LOCK: |
Alex Moshchuk | 9cf5a45d | 2017-08-14 18:50:57 | [diff] [blame] | 579 | // We should never attempt to reassign a different origin lock to a |
| 580 | // process. |
Robert Sesek | 1e07e37 | 2017-12-09 01:34:42 | [diff] [blame] | 581 | base::debug::SetCrashKeyString(bad_message::GetRequestedSiteURLKey(), |
| 582 | site_.spec()); |
| 583 | base::debug::SetCrashKeyString( |
| 584 | bad_message::GetKilledProcessOriginLockKey(), |
Alex Moshchuk | 4479b97c | 2017-10-17 23:20:32 | [diff] [blame] | 585 | policy->GetOriginLock(process_->GetID()).spec()); |
Alex Moshchuk | 54bf1304 | 2017-10-13 04:25:29 | [diff] [blame] | 586 | CHECK(false) << "Trying to lock a process to " << site_ |
| 587 | << " but the process is already locked to " |
| 588 | << policy->GetOriginLock(process_->GetID()); |
Alex Moshchuk | 9cf5a45d | 2017-08-14 18:50:57 | [diff] [blame] | 589 | break; |
Alex Moshchuk | 1656c67 | 2018-04-28 01:48:34 | [diff] [blame^] | 590 | case CheckOriginLockResult::HAS_EQUAL_LOCK: |
Alex Moshchuk | 9cf5a45d | 2017-08-14 18:50:57 | [diff] [blame] | 591 | // Process already has the right origin lock assigned. This case will |
| 592 | // happen for commits to |site_| after the first one. |
| 593 | break; |
| 594 | default: |
| 595 | NOTREACHED(); |
| 596 | } |
Alex Moshchuk | 47dd00f | 2017-10-10 17:15:23 | [diff] [blame] | 597 | } else { |
| 598 | // If the site that we've just committed doesn't require a dedicated |
| 599 | // process, make sure we aren't putting it in a process for a site that |
| 600 | // does. |
Alex Moshchuk | 1656c67 | 2018-04-28 01:48:34 | [diff] [blame^] | 601 | if (lock_state != CheckOriginLockResult::NO_LOCK) { |
| 602 | base::debug::SetCrashKeyString(bad_message::GetRequestedSiteURLKey(), |
| 603 | site_.spec()); |
| 604 | base::debug::SetCrashKeyString( |
| 605 | bad_message::GetKilledProcessOriginLockKey(), |
| 606 | policy->GetOriginLock(process_->GetID()).spec()); |
| 607 | CHECK(false) << "Trying to commit non-isolated site " << site_ |
| 608 | << " in process locked to " |
| 609 | << policy->GetOriginLock(process_->GetID()); |
| 610 | } |
[email protected] | 313b80bd | 2011-11-23 03:49:10 | [diff] [blame] | 611 | } |
| 612 | } |
[email protected] | 4648832 | 2012-10-30 03:22:20 | [diff] [blame] | 613 | |
| 614 | } // namespace content |