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

Unified Diff: chrome/browser/ui/extensions/hosted_app_browser_controller.cc

Issue 2478573004: Convert GURL::{host,path} to GURL::{host_piece,path_piece} for ==. (Closed)
Patch Set: rebase to #431874 Created 4 years, 1 month 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/ui/browser_navigator.cc ('k') | chrome/browser/ui/startup/startup_tab_provider.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/extensions/hosted_app_browser_controller.cc
diff --git a/chrome/browser/ui/extensions/hosted_app_browser_controller.cc b/chrome/browser/ui/extensions/hosted_app_browser_controller.cc
index 34b798ea87f4bb4456310aaa16384b6f985645af..1726f820a93a616fd2176a6751a870187602eaff 100644
--- a/chrome/browser/ui/extensions/hosted_app_browser_controller.cc
+++ b/chrome/browser/ui/extensions/hosted_app_browser_controller.cc
@@ -26,10 +26,10 @@ namespace {
bool IsSameOriginOrMoreSecure(const GURL& app_url, const GURL& page_url) {
const std::string www("www.");
- return (app_url.scheme() == page_url.scheme() ||
- page_url.scheme() == url::kHttpsScheme) &&
- (app_url.host() == page_url.host() ||
- www + app_url.host() == page_url.host()) &&
+ return (app_url.scheme_piece() == page_url.scheme_piece() ||
+ page_url.scheme_piece() == url::kHttpsScheme) &&
+ (app_url.host_piece() == page_url.host_piece() ||
+ www + app_url.host() == page_url.host_piece()) &&
app_url.port() == page_url.port();
}
« no previous file with comments | « chrome/browser/ui/browser_navigator.cc ('k') | chrome/browser/ui/startup/startup_tab_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698