Navigation: Replace hardcoded kAboutBlankURL checks with IsAboutBlank(). This is to use the more reliable GURL::IsAboutBlank() when checking for blank pages instead of just matching the urls. Change-Id: Ieb5692cf1da8e323baa2e72285d2801d2fc0d1ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1583786 Reviewed-by: Nasko Oskov <nasko@chromium.org> Commit-Queue: Mohamed Abdelhalim <zetamoo@chromium.org> Cr-Commit-Position: refs/heads/master@{#654066}
diff --git a/content/browser/site_instance_impl.cc b/content/browser/site_instance_impl.cc index 6bb834f7..ca5571f 100644 --- a/content/browser/site_instance_impl.cc +++ b/content/browser/site_instance_impl.cc
@@ -86,7 +86,7 @@ bool SiteInstanceImpl::ShouldAssignSiteForURL(const GURL& url) { // about:blank should not "use up" a new SiteInstance. The SiteInstance can // still be used for a normal web site. - if (url == url::kAboutBlankURL) + if (url.IsAboutBlank()) return false; // The embedder will then have the opportunity to determine if the URL @@ -453,8 +453,7 @@ // If the destination url is just a blank page, we treat them as part of the // same site. - GURL blank_page(url::kAboutBlankURL); - if (dest_url == blank_page) + if (dest_url.IsAboutBlank()) return true; // If the source and destination URLs are equal excluding the hash, they have