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

Side by Side Diff: net/http/http_response_body_drainer_unittest.cc

Issue 663043004: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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 "net/http/http_response_body_drainer.h" 5 #include "net/http/http_response_body_drainer.h"
6 6
7 #include <cstring> 7 #include <cstring>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 195 matching lines...) Expand 10 before | Expand all | Expand 10 after
206 protected: 206 protected:
207 HttpResponseBodyDrainerTest() 207 HttpResponseBodyDrainerTest()
208 : proxy_service_(ProxyService::CreateDirect()), 208 : proxy_service_(ProxyService::CreateDirect()),
209 ssl_config_service_(new SSLConfigServiceDefaults), 209 ssl_config_service_(new SSLConfigServiceDefaults),
210 http_server_properties_(new HttpServerPropertiesImpl()), 210 http_server_properties_(new HttpServerPropertiesImpl()),
211 transport_security_state_(new TransportSecurityState()), 211 transport_security_state_(new TransportSecurityState()),
212 session_(CreateNetworkSession()), 212 session_(CreateNetworkSession()),
213 mock_stream_(new MockHttpStream(&result_waiter_)), 213 mock_stream_(new MockHttpStream(&result_waiter_)),
214 drainer_(new HttpResponseBodyDrainer(mock_stream_)) {} 214 drainer_(new HttpResponseBodyDrainer(mock_stream_)) {}
215 215
216 virtual ~HttpResponseBodyDrainerTest() {} 216 ~HttpResponseBodyDrainerTest() override {}
217 217
218 HttpNetworkSession* CreateNetworkSession() const { 218 HttpNetworkSession* CreateNetworkSession() const {
219 HttpNetworkSession::Params params; 219 HttpNetworkSession::Params params;
220 params.proxy_service = proxy_service_.get(); 220 params.proxy_service = proxy_service_.get();
221 params.ssl_config_service = ssl_config_service_.get(); 221 params.ssl_config_service = ssl_config_service_.get();
222 params.http_server_properties = http_server_properties_->GetWeakPtr(); 222 params.http_server_properties = http_server_properties_->GetWeakPtr();
223 params.transport_security_state = transport_security_state_.get(); 223 params.transport_security_state = transport_security_state_.get();
224 return new HttpNetworkSession(params); 224 return new HttpNetworkSession(params);
225 } 225 }
226 226
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
299 too_many_chunks += 1; // Now it's too large. 299 too_many_chunks += 1; // Now it's too large.
300 300
301 mock_stream_->set_num_chunks(too_many_chunks); 301 mock_stream_->set_num_chunks(too_many_chunks);
302 drainer_->Start(session_.get()); 302 drainer_->Start(session_.get());
303 EXPECT_TRUE(result_waiter_.WaitForResult()); 303 EXPECT_TRUE(result_waiter_.WaitForResult());
304 } 304 }
305 305
306 } // namespace 306 } // namespace
307 307
308 } // namespace net 308 } // namespace net
OLDNEW
« no previous file with comments | « net/http/http_network_transaction_unittest.cc ('k') | net/http/http_server_properties_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698