Set navigationStart correctly for all load types.

content/renderer/render_frame_impl.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_RENDERER_RENDER_FRAME_IMPL_H_
 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 1090 matching lines @@
 
   // Temporarily holds state pertaining to a navigation that has been initiated
   // until the NavigationState corresponding to the new navigation is created in
   // didCreateDataSource().
   std::unique_ptr<NavigationParams> pending_navigation_params_;
 
   // Stores the current history item for this frame, so that updates to it can
   // be reported to the browser process via SendUpdateState.
   blink::WebHistoryItem current_history_item_;
 
+  // True if the frame contains the initial document (i.e. has not navigated
+  // yet) and it has been accessed by another page.  Always false after the
+  // first commit.
+  bool did_access_initial_document_;

[nasko, 2016/07/18 22:20:00]

Why can't we just expose the bit that is already kept in blink and check that
instead of introducing another piece of state to keep track of? Especially since
it will be false for the majority of the lifetime of this object.

[Alexander Semashko, 2016/07/19 10:59:07]

Indeed. Done.

+
 #if defined(ENABLE_PLUGINS)
   // Current text input composition text. Empty if no composition is in
   // progress.
   base::string16 pepper_composition_text_;
 
   PluginPowerSaverHelper* plugin_power_saver_helper_;
 
   PepperPluginInstanceImpl* plugin_find_handler_;
 #endif
 
@@ skipping 180 matching lines @@
   mojom::FrameHostPtr frame_host_;
 
   base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_