Breakdown ClientLoadedTime histograms into buckets based on number of clients. Fix RequestTimeThrot…

content/browser/loader/resource_scheduler.h

Index: content/browser/loader/resource_scheduler.h
diff --git a/content/browser/loader/resource_scheduler.h b/content/browser/loader/resource_scheduler.h
index 2b8d7373b4f2c92f20ee194a182e4680a413cb25..e5190d119e35b12b78467ecbb99aef2d91cb7f43 100644
--- a/content/browser/loader/resource_scheduler.h
+++ b/content/browser/loader/resource_scheduler.h
@@ -176,6 +176,9 @@ class CONTENT_EXPORT ResourceScheduler : public base::NonThreadSafe {
   // Called when a ScheduledResourceRequest is destroyed.
   void RemoveRequest(ScheduledResourceRequest* request);
 
+  // Get the total number of Clients the ResourceScheduler knows about.
+  size_t GetNumClients() const;

[mmenke, 2015/01/06 19:30:03]

optional:  Not sure having a method to do this gets us anything.

[aiolos (Not reviewing), 2015/01/07 22:28:16]

Done.

+
   // These calls may update the ThrottleState of all clients, and have the
   // potential to be re-entrant.
   // Called when a Client newly becomes active loading.
@@ -220,6 +223,7 @@ class CONTENT_EXPORT ResourceScheduler : public base::NonThreadSafe {
   bool should_coalesce_;
   bool should_throttle_;
   ClientMap client_map_;
+  size_t num_clients_;

[mmenke, 2015/01/06 19:30:03]

Not used any more.

[aiolos (Not reviewing), 2015/01/07 22:28:16]

Done.

   size_t active_clients_loading_;
   size_t coalesced_clients_;
   // This is a repeating timer to initiate requests on COALESCED Clients.