Hook up loading signal to ResourceScheduler.

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 8d0efc5155c4b8aad81770b635a5ae8652e60766..da5cd06233a535673ed907891d32b1054aeacf97 100644
--- a/content/browser/loader/resource_scheduler.h
+++ b/content/browser/loader/resource_scheduler.h
@@ -116,6 +116,12 @@ class CONTENT_EXPORT ResourceScheduler : public base::NonThreadSafe {
   // Called when a renderer is destroyed.
   void OnClientDeleted(int child_id, int route_id);
 
+  // Called when a renderer stops or restarts loading.
+  void OnLoadingStateChanged(int child_id, int route_id, bool is_loaded);
+
+  // Called when a Client is shown or hidden.
+  void OnVisibilityChanged(int child_id, int route_id, bool is_visible);
+
   // Signals from IPC messages directly from the renderers:
 
   // Called when a client navigates to a new main document.
@@ -139,13 +145,10 @@ class CONTENT_EXPORT ResourceScheduler : public base::NonThreadSafe {
   // Called when a Client starts or stops playing audio.
   void OnAudibilityChanged(int child_id, int route_id, bool is_audible);
 
-  // Called when a Client is shown or hidden.
-  void OnVisibilityChanged(int child_id, int route_id, bool is_visible);
-
-  void OnLoadingStateChanged(int child_id, int route_id, bool is_loaded);
-
   bool IsClientVisibleForTesting(int child_id, int route_id);
 
+  bool IsClientLoadedForTesting(int child_id, int route_id);

[mmenke, 2014/08/26 18:16:06]

This isn't currently used - I'm fine with having this if we use it in tests, but
if we're not going to use it, I don't think we should have it.

[aiolos (Not reviewing), 2014/08/26 18:28:28]

It's easy enough to add back in if it ends up in tests. Removed for now.

+
  private:
   class RequestQueue;
   class ScheduledResourceRequest;