DuplicateContentResourceHandler to monitor resources and track how many times th…

content/browser/renderer_host/resource_dispatcher_host_impl.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.
 
 // See http://dev.chromium.org/developers/design-documents/multi-process-resource-loading
 
 #include "content/browser/renderer_host/resource_dispatcher_host_impl.h"
 
 #include <set>
 #include <vector>
@@ skipping 17 matching lines @@
 #include "content/browser/cross_site_request_manager.h"
 #include "content/browser/download/download_file_manager.h"
 #include "content/browser/download/download_resource_handler.h"
 #include "content/browser/download/save_file_manager.h"
 #include "content/browser/download/save_file_resource_handler.h"
 #include "content/browser/fileapi/chrome_blob_storage_context.h"
 #include "content/browser/plugin_service_impl.h"
 #include "content/browser/renderer_host/async_resource_handler.h"
 #include "content/browser/renderer_host/buffered_resource_handler.h"
 #include "content/browser/renderer_host/cross_site_resource_handler.h"
+#include "content/browser/renderer_host/duplicate_resource_handler.h"
 #include "content/browser/renderer_host/redirect_to_file_resource_handler.h"
 #include "content/browser/renderer_host/render_view_host_delegate.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/renderer_host/resource_message_filter.h"
 #include "content/browser/renderer_host/transfer_navigation_resource_throttle.h"
 #include "content/browser/renderer_host/resource_request_info_impl.h"
 #include "content/browser/renderer_host/sync_resource_handler.h"
 #include "content/browser/renderer_host/throttling_resource_handler.h"
 #include "content/browser/resource_context_impl.h"
 #include "content/browser/worker_host/worker_service_impl.h"
@@ skipping 987 matching lines @@
     // Wrap the event handler to be sure the current page's onunload handler
     // has a chance to run before we render the new page.
     handler.reset(new CrossSiteResourceHandler(handler.Pass(), child_id,
                                                route_id, request));
   }
 
   // Insert a buffered event handler before the actual one.
   handler.reset(
       new BufferedResourceHandler(handler.Pass(), this, request));
 
+  // Install DuplicateResourceHandler, which keeps a hash of resources seen and

[darin (slow to review), 2012/07/23 23:30:14]

nit: This is not the best place to write a comment that defines what
DuplicateResourceHandler is.  That kind of comment belongs in the header file
for DuplicateResourceHandler.

BTW, the name DuplicateResourceHandler is a bit odd.

[frankwang, 2012/07/24 02:59:57]

Done. Do you have any suggestions for the name? DuplicateResourceHandler was the
first one that came to mind.

+  // based on url. It is an experiment meant to simulate cache hits for both a
+  // url-based and content-based cache.
+  // TODO(frankwang, gavinp): Clean up this experiment.
+  handler.reset(
+      new DuplicateResourceHandler(handler.Pass(), request_data.resource_type,
+                                   request));
+
   ScopedVector<ResourceThrottle> throttles;
   if (delegate_) {
     bool is_continuation_of_transferred_request =
         (deferred_loader.get() != NULL);
 
     delegate_->RequestBeginning(request,
                                 resource_context,
                                 request_data.resource_type,
                                 child_id,
                                 route_id,
@@ skipping 664 matching lines @@
 
   return i->second.get();
 }
 
 ResourceLoader* ResourceDispatcherHostImpl::GetLoader(int child_id,
                                                       int request_id) const {
   return GetLoader(GlobalRequestID(child_id, request_id));
 }
 
 }  // namespace content