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

Unified Diff: chrome/browser/renderer_host/sync_resource_handler.cc

Issue 18390: Change URLRequest to use a ref-counted buffer for actual IO.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/renderer_host/sync_resource_handler.h ('k') | chrome/common/chrome_plugin_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/renderer_host/sync_resource_handler.cc
===================================================================
--- chrome/browser/renderer_host/sync_resource_handler.cc (revision 8565)
+++ chrome/browser/renderer_host/sync_resource_handler.cc (working copy)
@@ -8,7 +8,8 @@
ResourceDispatcherHost::Receiver* receiver,
const GURL& url,
IPC::Message* result_message)
- : receiver_(receiver),
+ : read_buffer_(new net::IOBuffer(kReadBufSize)),
+ receiver_(receiver),
result_message_(result_message) {
result_.final_url = url;
result_.filter_policy = FilterPolicy::DONT_FILTER;
@@ -29,10 +30,10 @@
return true;
}
-bool SyncResourceHandler::OnWillRead(int request_id,
- char** buf, int* buf_size, int min_size) {
+bool SyncResourceHandler::OnWillRead(int request_id, net::IOBuffer** buf,
+ int* buf_size, int min_size) {
DCHECK(min_size == -1);
- *buf = read_buffer_;
+ *buf = read_buffer_.get();
*buf_size = kReadBufSize;
return true;
}
@@ -40,7 +41,7 @@
bool SyncResourceHandler::OnReadCompleted(int request_id, int* bytes_read) {
if (!*bytes_read)
return true;
- result_.data.append(read_buffer_, *bytes_read);
+ result_.data.append(read_buffer_->data(), *bytes_read);
return true;
}
« no previous file with comments | « chrome/browser/renderer_host/sync_resource_handler.h ('k') | chrome/common/chrome_plugin_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698