Prepare for per frame did{Start,Stop}Loading calls

content/renderer/load_progress_tracker.h

Index: content/renderer/load_progress_tracker.h
diff --git a/content/renderer/load_progress_tracker.h b/content/renderer/load_progress_tracker.h
index c4ed37eca9d679232fe47c7cbf59358c30179477..274b0a8e430b7a2255c7c01a6c0251a7c3519abe 100644
--- a/content/renderer/load_progress_tracker.h
+++ b/content/renderer/load_progress_tracker.h
@@ -5,15 +5,13 @@
 #ifndef CONTENT_RENDERER_LOAD_PROGRESS_TRACKER_H_
 #define CONTENT_RENDERER_LOAD_PROGRESS_TRACKER_H_
 
+#include "base/containers/hash_tables.h"
 #include "base/logging.h"
 #include "base/memory/weak_ptr.h"
 #include "base/time/time.h"
 
-namespace blink {
-class WebFrame;
-}
-
 namespace content {
+class RenderFrame;
 class RenderViewImpl;
 
 class LoadProgressTracker {
@@ -21,9 +19,10 @@ class LoadProgressTracker {
   explicit LoadProgressTracker(RenderViewImpl* render_view);
   ~LoadProgressTracker();
 
-  void DidStopLoading();
+  void DidStartLoading(RenderFrame* frame);
+  void DidStopLoading(RenderFrame* frame);
 
-  void DidChangeLoadProgress(blink::WebFrame* frame, double progress);
+  void DidChangeLoadProgress(RenderFrame* frame, double progress);
 
  private:
   void ResetStates();
@@ -32,9 +31,9 @@ class LoadProgressTracker {
 
   RenderViewImpl* render_view_;
 
-  blink::WebFrame* tracked_frame_;
-
-  double progress_;
+  typedef base::hash_map<int, double> ProgressMap;

[Charlie Reis, 2014/02/27 21:52:07]

Can you add a comment saying what the int and double represent?  (The double is
easy to guess, but would be useful to document the key.)

+  ProgressMap progresses_;
+  double total_progress_;
 
   base::TimeTicks last_time_progress_sent_;