Adding histograms showing fraction of page load times

chrome/browser/net/chrome_url_request_context.h

 // 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.
 
 #ifndef CHROME_BROWSER_NET_CHROME_URL_REQUEST_CONTEXT_H_
 #define CHROME_BROWSER_NET_CHROME_URL_REQUEST_CONTEXT_H_
 
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/prefs/pref_change_registrar.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_context_getter.h"
 
 class ChromeURLDataManagerBackend;
 class ChromeURLRequestContextFactory;
 class IOThread;
 class Profile;
 class ProfileIOData;
 
+namespace chrome_browser_net {
+class CacheStats;
+}
+
 // Subclass of net::URLRequestContext which can be used to store extra
 // information for requests.
 //
 // All methods of this class must be called from the IO thread,
 // including the constructor and destructor.
 class ChromeURLRequestContext : public net::URLRequestContext {
  public:
-  ChromeURLRequestContext();
+  enum ContextType {
+    CONTEXT_TYPE_MAIN,
+    CONTEXT_TYPE_MEDIA,
+    CONTEXT_TYPE_EXTENSIONS,
+    CONTEXT_TYPE_APP
+  };
+  ChromeURLRequestContext(ContextType type,
+                          chrome_browser_net::CacheStats* cache_stats);
   virtual ~ChromeURLRequestContext();
 
   base::WeakPtr<ChromeURLRequestContext> GetWeakPtr() {
     return weak_factory_.GetWeakPtr();
   }
 
   // Copies the state from |other| into this context.
   void CopyFrom(ChromeURLRequestContext* other);
 
   bool is_incognito() const {
@@ skipping 17 matching lines @@
   void OnAcceptLanguageChange(const std::string& accept_language);
 
   // Callback for when the default charset changes.
   void OnDefaultCharsetChange(const std::string& default_charset);
 
  private:
   base::WeakPtrFactory<ChromeURLRequestContext> weak_factory_;
 
   // ---------------------------------------------------------------------------
   // Important: When adding any new members below, consider whether they need to
   // be added to CopyFrom.

[rvargas (doing something else), 2012/07/26 18:47:33]

did you see this?

[tburkard, 2012/07/26 19:12:49]

Yes, nothing to be done here though, because it is taken care of in the
constructor (initialization of the newly added variables/state).

On 2012/07/26 18:47:33, rvargas wrote:

   // ---------------------------------------------------------------------------
 
   ChromeURLDataManagerBackend* chrome_url_data_manager_backend_;
   bool is_incognito_;
+  chrome_browser_net::CacheStats* cache_stats_;
 
   // ---------------------------------------------------------------------------
   // Important: When adding any new members above, consider whether they need to
   // be added to CopyFrom.
   // ---------------------------------------------------------------------------
 
   DISALLOW_COPY_AND_ASSIGN(ChromeURLRequestContext);
 };
 
 // A net::URLRequestContextGetter subclass used by the browser. This returns a
@@ skipping 98 matching lines @@
 
   // NULL if not yet initialized. Otherwise, it is the ChromeURLRequestContext
   // instance that was lazily created by GetURLRequestContext().
   // Access only from the IO thread.
   base::WeakPtr<ChromeURLRequestContext> url_request_context_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeURLRequestContextGetter);
 };
 
 #endif  // CHROME_BROWSER_NET_CHROME_URL_REQUEST_CONTEXT_H_