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

Unified Diff: url/url_canon_stdurl.cc

Issue 2403713002: Add suborigin logic to url::Origin (Closed)
Patch Set: Minor simplification Created 4 years, 2 months 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
Index: url/url_canon_stdurl.cc
diff --git a/url/url_canon_stdurl.cc b/url/url_canon_stdurl.cc
index 7d1758b1299987f9f1a20a32f997e1a374c430f2..7f9216d29cf7f4c4554118b5ba741f2a7d38cb98 100644
--- a/url/url_canon_stdurl.cc
+++ b/url/url_canon_stdurl.cc
@@ -120,6 +120,14 @@ int DefaultPortForScheme(const char* scheme, int scheme_len) {
if (!strncmp(scheme, kWsScheme, scheme_len))
default_port = 80;
break;
+ case 7:
+ if (!strncmp(scheme, kHttpSuboriginScheme, scheme_len))
+ default_port = 80;
+ break;
+ case 8:
+ if (!strncmp(scheme, kHttpsSuboriginScheme, scheme_len))
+ default_port = 80;
Mike West 2016/10/11 07:40:41 443? Both of these should have tests as well.
jww 2016/10/11 18:16:27 Done.
+ break;
}
return default_port;
}
« url/origin_unittest.cc ('K') | « url/origin_unittest.cc ('k') | url/url_constants.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698