Correctly record and pass around navigation start time.

components/web_view/navigation_entry.cc

 // Copyright 2015 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.
 
 #include "components/web_view/navigation_entry.h"
 
 namespace web_view {
 
 NavigationEntry::NavigationEntry(mojo::URLRequestPtr original_request)
     : url_request_(original_request.Pass()) {}
 
 NavigationEntry::NavigationEntry(const GURL& raw_url)
     : url_request_(raw_url) {}
 
 NavigationEntry::~NavigationEntry() {}
 
-mojo::URLRequestPtr NavigationEntry::BuildURLRequest() const {
-  return url_request_.Clone();
+mojo::URLRequestPtr NavigationEntry::BuildURLRequest() {

[sky, 2015/10/15 13:14:46]

Instead of implying whether the time is reset based on state can
NavigationController explicitly specify what it wants to happen? I believe
NavigationController knows back/forward vs new.

[yzshen1, 2015/10/15 18:23:08]

Done.

+  mojo::URLRequestPtr clone = url_request_.Clone();
+  if (clone->originating_time_ticks != 0) {
+    // Forget the originating time, so that we will use Now() when this entry is
+    // used for navigation again (e.g., back/forward navigation).
+    url_request_.set_originating_time(base::TimeTicks());
+  } else {
+    // Use Now() as the request originating time.
+    clone->originating_time_ticks = base::TimeTicks::Now().ToInternalValue();
+  }
+  return clone.Pass();
 }
 
 }  // namespace web_view