NonValidatingReload: Monitor reload operations in NavigationControllerImpl

content/browser/frame_host/navigation_controller_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_BROWSER_FRAME_HOST_NAVIGATION_CONTROLLER_IMPL_H_
 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_CONTROLLER_IMPL_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 432 matching lines @@
 
   // Used to smooth out timestamps from |get_timestamp_callback_|.
   // Without this, whenever there is a run of redirects or
   // code-generated navigations, those navigations may occur within
   // the timer resolution, leading to things sometimes showing up in
   // the wrong order in the history view.
   TimeSmoother time_smoother_;
 
   std::unique_ptr<NavigationEntryScreenshotManager> screenshot_manager_;
 
+  // Used to track the reload type triggered by an user action for the last
+  // navigation.

[Charlie Reis, 2016/09/19 22:23:51]

nit: Rephrase to:
Used for tracking consecutive reload requests.  If the last user-initiated
navigation (either browser-initiated or renderer-initiated with a user gesture)
was a reload, these hold the ReloadType and timestamp.  Otherwise these are
ReloadType::NONE and a null timestamp, respectively.

(Since they're closely related and can be covered by one comment, you can also
remove the blank line between them.)

[Takashi Toyoshima, 2016/09/26 12:23:34]

Done.

+  ReloadType last_committed_reload_type_;
+
+  // Used to track the timestamp for the last navigation when it was reload.
+  base::Time last_committed_reload_time_;
+
   DISALLOW_COPY_AND_ASSIGN(NavigationControllerImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_CONTROLLER_IMPL_H_