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

Side by Side Diff: chrome/browser/extensions/api/web_request/web_request_api_unittest.cc

Issue 1315643005: Use EnumerateHeaderLines to parse added headers in webRequest API (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <map> 5 #include <map>
6 #include <queue> 6 #include <queue>
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 1269 matching lines...) Expand 10 before | Expand all | Expand 10 after
1280 EXPECT_EQ("value3", value); 1280 EXPECT_EQ("value3", value);
1281 } 1281 }
1282 1282
1283 TEST(ExtensionWebRequestHelpersTest, TestCalculateOnHeadersReceivedDelta) { 1283 TEST(ExtensionWebRequestHelpersTest, TestCalculateOnHeadersReceivedDelta) {
1284 const bool cancel = true; 1284 const bool cancel = true;
1285 char base_headers_string[] = 1285 char base_headers_string[] =
1286 "HTTP/1.0 200 OK\r\n" 1286 "HTTP/1.0 200 OK\r\n"
1287 "Key1: Value1\r\n" 1287 "Key1: Value1\r\n"
1288 "Key2: Value2, Bar\r\n" 1288 "Key2: Value2, Bar\r\n"
1289 "Key3: Value3\r\n" 1289 "Key3: Value3\r\n"
1290 "Key5: Value5, end5\r\n"
1290 "\r\n"; 1291 "\r\n";
1291 scoped_refptr<net::HttpResponseHeaders> base_headers( 1292 scoped_refptr<net::HttpResponseHeaders> base_headers(
1292 new net::HttpResponseHeaders( 1293 new net::HttpResponseHeaders(
1293 net::HttpUtil::AssembleRawHeaders( 1294 net::HttpUtil::AssembleRawHeaders(
1294 base_headers_string, sizeof(base_headers_string)))); 1295 base_headers_string, sizeof(base_headers_string))));
1295 1296
1296 ResponseHeaders new_headers; 1297 ResponseHeaders new_headers;
1297 new_headers.push_back(ResponseHeader("kEy1", "Value1")); // Unchanged 1298 new_headers.push_back(ResponseHeader("kEy1", "Value1")); // Unchanged
1298 new_headers.push_back(ResponseHeader("Key2", "Value1")); // Modified 1299 new_headers.push_back(ResponseHeader("Key2", "Value1")); // Modified
1299 // Key3 is deleted 1300 // Key3 is deleted
1300 new_headers.push_back(ResponseHeader("Key4", "Value4")); // Added 1301 new_headers.push_back(ResponseHeader("Key4", "Value4")); // Added
1302 new_headers.push_back(ResponseHeader("Key5", "Value5, end5")); // Unchanged
1301 GURL effective_new_url; 1303 GURL effective_new_url;
1302 1304
1303 scoped_ptr<EventResponseDelta> delta( 1305 scoped_ptr<EventResponseDelta> delta(
1304 CalculateOnHeadersReceivedDelta("extid", 1306 CalculateOnHeadersReceivedDelta("extid",
1305 base::Time::Now(), 1307 base::Time::Now(),
1306 cancel, 1308 cancel,
1307 effective_new_url, 1309 effective_new_url,
1308 base_headers.get(), 1310 base_headers.get(),
1309 &new_headers)); 1311 &new_headers));
1310 ASSERT_TRUE(delta.get()); 1312 ASSERT_TRUE(delta.get());
(...skipping 936 matching lines...) Expand 10 before | Expand all | Expand 10 after
2247 EXPECT_TRUE(credentials_set); 2249 EXPECT_TRUE(credentials_set);
2248 EXPECT_FALSE(auth3.Empty()); 2250 EXPECT_FALSE(auth3.Empty());
2249 EXPECT_EQ(username, auth1.username()); 2251 EXPECT_EQ(username, auth1.username());
2250 EXPECT_EQ(password, auth1.password()); 2252 EXPECT_EQ(password, auth1.password());
2251 EXPECT_EQ(1u, warning_set.size()); 2253 EXPECT_EQ(1u, warning_set.size());
2252 EXPECT_TRUE(HasWarning(warning_set, "extid2")); 2254 EXPECT_TRUE(HasWarning(warning_set, "extid2"));
2253 EXPECT_EQ(3u, capturing_net_log.GetSize()); 2255 EXPECT_EQ(3u, capturing_net_log.GetSize());
2254 } 2256 }
2255 2257
2256 } // namespace extensions 2258 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698