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

Side by Side Diff: content/browser/loader/resource_dispatcher_host_unittest.cc

Issue 11439008: net: Change argument of URLRequest::set_upload from UploadData to UploadDataStream (Closed) Base URL: http://git.chromium.org/chromium/src.git@chunk
Patch Set: Fix android Created 8 years 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 <vector> 5 #include <vector>
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/file_path.h" 8 #include "base/file_path.h"
9 #include "base/memory/scoped_vector.h" 9 #include "base/memory/scoped_vector.h"
10 #include "base/message_loop.h" 10 #include "base/message_loop.h"
11 #include "base/process_util.h" 11 #include "base/process_util.h"
12 #include "base/string_number_conversions.h" 12 #include "base/string_number_conversions.h"
13 #include "base/string_split.h" 13 #include "base/string_split.h"
14 #include "content/browser/browser_thread_impl.h" 14 #include "content/browser/browser_thread_impl.h"
15 #include "content/browser/child_process_security_policy_impl.h" 15 #include "content/browser/child_process_security_policy_impl.h"
16 #include "content/browser/loader/resource_dispatcher_host_impl.h" 16 #include "content/browser/loader/resource_dispatcher_host_impl.h"
17 #include "content/browser/loader/resource_message_filter.h" 17 #include "content/browser/loader/resource_message_filter.h"
18 #include "content/common/child_process_host_impl.h" 18 #include "content/common/child_process_host_impl.h"
19 #include "content/common/resource_messages.h" 19 #include "content/common/resource_messages.h"
20 #include "content/common/view_messages.h" 20 #include "content/common/view_messages.h"
21 #include "content/public/browser/global_request_id.h" 21 #include "content/public/browser/global_request_id.h"
22 #include "content/public/browser/resource_context.h" 22 #include "content/public/browser/resource_context.h"
23 #include "content/public/browser/resource_dispatcher_host_delegate.h" 23 #include "content/public/browser/resource_dispatcher_host_delegate.h"
24 #include "content/public/browser/resource_throttle.h" 24 #include "content/public/browser/resource_throttle.h"
25 #include "content/public/common/resource_response.h" 25 #include "content/public/common/resource_response.h"
26 #include "content/public/test/test_browser_context.h" 26 #include "content/public/test/test_browser_context.h"
27 #include "content/test/test_content_browser_client.h" 27 #include "content/test/test_content_browser_client.h"
28 #include "net/base/net_errors.h" 28 #include "net/base/net_errors.h"
29 #include "net/base/upload_data.h" 29 #include "net/base/upload_bytes_element_reader.h"
30 #include "net/base/upload_data_stream.h"
30 #include "net/http/http_util.h" 31 #include "net/http/http_util.h"
31 #include "net/url_request/url_request.h" 32 #include "net/url_request/url_request.h"
32 #include "net/url_request/url_request_context.h" 33 #include "net/url_request/url_request_context.h"
33 #include "net/url_request/url_request_job.h" 34 #include "net/url_request/url_request_job.h"
34 #include "net/url_request/url_request_simple_job.h" 35 #include "net/url_request/url_request_simple_job.h"
35 #include "net/url_request/url_request_test_job.h" 36 #include "net/url_request/url_request_test_job.h"
36 #include "testing/gtest/include/gtest/gtest.h" 37 #include "testing/gtest/include/gtest/gtest.h"
37 #include "webkit/appcache/appcache_interfaces.h" 38 #include "webkit/appcache/appcache_interfaces.h"
38 39
39 // TODO(eroman): Write unit tests for SafeBrowsing that exercise 40 // TODO(eroman): Write unit tests for SafeBrowsing that exercise
(...skipping 1242 matching lines...) Expand 10 before | Expand all | Expand 10 after
1282 1283
1283 // Add 9 bytes of referrer. 1284 // Add 9 bytes of referrer.
1284 req.set_referrer("123456789"); 1285 req.set_referrer("123456789");
1285 EXPECT_EQ(4436, 1286 EXPECT_EQ(4436,
1286 ResourceDispatcherHostImpl::CalculateApproximateMemoryCost(&req)); 1287 ResourceDispatcherHostImpl::CalculateApproximateMemoryCost(&req));
1287 1288
1288 // Add 33 bytes of upload content. 1289 // Add 33 bytes of upload content.
1289 std::string upload_content; 1290 std::string upload_content;
1290 upload_content.resize(33); 1291 upload_content.resize(33);
1291 std::fill(upload_content.begin(), upload_content.end(), 'x'); 1292 std::fill(upload_content.begin(), upload_content.end(), 'x');
1292 scoped_refptr<net::UploadData> upload_data(new net::UploadData()); 1293 scoped_ptr<net::UploadElementReader> reader(new net::UploadBytesElementReader(
1293 upload_data->AppendBytes(upload_content.data(), upload_content.size()); 1294 upload_content.data(), upload_content.size()));
1294 req.set_upload(upload_data); 1295 req.set_upload(make_scoped_ptr(
1296 net::UploadDataStream::CreateWithReader(reader.Pass(), 0)));
1295 1297
1296 // Since the upload throttling is disabled, this has no effect on the cost. 1298 // Since the upload throttling is disabled, this has no effect on the cost.
1297 EXPECT_EQ(4436, 1299 EXPECT_EQ(4436,
1298 ResourceDispatcherHostImpl::CalculateApproximateMemoryCost(&req)); 1300 ResourceDispatcherHostImpl::CalculateApproximateMemoryCost(&req));
1299 } 1301 }
1300 1302
1301 // Test the private helper method "IncrementOutstandingRequestsMemoryCost()". 1303 // Test the private helper method "IncrementOutstandingRequestsMemoryCost()".
1302 TEST_F(ResourceDispatcherHostTest, IncrementOutstandingRequestsMemoryCost) { 1304 TEST_F(ResourceDispatcherHostTest, IncrementOutstandingRequestsMemoryCost) {
1303 // Add some counts for render_process_host=7 1305 // Add some counts for render_process_host=7
1304 EXPECT_EQ(0, host_.GetOutstandingRequestsMemoryCost(7)); 1306 EXPECT_EQ(0, host_.GetOutstandingRequestsMemoryCost(7));
(...skipping 642 matching lines...) Expand 10 before | Expand all | Expand 10 after
1947 } 1949 }
1948 1950
1949 MessageLoop::current()->RunUntilIdle(); 1951 MessageLoop::current()->RunUntilIdle();
1950 1952
1951 msgs.clear(); 1953 msgs.clear();
1952 accum_.GetClassifiedMessages(&msgs); 1954 accum_.GetClassifiedMessages(&msgs);
1953 } 1955 }
1954 } 1956 }
1955 1957
1956 } // namespace content 1958 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/loader/resource_dispatcher_host_impl.cc ('k') | net/base/upload_bytes_element_reader.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698