Adding histograms showing fraction of page load times

chrome/browser/net/chrome_network_delegate.cc

Index: chrome/browser/net/chrome_network_delegate.cc
===================================================================
--- chrome/browser/net/chrome_network_delegate.cc	(revision 147723)
+++ chrome/browser/net/chrome_network_delegate.cc	(working copy)
@@ -14,6 +14,7 @@
 #include "chrome/browser/extensions/extension_event_router_forwarder.h"
 #include "chrome/browser/extensions/extension_info_map.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
+#include "chrome/browser/net/cache_stats.h"
 #include "chrome/browser/prefs/pref_member.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/task_manager/task_manager.h"
@@ -133,7 +134,8 @@
       extension_info_map_(extension_info_map),
       enable_referrers_(enable_referrers),
       never_throttle_requests_(false),
-      url_blacklist_manager_(url_blacklist_manager) {
+      url_blacklist_manager_(url_blacklist_manager),
+      cache_stats_(chrome_browser_net::CacheStats::GetInstance()) {

[rvargas (doing something else), 2012/07/23 22:22:36]

This bundles together info from all profiles / request contexts, right? Each
cache type has a different backend implementation so putting all the data
together is not that good.

[tburkard, 2012/07/24 01:03:12]

Changed it to only operate on non-OTR profiles, which are all be using the disk
cache (and no in memory cache).  Would that be better?
On 2012/07/23 22:22:36, rvargas wrote:

[rvargas (doing something else), 2012/07/24 03:10:38]

That still treats the media cache and the http cache as the same, and the
implementation is not the same.

[tburkard, 2012/07/24 22:23:26]

I will get back to you on this one in a separate email/update.

On 2012/07/24 03:10:38, rvargas wrote:

   DCHECK(event_router);
   DCHECK(enable_referrers);
   DCHECK(!profile || cookie_settings);
@@ -389,3 +391,9 @@
 #endif
   return net::OK;
 }
+
+void ChromeNetworkDelegate::OnCacheWaitStateChange(
+    const net::URLRequest& request,
+    CacheWaitState state) {
+  cache_stats_->OnCacheWaitStateChange(request, state);
+}