PlzNavigate: Get StreamPrivate API to work.

content/browser/loader/navigation_resource_handler.cc

Index: content/browser/loader/navigation_resource_handler.cc
diff --git a/content/browser/loader/navigation_resource_handler.cc b/content/browser/loader/navigation_resource_handler.cc
index bd2e5b496b8ade249986664364b169912ba29ce9..cc6ba9f56fe40bbea9c7301ff66fc4984a881dea 100644
--- a/content/browser/loader/navigation_resource_handler.cc
+++ b/content/browser/loader/navigation_resource_handler.cc
@@ -103,7 +103,7 @@ bool NavigationResourceHandler::OnResponseStarted(ResourceResponse* response,
   // TODO(davidben): Move the dispatch out of MimeTypeResourceHandler. Perhaps
   // all the way to the UI thread. Downloads, user certificates, etc., should be
   // dispatched at the navigation layer.
-  if (info->IsDownload() || info->is_stream())
+  if (info->IsDownload())
     return true;
 
   StreamContext* stream_context =
@@ -132,8 +132,10 @@ bool NavigationResourceHandler::OnResponseStarted(ResourceResponse* response,
 
   core_->NotifyResponseStarted(response, writer_.stream()->CreateHandle(),
                                ssl_status, std::move(cloned_data));
-  *defer = true;
-
+  // Don't defer stream based requests. This includes requests initiated via
+  // mime type sniffing, etc.

[clamy, 2016/09/15 13:15:24]

Could you add a TODO to check if they should still go through the throttle
checks? I understand that this doesn't work currently because the
InterceptingResourceHandler placed before it doesn't expect the old handler to
defer the request.

[ananta, 2016/09/15 21:25:03]

Done.

+  if (!info->is_stream())
+    *defer = true;
   return true;
 }