Commit 6697f8ba authored by Yeol Park's avatar Yeol Park Committed by Commit Bot

Uses AddHeaderFromString instead of AddHeadersFromString, for code health.

So it replaced AddHeadersFromString with AddHeaderFromString
in the net_unittests.

Bug: 862175
Change-Id: I3296f86f44ef5fca4b54fc267fdf4550707a3148
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1573420Reviewed-by: default avatarMatt Menke <mmenke@chromium.org>
Commit-Queue: Yeol Park <peary2@gmail.com>
Cr-Commit-Position: refs/heads/master@{#652418}
parent 53a914ea
......@@ -172,13 +172,13 @@ TEST(RedirectUtilTest, RemovedHeaders) {
HttpRequestHeaders initial_headers, modified_headers, final_headers;
std::vector<std::string> removed_headers;
for (const char* header : test.initial_headers)
initial_headers.AddHeadersFromString(header);
initial_headers.AddHeaderFromString(header);
for (const char* header : test.modified_headers)
modified_headers.AddHeadersFromString(header);
modified_headers.AddHeaderFromString(header);
for (const char* header : test.removed_headers)
removed_headers.push_back(header);
for (const char* header : test.final_headers)
final_headers.AddHeadersFromString(header);
final_headers.AddHeaderFromString(header);
bool should_clear_upload(false); // unused.
RedirectUtil::UpdateHttpRequest(GURL(), // original_url
......@@ -196,8 +196,8 @@ TEST(RedirectUtilTest, RemovedHeaders) {
// Test with removed_headers = base::nullopt.
TEST(RedirectUtilTest, RemovedHeadersNullOpt) {
HttpRequestHeaders initial_headers, final_headers;
initial_headers.AddHeadersFromString("A:0");
final_headers.AddHeadersFromString("A:0");
initial_headers.SetHeader("A", "0");
final_headers.SetHeader("A", "0");
base::Optional<std::vector<std::string>> removed_headers(base::nullopt);
base::Optional<HttpRequestHeaders> modified_headers(base::in_place);
bool should_clear_upload(false); // unused.
......@@ -216,8 +216,8 @@ TEST(RedirectUtilTest, RemovedHeadersNullOpt) {
// Test with modified_headers = base::nullopt.
TEST(RedirectUtilTest, ModifyHeadersNullopt) {
HttpRequestHeaders initial_headers, final_headers;
initial_headers.AddHeadersFromString("A:0");
final_headers.AddHeadersFromString("A:0");
initial_headers.SetHeader("A", "0");
final_headers.SetHeader("A", "0");
base::Optional<std::vector<std::string>> removed_headers(base::in_place);
base::Optional<HttpRequestHeaders> modified_headers(base::nullopt);
bool should_clear_upload(false); // unused.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment