Adding histograms showing fraction of page load times

chrome/browser/net/chrome_network_delegate.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.
 
 #include "chrome/browser/net/chrome_network_delegate.h"
 
 #include "base/logging.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/content_settings/cookie_settings.h"
 #include "chrome/browser/content_settings/tab_specific_content_settings.h"
 #include "chrome/browser/custom_handlers/protocol_handler_registry.h"
 #include "chrome/browser/extensions/api/proxy/proxy_api.h"
 #include "chrome/browser/extensions/api/web_request/web_request_api.h"
 #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"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/resource_request_info.h"
 #include "net/base/host_port_pair.h"
 #include "net/base/net_errors.h"
@@ skipping 99 matching lines @@
     const policy::URLBlacklistManager* url_blacklist_manager,
     void* profile,
     CookieSettings* cookie_settings,
     BooleanPrefMember* enable_referrers)
     : event_router_(event_router),
       profile_(profile),
       cookie_settings_(cookie_settings),
       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);
 }
 
 ChromeNetworkDelegate::~ChromeNetworkDelegate() {}
 
 void ChromeNetworkDelegate::NeverThrottleRequests() {
   never_throttle_requests_ = true;
 }
@@ skipping 235 matching lines @@
     // URL access blocked by policy.
     socket->net_log()->AddEvent(
         net::NetLog::TYPE_CHROME_POLICY_ABORTED_REQUEST,
         net::NetLog::StringCallback("url",
                                     &socket->url().possibly_invalid_spec()));
     return net::ERR_NETWORK_ACCESS_DENIED;
   }
 #endif
   return net::OK;
 }
+
+void ChromeNetworkDelegate::OnCacheWaitStateChange(
+    const net::URLRequest& request,
+    CacheWaitState state) {
+  cache_stats_->OnCacheWaitStateChange(request, state);
+}