Rewrite guts of ResourceLoader and BufferedResourceHandler to suck less.

content/browser/renderer_host/sync_resource_handler.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/sync_resource_handler.h"
 
 #include "base/logging.h"
 #include "content/browser/debugger/devtools_netlog_observer.h"
 #include "content/browser/renderer_host/resource_dispatcher_host_impl.h"
 #include "content/browser/renderer_host/resource_message_filter.h"
@@ skipping 85 matching lines @@
 }
 
 bool SyncResourceHandler::OnWillRead(int request_id, net::IOBuffer** buf,
                                      int* buf_size, int min_size) {
   DCHECK(min_size == -1);
   *buf = read_buffer_.get();
   *buf_size = kReadBufSize;
   return true;
 }
 
-bool SyncResourceHandler::OnReadCompleted(int request_id, int* bytes_read,
+bool SyncResourceHandler::OnReadCompleted(int request_id, int bytes_read,
                                           bool* defer) {
-  if (!*bytes_read)
+  if (!bytes_read)
     return true;
-  result_.data.append(read_buffer_->data(), *bytes_read);
+  result_.data.append(read_buffer_->data(), bytes_read);
   return true;
 }
 
 bool SyncResourceHandler::OnResponseCompleted(
     int request_id,
     const net::URLRequestStatus& status,
     const std::string& security_info) {
   result_.status = status;
 
   result_.encoded_data_length =
       DevToolsNetLogObserver::GetAndResetEncodedDataLength(request_);
 
   ResourceHostMsg_SyncLoad::WriteReplyParams(result_message_, result_);
   filter_->Send(result_message_);
   result_message_ = NULL;
   return true;
 }
 
 }  // namespace content