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

Side by Side Diff: chromecast/net/connectivity_checker_impl.cc

Issue 1303493002: Make UrlRequestContextBuilder take scoped_ptr's (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 4 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "chromecast/net/connectivity_checker_impl.h" 5 #include "chromecast/net/connectivity_checker_impl.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "chromecast/net/net_switches.h" 10 #include "chromecast/net/net_switches.h"
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 } 49 }
50 50
51 void ConnectivityCheckerImpl::Initialize() { 51 void ConnectivityCheckerImpl::Initialize() {
52 base::CommandLine* command_line = base::CommandLine::ForCurrentProcess(); 52 base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
53 base::CommandLine::StringType check_url_str = 53 base::CommandLine::StringType check_url_str =
54 command_line->GetSwitchValueNative(switches::kConnectivityCheckUrl); 54 command_line->GetSwitchValueNative(switches::kConnectivityCheckUrl);
55 connectivity_check_url_.reset(new GURL( 55 connectivity_check_url_.reset(new GURL(
56 check_url_str.empty() ? kDefaultConnectivityCheckUrl : check_url_str)); 56 check_url_str.empty() ? kDefaultConnectivityCheckUrl : check_url_str));
57 57
58 net::URLRequestContextBuilder builder; 58 net::URLRequestContextBuilder builder;
59 builder.set_proxy_config_service( 59 builder.set_proxy_config_service(make_scoped_ptr(
60 new net::ProxyConfigServiceFixed(net::ProxyConfig::CreateDirect())); 60 new net::ProxyConfigServiceFixed(net::ProxyConfig::CreateDirect())));
61 builder.DisableHttpCache(); 61 builder.DisableHttpCache();
62 url_request_context_.reset(builder.Build()); 62 url_request_context_ = builder.Build().Pass();
63 63
64 net::NetworkChangeNotifier::AddNetworkChangeObserver(this); 64 net::NetworkChangeNotifier::AddNetworkChangeObserver(this);
65 task_runner_->PostTask(FROM_HERE, 65 task_runner_->PostTask(FROM_HERE,
66 base::Bind(&ConnectivityCheckerImpl::Check, this)); 66 base::Bind(&ConnectivityCheckerImpl::Check, this));
67 } 67 }
68 68
69 ConnectivityCheckerImpl::~ConnectivityCheckerImpl() { 69 ConnectivityCheckerImpl::~ConnectivityCheckerImpl() {
70 DCHECK(task_runner_.get()); 70 DCHECK(task_runner_.get());
71 net::NetworkChangeNotifier::RemoveNetworkChangeObserver(this); 71 net::NetworkChangeNotifier::RemoveNetworkChangeObserver(this);
72 task_runner_->DeleteSoon(FROM_HERE, url_request_.release()); 72 task_runner_->DeleteSoon(FROM_HERE, url_request_.release());
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
186 186
187 void ConnectivityCheckerImpl::Cancel() { 187 void ConnectivityCheckerImpl::Cancel() {
188 if (!url_request_.get()) 188 if (!url_request_.get())
189 return; 189 return;
190 VLOG(2) << "Cancel connectivity check in progress"; 190 VLOG(2) << "Cancel connectivity check in progress";
191 timeout_.Cancel(); 191 timeout_.Cancel();
192 url_request_.reset(NULL); // URLRequest::Cancel() is called in destructor. 192 url_request_.reset(NULL); // URLRequest::Cancel() is called in destructor.
193 } 193 }
194 194
195 } // namespace chromecast 195 } // namespace chromecast
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698