Set navigationStart correctly for all load types.

content/public/test/mock_render_thread.h

Index: content/public/test/mock_render_thread.h
diff --git a/content/public/test/mock_render_thread.h b/content/public/test/mock_render_thread.h
index fcfb53fbfe546e954c2d01485f54736988c96b7f..125a24e277e6cfc3b85bfe6d731d3baeda2463a2 100644
--- a/content/public/test/mock_render_thread.h
+++ b/content/public/test/mock_render_thread.h
@@ -94,8 +94,18 @@ class MockRenderThread : public RenderThread {
 
   int32_t opener_id() const { return opener_id_; }
 
+  int32_t new_window_routing_id() const { return new_window_routing_id_; }
+
   void set_new_window_routing_id(int32_t id) { new_window_routing_id_ = id; }
 
+  int32_t new_window_main_frame_routing_id() const {
+    return new_window_main_frame_routing_id_;
+  }
+
+  void set_new_window_main_frame_routing_id(int32_t id) {

[clamy, 2016/07/21 11:52:44]

What is this used for?

[Alexander Semashko, 2016/07/21 14:00:22]

This id is put into the response to ViewHostMsg_CreateWindow, which makes the
new RenderView create a RenderFrame with given id. The field was there, it was
just not hooked up. Note that it is initialized to 0, which is still a valid
route id, and the frame is created even if the test does not call
set_new_window_main_frame_routing_id. Probably we should initialize them to
MSG_ROUTING_NONE and fix tests, but that's a separate issue and I did not have
time for it yet.

+    new_window_main_frame_routing_id_ = id;
+  }
+
   void set_new_window_main_frame_widget_routing_id(int32_t id) {
     new_window_main_frame_widget_routing_id_ = id;
   }