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

content/renderer/load_progress_tracker.h

 // Copyright (c) 2011 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 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 {
  public:
   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();
 
   void SendChangeLoadProgress();
 
   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_;
 
   base::WeakPtrFactory<LoadProgressTracker> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(LoadProgressTracker);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_LOAD_PROGRESS_TRACKER_H_