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

Side by Side Diff: net/url_request/url_request_http_job.h

Issue 10911151: URLRequestHttpJob::StartTransaction should honour network delegate. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 2 tests split into 6 and only 4 disabled for CF Created 8 years, 2 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 | Annotate | Revision Log
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 #ifndef NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_ 5 #ifndef NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_
6 #define NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_ 6 #define NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
152 152
153 virtual void UpdatePacketReadTimes() OVERRIDE; 153 virtual void UpdatePacketReadTimes() OVERRIDE;
154 void RecordPacketStats(FilterContext::StatisticSelector statistic) const; 154 void RecordPacketStats(FilterContext::StatisticSelector statistic) const;
155 155
156 void RecordCompressionHistograms(); 156 void RecordCompressionHistograms();
157 bool IsCompressibleContent() const; 157 bool IsCompressibleContent() const;
158 158
159 // Starts the transaction if extensions using the webrequest API do not 159 // Starts the transaction if extensions using the webrequest API do not
160 // object. 160 // object.
161 void StartTransaction(); 161 void StartTransaction();
162 // If |result| is net::OK, calls StartTransactionInternal. Otherwise notifies
163 // cancellation.
164 void MaybeStartTransactionInternal(int result);
162 void StartTransactionInternal(); 165 void StartTransactionInternal();
163 166
164 void RecordPerfHistograms(CompletionCause reason); 167 void RecordPerfHistograms(CompletionCause reason);
165 void DoneWithRequest(CompletionCause reason); 168 void DoneWithRequest(CompletionCause reason);
166 169
167 // Callback functions for Cookie Monster 170 // Callback functions for Cookie Monster
168 void DoLoadCookies(); 171 void DoLoadCookies();
169 void CheckCookiePolicyAndLoad(const CookieList& cookie_list); 172 void CheckCookiePolicyAndLoad(const CookieList& cookie_list);
170 void OnCookiesLoaded( 173 void OnCookiesLoaded(
171 const std::string& cookie_line, 174 const std::string& cookie_line,
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
240 bool awaiting_callback_; 243 bool awaiting_callback_;
241 244
242 scoped_ptr<HttpTransactionDelegateImpl> http_transaction_delegate_; 245 scoped_ptr<HttpTransactionDelegateImpl> http_transaction_delegate_;
243 246
244 DISALLOW_COPY_AND_ASSIGN(URLRequestHttpJob); 247 DISALLOW_COPY_AND_ASSIGN(URLRequestHttpJob);
245 }; 248 };
246 249
247 } // namespace net 250 } // namespace net
248 251
249 #endif // NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_ 252 #endif // NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698