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

Side by Side Diff: components/cronet/android/cronet_url_request_adapter.cc

Issue 726013002: [Cronet] Hook up library loader, system proxy and network change notifier to async api. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Self review, cl format Created 5 years, 11 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "cronet_url_request_adapter.h" 5 #include "cronet_url_request_adapter.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "components/cronet/android/cronet_url_request_context_adapter.h" 10 #include "components/cronet/android/cronet_url_request_context_adapter.h"
(...skipping 23 matching lines...) Expand all
34 CronetURLRequestAdapter::~CronetURLRequestAdapter() { 34 CronetURLRequestAdapter::~CronetURLRequestAdapter() {
35 DCHECK(IsOnNetworkThread()); 35 DCHECK(IsOnNetworkThread());
36 } 36 }
37 37
38 void CronetURLRequestAdapter::AddRequestHeader(const std::string& name, 38 void CronetURLRequestAdapter::AddRequestHeader(const std::string& name,
39 const std::string& value) { 39 const std::string& value) {
40 DCHECK(!IsOnNetworkThread()); 40 DCHECK(!IsOnNetworkThread());
41 initial_request_headers_.SetHeader(name, value); 41 initial_request_headers_.SetHeader(name, value);
42 } 42 }
43 43
44 bool CronetURLRequestAdapter::PostTaskToNetworkThread( 44 void CronetURLRequestAdapter::PostTaskToNetworkThread(
45 const tracked_objects::Location& from_here, 45 const tracked_objects::Location& from_here,
46 const base::Closure& task) { 46 const base::Closure& task) {
47 DCHECK(!IsOnNetworkThread()); 47 DCHECK(!IsOnNetworkThread());
48 return context_->GetNetworkTaskRunner()->PostTask(from_here, task); 48 context_->PostTaskToNetworkThread(from_here, task);
49 } 49 }
50 50
51 bool CronetURLRequestAdapter::IsOnNetworkThread() const { 51 bool CronetURLRequestAdapter::IsOnNetworkThread() const {
52 return context_->GetNetworkTaskRunner()->BelongsToCurrentThread(); 52 return context_->GetNetworkTaskRunner()->BelongsToCurrentThread();
53 } 53 }
54 54
55 void CronetURLRequestAdapter::Start() { 55 void CronetURLRequestAdapter::Start() {
56 DCHECK(IsOnNetworkThread()); 56 DCHECK(IsOnNetworkThread());
57 VLOG(1) << "Starting chromium request: " 57 VLOG(1) << "Starting chromium request: "
58 << initial_url_.possibly_invalid_spec().c_str() 58 << initial_url_.possibly_invalid_spec().c_str()
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 DCHECK_EQ(request, url_request_); 151 DCHECK_EQ(request, url_request_);
152 if (url_request_->status().is_success()) 152 if (url_request_->status().is_success())
153 return false; 153 return false;
154 VLOG(1) << "Error " << url_request_->status().error() 154 VLOG(1) << "Error " << url_request_->status().error()
155 << " on chromium request: " << initial_url_.possibly_invalid_spec(); 155 << " on chromium request: " << initial_url_.possibly_invalid_spec();
156 delegate_->OnError(url_request_->status().error()); 156 delegate_->OnError(url_request_->status().error());
157 return true; 157 return true;
158 } 158 }
159 159
160 } // namespace cronet 160 } // namespace cronet
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698