Component updater on-demand logic with ResourceThrottle

chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

Index: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
===================================================================
--- chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc	(revision 231370)
+++ chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc	(working copy)
@@ -11,6 +11,7 @@
 #include "base/metrics/histogram.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
+#include "chrome/browser/component_updater/component_updater_service.h"
 #include "chrome/browser/content_settings/host_content_settings_map.h"
 #include "chrome/browser/download/download_request_limiter.h"
 #include "chrome/browser/download/download_resource_throttle.h"
@@ -193,6 +194,30 @@
   }
 }
 
+void AppendComponentUpdaterThrottles(
+    net::URLRequest* request,
+    content::ResourceContext* resource_context,
+    ResourceType::Type resource_type,
+    ScopedVector<content::ResourceThrottle>* throttles) {
+  const char* crx_id = NULL;
+  // Check for PNaCL nexe request.
+  if (resource_type == ResourceType::OBJECT) {
+    const net::HttpRequestHeaders& headers = request->extra_request_headers();
+    std::string accept_headers;
+    if (headers.GetHeader("Accept", &accept_headers)) {
+      if (accept_headers.find("application/x-pnacl") != std::string::npos)
+        crx_id = "hnimpnehoodheedghdeeijklkeaacbdc";
+    }
+  }
+
+  if (crx_id) {
+    // We got a component we need to install, so throttle the resource
+    // until the component is installed.
+    ComponentUpdateService* cus = g_browser_process->component_updater();
+    throttles->push_back(cus->GetOnDemandResourceThrottle(request, crx_id));
+  }
+}
+
 }  // end namespace
 
 ChromeResourceDispatcherHostDelegate::ChromeResourceDispatcherHostDelegate(
@@ -319,6 +344,12 @@
                                   resource_context,
                                   resource_type,
                                   throttles);
+  if (!is_prerendering) {

[darin (slow to review), 2013/11/01 21:14:41]

we should probably make sure that the presence of a pnacl module causes
prerendering to be disabled, or we should not exclude prerendering here.

[cpu_(ooo_6.6-7.5), 2013/11/01 22:43:37]

Yeah, nacl is disabled since the infamous PinkyPie exploit.

On 2013/11/01 21:14:41, darin wrote:

+    AppendComponentUpdaterThrottles(request,
+                                   resource_context,

[darin (slow to review), 2013/11/01 21:14:41]

nit: indentation

[cpu_(ooo_6.6-7.5), 2013/11/01 22:43:37]

Done.

+                                   resource_type,
+                                   throttles);
+  }
 
   if (io_data->resource_prefetch_predictor_observer()) {
     io_data->resource_prefetch_predictor_observer()->OnRequestStarted(