Fix some DCHECKS when using BrowserPlugin for extension mime type handlers

content/browser/loader/buffered_resource_handler.cc

Index: content/browser/loader/buffered_resource_handler.cc
diff --git a/content/browser/loader/buffered_resource_handler.cc b/content/browser/loader/buffered_resource_handler.cc
index f609cbfbef752167b0ad743768d482eff9ac9c65..8fb8c5d4484c1d4ef473ed551a3dfe5613598ec7 100644
--- a/content/browser/loader/buffered_resource_handler.cc
+++ b/content/browser/loader/buffered_resource_handler.cc
@@ -386,7 +386,7 @@ bool BufferedResourceHandler::UseAlternateNextHandler(
     scoped_refptr<net::IOBuffer> buf;
     int size = 0;
 
-    next_handler_->OnWillRead(&buf, &size, payload_for_old_handler.length());
+    next_handler_->OnWillRead(&buf, &size, -1);

[mmenke, 2014/06/19 15:08:42]

Why this change?

[Zachary Kuznia, 2014/06/19 18:02:24]

This always ends up forwarding to AsyncResourceHandler::OnWillRead(), which does
a DCHECK_EQ(-1, min_size);

[mmenke, 2014/06/19 19:36:26]

Ahh...right.  Ugly, but we're really bad about how that works, as-is.  Since we
have a CHECK just below, this is fine.  We really should figure out a better way
to do all this, but that ball's not in your court.

     CHECK_GE(size, static_cast<int>(payload_for_old_handler.length()));
 
     memcpy(buf->data(), payload_for_old_handler.c_str(),