DuplicateContentResourceHandler to monitor resources and track how many times th…

content/browser/renderer_host/resource_dispatcher_host_impl.cc

Index: content/browser/renderer_host/resource_dispatcher_host_impl.cc
===================================================================
--- content/browser/renderer_host/resource_dispatcher_host_impl.cc	(revision 147396)
+++ content/browser/renderer_host/resource_dispatcher_host_impl.cc	(working copy)
@@ -35,6 +35,7 @@
 #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"
@@ -1042,6 +1043,14 @@
   handler.reset(
       new BufferedResourceHandler(handler.Pass(), this, request));
 
+  // Install DuplicateResourceHandler, which keeps a hash of resources
+  // seen in general and based on url. It is an experiment meant to
+  // to simulate cache hits for both a url-based and content-based cache.
+  // TODO(frankwang) Clean up this experiment.

[gavinp, 2012/07/20 01:37:59]

Add : after ). Remove "in general". Wrap everything at 80 columns. Add ",gavinp"
to frankwang.

[frankwang, 2012/07/20 04:50:00]

Done.

+  handler.reset(
+      new DuplicateResourceHandler(handler.Pass(), request_data.resource_type,
+                                   request));
+
   ScopedVector<ResourceThrottle> throttles;
   if (delegate_) {
     bool is_continuation_of_transferred_request =