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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/http/http_auth_filter.cc

Issue 1151843002: DO NOT LAND Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: More. Created 5 years, 7 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
« net/http/http_auth_cache.cc ('K') | « net/http/http_auth_filter.h ('k') | net/http/http_auth_filter_unittest.cc » ('j') | url/origin.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_auth_filter.cc
diff --git a/net/http/http_auth_filter.cc b/net/http/http_auth_filter.cc
index 53d81eaab4b094cba14204674d3a254660dc9a0d..c445a79f36b0bccfa7424d3e73ef11590cdc3bfe 100644
--- a/net/http/http_auth_filter.cc
+++ b/net/http/http_auth_filter.cc
@@ -40,14 +40,14 @@ void HttpAuthFilterWhitelist::AddRuleToBypassLocal() {
rules_.AddRuleToBypassLocal();
}
-bool HttpAuthFilterWhitelist::IsValid(const GURL& url,
+bool HttpAuthFilterWhitelist::IsValid(const url::Origin& origin,
HttpAuth::Target target) const {
if ((target != HttpAuth::AUTH_SERVER) && (target != HttpAuth::AUTH_PROXY))
return false;
// All proxies pass
if (target == HttpAuth::AUTH_PROXY)
return true;
- return rules_.Matches(url);
+ return rules_.Matches(GURL(origin.serialize()));
}
void HttpAuthFilterWhitelist::SetWhitelist(
« net/http/http_auth_cache.cc ('K') | « net/http/http_auth_filter.h ('k') | net/http/http_auth_filter_unittest.cc » ('j') | url/origin.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698