Send navigation_start to the browser in DidStartProvisionalLoad IPC

content/public/test/render_view_test.h

Index: content/public/test/render_view_test.h
diff --git a/content/public/test/render_view_test.h b/content/public/test/render_view_test.h
index 5ae89cfff3f3e97c501faf464be268cb2965237e..02fb396f726b4ec0de55600fbe3912a5fe8584b4 100644
--- a/content/public/test/render_view_test.h
+++ b/content/public/test/render_view_test.h
@@ -36,6 +36,7 @@ class RendererScheduler;
 }
 
 namespace content {
+struct CommonNavigationParams;

[clamy, 2015/11/05 17:35:06]

I don't think these content internal classes should appear in the public API.

 class ContentBrowserClient;
 class ContentClient;
 class ContentRendererClient;
@@ -45,6 +46,8 @@ class PageState;
 class RendererMainPlatformDelegate;
 class RendererBlinkPlatformImplNoSandboxImpl;
 class RenderView;
+struct RequestNavigationParams;
+struct StartNavigationParams;
 
 class RenderViewTest : public testing::Test, blink::WebLeakDetectorClient {
  public:
@@ -197,7 +200,13 @@ class RenderViewTest : public testing::Test, blink::WebLeakDetectorClient {
   scoped_ptr<base::mac::ScopedNSAutoreleasePool> autorelease_pool_;
 #endif
 
- private:
+ protected:
+  // Will overwrite fields in request_params to prepare for navigation.
+  void GoToOffsetWithParams(int offset,

[clamy, 2015/11/05 17:35:06]

These content internal classes should not appear on the public API, especially
in a protected method (that could be used by a subclass outside of content).

+                            const PageState& state,
+                            const CommonNavigationParams common_params,
+                            const StartNavigationParams start_params,
+                            RequestNavigationParams request_params);
   void GoToOffset(int offset, const PageState& state);
 };