Move out DidStartProvisionalLoad from WebContentsImpl into Navigator.

content/browser/frame_host/navigator.h

+// 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.
+
+#ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_
+#define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_
+
+#include "base/i18n/rtl.h"
+
+class GURL;
+
+namespace content {
+
+class NavigationControllerImpl;
+class NavigatorDelegate;
+class NavigatorObserver;
+class RenderFrameHost;
+
+class Navigator {
+ public:
+  Navigator(
+      NavigationControllerImpl* nav_controller,
+      NavigatorDelegate* delegate);
+
+  // The RenderFrame started a provisional load.
+  void DidStartProvisionalLoad(
+      RenderFrameHost* render_frame_host,
+      int64 frame_id,

[Charlie Reis, 2013/11/07 01:22:43]

Do we need to take in the frame_id and parent_frame_id if we have the
render_frame_host?  (It's ok if that's just temporary, until we switch over to
using browser-global FrameTreeNode IDs and renderer-specific RFH routing IDs.)

+      int64 parent_frame_id,
+      bool main_frame,
+      const GURL& url);
+
+  NavigationControllerImpl* controller() {
+    return controller_;
+  }
+
+  NavigatorDelegate* delegate() {
+    return delegate_;
+  }
+
+ private:
+  NavigationControllerImpl* controller_;
+
+  NavigatorDelegate* delegate_;
+};
+
+}

[awong, 2013/11/06 21:22:59]

nit: 
}  // namespace content

+
+#endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_