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

Unified Diff: sync/internal_api/http_bridge.cc

Issue 16298002: Update sync/, sql/, and printing/ to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sync/engine/model_changing_syncer_command.cc ('k') | sync/internal_api/http_bridge_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/internal_api/http_bridge.cc
diff --git a/sync/internal_api/http_bridge.cc b/sync/internal_api/http_bridge.cc
index c3cb4b51c84779d4306fb63b40c58d71d73111fa..d429903fdc0df4567f8780fbc2b42085da16d5db 100644
--- a/sync/internal_api/http_bridge.cc
+++ b/sync/internal_api/http_bridge.cc
@@ -29,8 +29,8 @@ HttpBridge::RequestContextGetter::RequestContextGetter(
network_task_runner_(
baseline_context_getter_->GetNetworkTaskRunner()),
user_agent_(user_agent) {
- DCHECK(baseline_context_getter_);
- DCHECK(network_task_runner_);
+ DCHECK(baseline_context_getter_.get());
+ DCHECK(network_task_runner_.get());
DCHECK(!user_agent_.empty());
}
@@ -70,7 +70,7 @@ HttpBridgeFactory::~HttpBridgeFactory() {
}
HttpPostProviderInterface* HttpBridgeFactory::Create() {
- HttpBridge* http = new HttpBridge(request_context_getter_,
+ HttpBridge* http = new HttpBridge(request_context_getter_.get(),
network_time_update_callback_);
http->AddRef();
return http;
@@ -228,7 +228,7 @@ void HttpBridge::MakeAsynchronousPost() {
fetch_state_.url_poster = net::URLFetcher::Create(
url_for_request_, net::URLFetcher::POST, this);
- fetch_state_.url_poster->SetRequestContext(context_getter_for_request_);
+ fetch_state_.url_poster->SetRequestContext(context_getter_for_request_.get());
fetch_state_.url_poster->SetUploadData(content_type_, request_content_);
fetch_state_.url_poster->SetExtraRequestHeaders(extra_headers_);
fetch_state_.url_poster->SetLoadFlags(net::LOAD_DO_NOT_SEND_COOKIES);
@@ -323,7 +323,7 @@ void HttpBridge::OnURLFetchComplete(const net::URLFetcher* source) {
net::URLRequestContextGetter* HttpBridge::GetRequestContextGetterForTest()
const {
- return context_getter_for_request_;
+ return context_getter_for_request_.get();
}
void HttpBridge::UpdateNetworkTime() {
« no previous file with comments | « sync/engine/model_changing_syncer_command.cc ('k') | sync/internal_api/http_bridge_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698