Handle file descriptor exhaustions a little more gracefully. This prevents th...

chrome/browser/renderer_host/buffered_resource_handler.cc

 // Copyright (c) 2006-2008 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 "chrome/browser/renderer_host/buffered_resource_handler.h"
 
 #include "base/histogram.h"
 #include "base/string_util.h"
 #include "net/base/mime_sniffer.h"
 #include "net/base/net_errors.h"
@@ skipping 76 matching lines @@
     DCHECK(!my_buffer_.get());
     my_buffer_ = new net::IOBuffer(kMaxBytesToSniff);
     *buf = my_buffer_.get();
     *buf_size = kMaxBytesToSniff;
     return true;
   }
 
   if (finished_)
     return false;
 
-  bool ret = real_handler_->OnWillRead(request_id, buf, buf_size, min_size);
+  if (!real_handler_->OnWillRead(request_id, buf, buf_size, min_size)) {
+    return false;
+  }
   read_buffer_ = *buf;
   read_buffer_size_ = *buf_size;
   DCHECK(read_buffer_size_ >= kMaxBytesToSniff * 2);
   bytes_read_ = 0;
-  return ret;
+  return true;
 }
 
 bool BufferedResourceHandler::OnReadCompleted(int request_id, int* bytes_read) {
   if (sniff_content_ || should_buffer_) {
     if (KeepBuffering(*bytes_read))
       return true;
 
     LOG(INFO) << "Finished buffering " << request_->url().spec();
     sniff_content_ = should_buffer_ = false;
     *bytes_read = bytes_read_;
@@ skipping 168 matching lines @@
     real_handler_ = download_handler;
   }
   return real_handler_->OnResponseStarted(request_id, response_);
 }
 
 bool BufferedResourceHandler::DidBufferEnough(int bytes_read) {
   const int kRequiredLength = 256;
 
   return bytes_read >= kRequiredLength;
 }