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

Unified Diff: net/http/http_auth_filter_unittest.cc

Issue 266243004: Clang format slam. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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
Index: net/http/http_auth_filter_unittest.cc
diff --git a/net/http/http_auth_filter_unittest.cc b/net/http/http_auth_filter_unittest.cc
index 25723c136a3ef89e7ba518b6d7461872f98ef25a..cf85a7baed77f632264aee51225e2dfa70ede911 100644
--- a/net/http/http_auth_filter_unittest.cc
+++ b/net/http/http_auth_filter_unittest.cc
@@ -4,7 +4,6 @@
#include <ostream>
-
#include "base/memory/scoped_ptr.h"
#include "net/http/http_auth_filter.h"
#include "testing/gtest/include/gtest/gtest.h"
@@ -15,17 +14,9 @@ namespace net {
namespace {
static const char* const server_whitelist_array[] = {
- "google.com",
- "linkedin.com",
- "book.com",
- ".chromium.org",
- ".gag",
- "gog"
-};
+ "google.com", "linkedin.com", "book.com", ".chromium.org", ".gag", "gog"};
-enum {
- ALL_SERVERS_MATCH = (1 << arraysize(server_whitelist_array)) - 1
-};
+enum { ALL_SERVERS_MATCH = (1 << arraysize(server_whitelist_array)) - 1 };
struct UrlData {
GURL url;
@@ -35,40 +26,43 @@ struct UrlData {
};
static const UrlData urls[] = {
- { GURL(std::string()), HttpAuth::AUTH_NONE, false, 0 },
- { GURL("http://foo.cn"), HttpAuth::AUTH_PROXY, true, ALL_SERVERS_MATCH },
- { GURL("http://foo.cn"), HttpAuth::AUTH_SERVER, false, 0 },
- { GURL("http://slashdot.org"), HttpAuth::AUTH_NONE, false, 0 },
- { GURL("http://www.google.com"), HttpAuth::AUTH_SERVER, true, 1 << 0 },
- { GURL("http://www.google.com"), HttpAuth::AUTH_PROXY, true,
- ALL_SERVERS_MATCH },
- { GURL("https://login.facebook.com/login.php?login_attempt=1"),
- HttpAuth::AUTH_NONE, false, 0 },
- { GURL("http://codereview.chromium.org/634002/show"), HttpAuth::AUTH_SERVER,
- true, 1 << 3 },
- { GURL("http://code.google.com/p/chromium/issues/detail?id=34505"),
- HttpAuth::AUTH_SERVER, true, 1 << 0 },
- { GURL("http://code.google.com/p/chromium/issues/list?can=2&q=label:"
+ {GURL(std::string()), HttpAuth::AUTH_NONE, false, 0},
+ {GURL("http://foo.cn"), HttpAuth::AUTH_PROXY, true, ALL_SERVERS_MATCH},
+ {GURL("http://foo.cn"), HttpAuth::AUTH_SERVER, false, 0},
+ {GURL("http://slashdot.org"), HttpAuth::AUTH_NONE, false, 0},
+ {GURL("http://www.google.com"), HttpAuth::AUTH_SERVER, true, 1 << 0},
+ {GURL("http://www.google.com"), HttpAuth::AUTH_PROXY, true,
+ ALL_SERVERS_MATCH},
+ {GURL("https://login.facebook.com/login.php?login_attempt=1"),
+ HttpAuth::AUTH_NONE, false, 0},
+ {GURL("http://codereview.chromium.org/634002/show"), HttpAuth::AUTH_SERVER,
+ true, 1 << 3},
+ {GURL("http://code.google.com/p/chromium/issues/detail?id=34505"),
+ HttpAuth::AUTH_SERVER, true, 1 << 0},
+ {GURL(
+ "http://code.google.com/p/chromium/issues/list?can=2&q=label:"
"spdy&sort=owner&colspec=ID%20Stars%20Pri%20Area%20Type%20Status%20"
"Summary%20Modified%20Owner%20Mstone%20OS"),
- HttpAuth::AUTH_SERVER, true, 1 << 3 },
- { GURL("https://www.linkedin.com/secure/login?trk=hb_signin"),
- HttpAuth::AUTH_SERVER, true, 1 << 1 },
- { GURL("http://www.linkedin.com/mbox?displayMBoxItem=&"
+ HttpAuth::AUTH_SERVER, true, 1 << 3},
+ {GURL("https://www.linkedin.com/secure/login?trk=hb_signin"),
+ HttpAuth::AUTH_SERVER, true, 1 << 1},
+ {GURL(
+ "http://www.linkedin.com/mbox?displayMBoxItem=&"
"itemID=I1717980652_2&trk=COMM_HP_MSGVW_MEBC_MEBC&goback=.hom"),
- HttpAuth::AUTH_SERVER, true, 1 << 1 },
- { GURL("http://news.slashdot.org/story/10/02/18/190236/"
+ HttpAuth::AUTH_SERVER, true, 1 << 1},
+ {GURL(
+ "http://news.slashdot.org/story/10/02/18/190236/"
"New-Plan-Lets-Top-HS-Students-Graduate-2-Years-Early"),
- HttpAuth::AUTH_PROXY, true, ALL_SERVERS_MATCH },
- { GURL("http://codereview.chromium.org/646068/diff/4001/5003"),
- HttpAuth::AUTH_SERVER, true, 1 << 3 },
- { GURL("http://codereview.chromium.gag/646068/diff/4001/5003"),
- HttpAuth::AUTH_SERVER, true, 1 << 4 },
- { GURL("http://codereview.chromium.gog/646068/diff/4001/5003"),
- HttpAuth::AUTH_SERVER, true, 1 << 5 },
+ HttpAuth::AUTH_PROXY, true, ALL_SERVERS_MATCH},
+ {GURL("http://codereview.chromium.org/646068/diff/4001/5003"),
+ HttpAuth::AUTH_SERVER, true, 1 << 3},
+ {GURL("http://codereview.chromium.gag/646068/diff/4001/5003"),
+ HttpAuth::AUTH_SERVER, true, 1 << 4},
+ {GURL("http://codereview.chromium.gog/646068/diff/4001/5003"),
+ HttpAuth::AUTH_SERVER, true, 1 << 5},
};
-} // namespace
+} // namespace
TEST(HttpAuthFilterTest, EmptyFilter) {
// Create an empty filter
@@ -96,4 +90,4 @@ TEST(HttpAuthFilterTest, NonEmptyFilter) {
}
}
-} // namespace net
+} // namespace net

Powered by Google App Engine
This is Rietveld 408576698