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

Side by Side Diff: android_webview/browser/net/android_stream_reader_url_request_job.h

Issue 1439953006: Reland: URLRequestJob: change ReadRawData contract (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address David's comment Created 5 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
« no previous file with comments | « no previous file | android_webview/browser/net/android_stream_reader_url_request_job.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 ANDROID_WEBVIEW_BROWSER_NET_ANDROID_STREAM_READER_URL_REQUEST_JOB_H_ 5 #ifndef ANDROID_WEBVIEW_BROWSER_NET_ANDROID_STREAM_READER_URL_REQUEST_JOB_H_
6 #define ANDROID_WEBVIEW_BROWSER_NET_ANDROID_STREAM_READER_URL_REQUEST_JOB_H_ 6 #define ANDROID_WEBVIEW_BROWSER_NET_ANDROID_STREAM_READER_URL_REQUEST_JOB_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/android/scoped_java_ref.h" 10 #include "base/android/scoped_java_ref.h"
11 #include "base/callback.h" 11 #include "base/callback.h"
12 #include "base/location.h" 12 #include "base/location.h"
13 #include "base/macros.h" 13 #include "base/macros.h"
14 #include "base/memory/ref_counted.h" 14 #include "base/memory/ref_counted.h"
15 #include "base/memory/scoped_ptr.h" 15 #include "base/memory/scoped_ptr.h"
16 #include "base/memory/weak_ptr.h" 16 #include "base/memory/weak_ptr.h"
17 #include "base/threading/thread_checker.h" 17 #include "base/threading/thread_checker.h"
18 #include "net/base/net_errors.h"
18 #include "net/http/http_byte_range.h" 19 #include "net/http/http_byte_range.h"
19 #include "net/url_request/url_request_job.h" 20 #include "net/url_request/url_request_job.h"
20 21
21 namespace base { 22 namespace base {
22 class TaskRunner; 23 class TaskRunner;
23 } 24 }
24 25
25 namespace net { 26 namespace net {
26 class HttpResponseHeaders; 27 class HttpResponseHeaders;
27 class HttpResponseInfo; 28 class HttpResponseInfo;
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 scoped_ptr<Delegate> delegate); 90 scoped_ptr<Delegate> delegate);
90 AndroidStreamReaderURLRequestJob( 91 AndroidStreamReaderURLRequestJob(
91 net::URLRequest* request, 92 net::URLRequest* request,
92 net::NetworkDelegate* network_delegate, 93 net::NetworkDelegate* network_delegate,
93 scoped_ptr<DelegateObtainer> delegate_obtainer, 94 scoped_ptr<DelegateObtainer> delegate_obtainer,
94 bool); // resolve ambiguity 95 bool); // resolve ambiguity
95 96
96 // URLRequestJob: 97 // URLRequestJob:
97 void Start() override; 98 void Start() override;
98 void Kill() override; 99 void Kill() override;
99 bool ReadRawData(net::IOBuffer* buf, int buf_size, int* bytes_read) override; 100 int ReadRawData(net::IOBuffer* buf, int buf_size) override;
100 void SetExtraRequestHeaders(const net::HttpRequestHeaders& headers) override; 101 void SetExtraRequestHeaders(const net::HttpRequestHeaders& headers) override;
101 bool GetMimeType(std::string* mime_type) const override; 102 bool GetMimeType(std::string* mime_type) const override;
102 bool GetCharset(std::string* charset) override; 103 bool GetCharset(std::string* charset) override;
103 int GetResponseCode() const override; 104 int GetResponseCode() const override;
104 void GetResponseInfo(net::HttpResponseInfo* info) override; 105 void GetResponseInfo(net::HttpResponseInfo* info) override;
105 106
106 protected: 107 protected:
107 ~AndroidStreamReaderURLRequestJob() override; 108 ~AndroidStreamReaderURLRequestJob() override;
108 109
109 // Gets the TaskRunner for the worker thread. 110 // Gets the TaskRunner for the worker thread.
(...skipping 14 matching lines...) Expand all
124 125
125 void HeadersComplete(int status_code, const std::string& status_text); 126 void HeadersComplete(int status_code, const std::string& status_text);
126 127
127 void OnInputStreamOpened( 128 void OnInputStreamOpened(
128 scoped_ptr<Delegate> delegate, 129 scoped_ptr<Delegate> delegate,
129 scoped_ptr<android_webview::InputStream> input_stream); 130 scoped_ptr<android_webview::InputStream> input_stream);
130 void OnReaderSeekCompleted(int content_size); 131 void OnReaderSeekCompleted(int content_size);
131 void OnReaderReadCompleted(int bytes_read); 132 void OnReaderReadCompleted(int bytes_read);
132 133
133 net::HttpByteRange byte_range_; 134 net::HttpByteRange byte_range_;
135 net::Error range_parse_result_;
134 scoped_ptr<net::HttpResponseInfo> response_info_; 136 scoped_ptr<net::HttpResponseInfo> response_info_;
135 scoped_ptr<Delegate> delegate_; 137 scoped_ptr<Delegate> delegate_;
136 scoped_ptr<DelegateObtainer> delegate_obtainer_; 138 scoped_ptr<DelegateObtainer> delegate_obtainer_;
137 scoped_refptr<InputStreamReaderWrapper> input_stream_reader_wrapper_; 139 scoped_refptr<InputStreamReaderWrapper> input_stream_reader_wrapper_;
138 base::ThreadChecker thread_checker_; 140 base::ThreadChecker thread_checker_;
139 141
140 base::WeakPtrFactory<AndroidStreamReaderURLRequestJob> weak_factory_; 142 base::WeakPtrFactory<AndroidStreamReaderURLRequestJob> weak_factory_;
141 143
142 DISALLOW_COPY_AND_ASSIGN(AndroidStreamReaderURLRequestJob); 144 DISALLOW_COPY_AND_ASSIGN(AndroidStreamReaderURLRequestJob);
143 }; 145 };
144 146
145 } // namespace android_webview 147 } // namespace android_webview
146 148
147 #endif // ANDROID_WEBVIEW_BROWSER_NET_ANDROID_STREAM_READER_URL_REQUEST_JOB_H_ 149 #endif // ANDROID_WEBVIEW_BROWSER_NET_ANDROID_STREAM_READER_URL_REQUEST_JOB_H_
OLDNEW
« no previous file with comments | « no previous file | android_webview/browser/net/android_stream_reader_url_request_job.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698