Set navigationStart correctly for all load types.

content/public/test/mock_render_thread.h

 // Copyright (c) 2012 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_PUBLIC_TEST_MOCK_RENDER_THREAD_H_
 #define CONTENT_PUBLIC_TEST_MOCK_RENDER_THREAD_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 76 matching lines @@
   shell::InterfaceRegistry* GetInterfaceRegistry() override;
   shell::InterfaceProvider* GetRemoteInterfaces() override;
 
   //////////////////////////////////////////////////////////////////////////
   // The following functions are called by the test itself.
 
   void set_routing_id(int32_t id) { routing_id_ = id; }
 
   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;
   }
 
   void set_new_frame_routing_id(int32_t id) { new_frame_routing_id_ = id; }
 
   // Simulates the Widget receiving a close message. This should result
   // on releasing the internal reference counts and destroying the internal
   // state.
   void SendCloseMessage();
@@ skipping 54 matching lines @@
   cc::TestSharedBitmapManager shared_bitmap_manager_;
   std::unique_ptr<shell::InterfaceRegistry> interface_registry_;
   std::unique_ptr<shell::InterfaceProvider> remote_interfaces_;
   shell::mojom::InterfaceProviderRequest
       pending_remote_interface_provider_request_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_TEST_MOCK_RENDER_THREAD_H_