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

Unified Diff: net/http/http_byte_range_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_byte_range_unittest.cc
diff --git a/net/http/http_byte_range_unittest.cc b/net/http/http_byte_range_unittest.cc
index 5dc6e5237f32c08ba1dc33e3da83bd8da7c2fa32..bd5baa9313b7bbd4ebd486344ffcc783368f5261 100644
--- a/net/http/http_byte_range_unittest.cc
+++ b/net/http/http_byte_range_unittest.cc
@@ -12,16 +12,16 @@ TEST(HttpByteRangeTest, ValidRanges) {
int64 suffix_length;
bool valid;
} tests[] = {
- { -1, -1, 0, false },
- { 0, 0, 0, true },
- { -10, 0, 0, false },
- { 10, 0, 0, false },
- { 10, -1, 0, true },
- { -1, -1, -1, false },
- { -1, 50, 0, false },
- { 10, 10000, 0, true },
- { -1, -1, 100000, true },
- };
+ {-1, -1, 0, false},
+ {0, 0, 0, true},
+ {-10, 0, 0, false},
+ {10, 0, 0, false},
+ {10, -1, 0, true},
+ {-1, -1, -1, false},
+ {-1, 50, 0, false},
+ {10, 10000, 0, true},
+ {-1, -1, 100000, true},
+ };
for (size_t i = 0; i < ARRAYSIZE_UNSAFE(tests); ++i) {
net::HttpByteRange range;
@@ -42,18 +42,19 @@ TEST(HttpByteRangeTest, SetInstanceSize) {
int64 expected_lower_bound;
int64 expected_upper_bound;
} tests[] = {
- { -10, 0, -1, 0, false, -1, -1 },
- { 10, 0, -1, 0, false, -1, -1 },
- // Zero instance size is valid, this is the case that user has to handle.
- { -1, -1, -1, 0, true, 0, -1 },
- { -1, -1, 500, 0, true, 0, -1 },
- { -1, 50, -1, 0, false, -1, -1 },
- { -1, -1, 500, 300, true, 0, 299 },
- { -1, -1, -1, 100, true, 0, 99 },
- { 10, -1, -1, 100, true, 10, 99 },
- { -1, -1, 500, 1000, true, 500, 999 },
- { 10, 10000, -1, 1000000, true, 10, 10000 },
- };
+ {-10, 0, -1, 0, false, -1, -1},
+ {10, 0, -1, 0, false, -1, -1},
+ // Zero instance size is valid, this is the case that user has to
+ // handle.
+ {-1, -1, -1, 0, true, 0, -1},
+ {-1, -1, 500, 0, true, 0, -1},
+ {-1, 50, -1, 0, false, -1, -1},
+ {-1, -1, 500, 300, true, 0, 299},
+ {-1, -1, -1, 100, true, 0, 99},
+ {10, -1, -1, 100, true, 10, 99},
+ {-1, -1, 500, 1000, true, 500, 999},
+ {10, 10000, -1, 1000000, true, 10, 10000},
+ };
for (size_t i = 0; i < ARRAYSIZE_UNSAFE(tests); ++i) {
net::HttpByteRange range;
@@ -81,11 +82,13 @@ TEST(HttpByteRangeTest, GetHeaderValue) {
static const struct {
net::HttpByteRange range;
const char* expected;
- } tests[] = {{net::HttpByteRange::Bounded(0, 0), "bytes=0-0"},
- {net::HttpByteRange::Bounded(0, 100), "bytes=0-100"},
- {net::HttpByteRange::Bounded(0, -1), "bytes=0-"},
- {net::HttpByteRange::RightUnbounded(100), "bytes=100-"},
- {net::HttpByteRange::Suffix(100), "bytes=-100"}, };
+ } tests[] = {
+ {net::HttpByteRange::Bounded(0, 0), "bytes=0-0"},
+ {net::HttpByteRange::Bounded(0, 100), "bytes=0-100"},
+ {net::HttpByteRange::Bounded(0, -1), "bytes=0-"},
+ {net::HttpByteRange::RightUnbounded(100), "bytes=100-"},
+ {net::HttpByteRange::Suffix(100), "bytes=-100"},
+ };
for (size_t i = 0; i < ARRAYSIZE_UNSAFE(tests); ++i) {
EXPECT_EQ(tests[i].expected, tests[i].range.GetHeaderValue());
}

Powered by Google App Engine
This is Rietveld 408576698