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

Unified Diff: net/http/http_auth_filter.cc

Issue 992733002: Remove //net (except for Android test stuff) and sdch (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 9 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
« no previous file with comments | « net/http/http_auth_filter.h ('k') | net/http/http_auth_filter_unittest.cc » ('j') | no next file with comments »
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
deleted file mode 100644
index 53d81eaab4b094cba14204674d3a254660dc9a0d..0000000000000000000000000000000000000000
--- a/net/http/http_auth_filter.cc
+++ /dev/null
@@ -1,58 +0,0 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "base/strings/string_util.h"
-#include "net/http/http_auth_filter.h"
-#include "url/gurl.h"
-
-namespace net {
-
-// Using a std::set<> has the benefit of removing duplicates automatically.
-typedef std::set<base::string16> RegistryWhitelist;
-
-// TODO(ahendrickson) -- Determine if we want separate whitelists for HTTP and
-// HTTPS, one for both, or only an HTTP one. My understanding is that the HTTPS
-// entries in the registry mean that you are only allowed to connect to the site
-// via HTTPS and still be considered 'safe'.
-
-HttpAuthFilterWhitelist::HttpAuthFilterWhitelist(
- const std::string& server_whitelist) {
- SetWhitelist(server_whitelist);
-}
-
-HttpAuthFilterWhitelist::~HttpAuthFilterWhitelist() {
-}
-
-// Add a new domain |filter| to the whitelist, if it's not already there
-bool HttpAuthFilterWhitelist::AddFilter(const std::string& filter,
- HttpAuth::Target target) {
- if ((target != HttpAuth::AUTH_SERVER) && (target != HttpAuth::AUTH_PROXY))
- return false;
- // All proxies pass
- if (target == HttpAuth::AUTH_PROXY)
- return true;
- rules_.AddRuleFromString(filter);
- return true;
-}
-
-void HttpAuthFilterWhitelist::AddRuleToBypassLocal() {
- rules_.AddRuleToBypassLocal();
-}
-
-bool HttpAuthFilterWhitelist::IsValid(const GURL& url,
- 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);
-}
-
-void HttpAuthFilterWhitelist::SetWhitelist(
- const std::string& server_whitelist) {
- rules_.ParseFromString(server_whitelist);
-}
-
-} // namespace net
« no previous file with comments | « net/http/http_auth_filter.h ('k') | net/http/http_auth_filter_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698