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 147972)
+++ 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()) {

[willchan no longer on Chromium, 2012/07/24 23:17:13]

https://groups.google.com/a/chromium.org/forum/#!msg/chromium-dev/P9tWR4mlUpE...
TL;DR: DestructionObserver results in problems due to shutdown ordering. We
generally try to avoid it. What's better is for you to provide defined
ownership. Put it in IOThread::Globals since it is a global that lives on the IO
thread. That way, it will be destroyed properly on shutdown when the IO thread
goes away. ProfileIOData has a pointer to IOThread::Globals on the IO thread, so
you can pass in the CacheStats object to ChromeNetworkDelegate via the
constructor.

It's possible that you could be cleaner and avoid having to check OTR in
CacheStats by simply having the ChromeNetworkDelegate for
OffTheRecordProfileIOData constructed with a NULL CacheStats so it doesn't try
to update it. Doesn't matter too much.

[tburkard, 2012/07/25 20:06:52]

Done.

   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);
+}