Move LoadProgressTracker to the browser process.

content/browser/frame_host/navigator_impl.cc

 // Copyright 2013 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.
 
 #include "content/browser/frame_host/navigator_impl.h"
 
 #include "base/command_line.h"
 #include "content/browser/frame_host/frame_tree.h"
 #include "content/browser/frame_host/frame_tree_node.h"
+#include "content/browser/frame_host/load_progress_tracker.h"
 #include "content/browser/frame_host/navigation_controller_impl.h"
 #include "content/browser/frame_host/navigation_entry_impl.h"
 #include "content/browser/frame_host/navigator_delegate.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/site_instance_impl.h"
 #include "content/browser/webui/web_ui_controller_factory_registry.h"
 #include "content/browser/webui/web_ui_impl.h"
 #include "content/common/frame_messages.h"
 #include "content/common/view_messages.h"
@@ skipping 107 matching lines @@
   return rfh->frame_tree_node()->frame_tree()->root()->render_manager();
 }
 
 }  // namespace
 
 
 NavigatorImpl::NavigatorImpl(
     NavigationControllerImpl* navigation_controller,
     NavigatorDelegate* delegate)
     : controller_(navigation_controller),
-      delegate_(delegate) {
+      delegate_(delegate),
+      load_progress_tracker_(new LoadProgressTracker(delegate)) {
+}
+
+NavigatorImpl::~NavigatorImpl() {
 }
 
 NavigationController* NavigatorImpl::GetController() {
   return controller_;
 }
 
 void NavigatorImpl::DidStartProvisionalLoad(
     RenderFrameHostImpl* render_frame_host,
     int parent_routing_id,
     const GURL& url) {
@@ skipping 236 matching lines @@
 
 bool NavigatorImpl::NavigateToPendingEntry(
     RenderFrameHostImpl* render_frame_host,
     NavigationController::ReloadType reload_type) {
   return NavigateToEntry(
       render_frame_host,
       *NavigationEntryImpl::FromNavigationEntry(controller_->GetPendingEntry()),
       reload_type);
 }
 
+void NavigatorImpl::DidStartLoading(RenderFrameHostImpl* render_frame_host,
+                                    bool to_different_document) {
+  if (delegate_)
+    delegate_->DidStartLoading(render_frame_host, to_different_document);
+  load_progress_tracker_->DidStartLoading(render_frame_host->GetRoutingID());
+}
+
+void NavigatorImpl::DidStopLoading(RenderFrameHostImpl* render_frame_host) {
+  if (delegate_)

[nasko, 2014/05/02 22:48:23]

Don't we have to track that all frames are complete loading before dispatching
this to the delegate? If we have two frames in two processes, each will send
DidStopLoading. This will cause the delegate to think everything is loaded
whenever the first frame is complete.

[Avi (use Gerrit), 2014/05/05 15:15:02]

Let me think about this one for a while.

+    delegate_->DidStopLoading(render_frame_host);
+  load_progress_tracker_->DidStopLoading(render_frame_host->GetRoutingID());
+}
+
+void NavigatorImpl::DidChangeLoadProgress(
+    RenderFrameHostImpl* render_frame_host,
+    double load_progress) {
+  load_progress_tracker_->DidChangeLoadProgress(
+      render_frame_host->GetRoutingID(), load_progress);
+}
+
 base::TimeTicks NavigatorImpl::GetCurrentLoadStart() {
   return current_load_start_;
 }
 
 void NavigatorImpl::DidNavigate(
     RenderFrameHostImpl* render_frame_host,
     const FrameHostMsg_DidCommitProvisionalLoad_Params& input_params) {
   FrameHostMsg_DidCommitProvisionalLoad_Params params(input_params);
   FrameTree* frame_tree = render_frame_host->frame_tree_node()->frame_tree();
   bool use_site_per_process =
@@ skipping 210 matching lines @@
 
     // Navigations in Web UI pages count as browser-initiated navigations.
     params.is_renderer_initiated = false;
   }
 
   if (delegate_)
     delegate_->RequestOpenURL(render_frame_host, params);
 }
 
 }  // namespace content