chrome.loadTimes() shouldn't be affected by in-document navigation.

chrome/renderer/loadtimes_extension_bindings.cc

 // Copyright (c) 2006-2009 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 "chrome/renderer/loadtimes_extension_bindings.h"
 
 #include <math.h>
 
 #include "base/time.h"
 #include "content/public/renderer/navigation_state.h"
@@ skipping 84 matching lines @@
     return kTransitionOther;
   }
 
   static v8::Handle<v8::Value> GetLoadTimes(const v8::Arguments& args) {
     WebFrame* frame = WebFrame::frameForCurrentContext();
     if (frame) {
       WebDataSource* data_source = frame->dataSource();
       if (data_source) {
         NavigationState* navigation_state =
             NavigationState::FromDataSource(data_source);
+        NavigationState::LoadTimes* navigation_times =
+            navigation_state->load_times();
         v8::Local<v8::Object> load_times = v8::Object::New();
         load_times->Set(
             v8::String::New("requestTime"),
-            v8::Number::New(navigation_state->request_time().ToDoubleT()));
+            v8::Number::New(navigation_times->request_time().ToDoubleT()));
         load_times->Set(
             v8::String::New("startLoadTime"),
-            v8::Number::New(navigation_state->start_load_time().ToDoubleT()));
+            v8::Number::New(navigation_times->start_load_time().ToDoubleT()));
         load_times->Set(
             v8::String::New("commitLoadTime"),
-            v8::Number::New(navigation_state->commit_load_time().ToDoubleT()));
+            v8::Number::New(navigation_times->commit_load_time().ToDoubleT()));
         load_times->Set(
             v8::String::New("finishDocumentLoadTime"),
             v8::Number::New(
-                navigation_state->finish_document_load_time().ToDoubleT()));
+                navigation_times->finish_document_load_time().ToDoubleT()));
         load_times->Set(
             v8::String::New("finishLoadTime"),
-            v8::Number::New(navigation_state->finish_load_time().ToDoubleT()));
+            v8::Number::New(navigation_times->finish_load_time().ToDoubleT()));
         load_times->Set(
             v8::String::New("firstPaintTime"),
-            v8::Number::New(navigation_state->first_paint_time().ToDoubleT()));
+            v8::Number::New(navigation_times->first_paint_time().ToDoubleT()));
         load_times->Set(
             v8::String::New("firstPaintAfterLoadTime"),
             v8::Number::New(
-                navigation_state->first_paint_after_load_time().ToDoubleT()));
+                navigation_times->first_paint_after_load_time().ToDoubleT()));
         load_times->Set(
             v8::String::New("navigationType"),
             v8::String::New(GetNavigationType(data_source->navigationType())));
         load_times->Set(
             v8::String::New("wasFetchedViaSpdy"),
-            v8::Boolean::New(navigation_state->was_fetched_via_spdy()));
+            v8::Boolean::New(navigation_times->was_fetched_via_spdy()));
         load_times->Set(
             v8::String::New("wasNpnNegotiated"),
-            v8::Boolean::New(navigation_state->was_npn_negotiated()));
+            v8::Boolean::New(navigation_times->was_npn_negotiated()));
         load_times->Set(
             v8::String::New("wasAlternateProtocolAvailable"),
             v8::Boolean::New(
-                navigation_state->was_alternate_protocol_available()));
+                navigation_times->was_alternate_protocol_available()));
         return load_times;
       }
     }
     return v8::Null();
   }
 
   static v8::Handle<v8::Value> GetCSI(const v8::Arguments& args) {
     WebFrame* frame = WebFrame::frameForCurrentContext();
     if (frame) {
       WebDataSource* data_source = frame->dataSource();
       if (data_source) {
         NavigationState* navigation_state =
             NavigationState::FromDataSource(data_source);
+        NavigationState::LoadTimes* navigation_times =
+            navigation_state->load_times();
         v8::Local<v8::Object> csi = v8::Object::New();
         base::Time now = base::Time::Now();
-        base::Time start = navigation_state->request_time().is_null() ?
-            navigation_state->start_load_time() :
-            navigation_state->request_time();
-        base::Time onload = navigation_state->finish_document_load_time();
+        base::Time start = navigation_times->request_time().is_null() ?
+            navigation_times->start_load_time() :
+            navigation_times->request_time();
+        base::Time onload = navigation_times->finish_document_load_time();
         base::TimeDelta page = now - start;
         csi->Set(
             v8::String::New("startE"),
             v8::Number::New(floor(start.ToDoubleT() * 1000)));
         csi->Set(
             v8::String::New("onloadT"),
             v8::Number::New(floor(onload.ToDoubleT() * 1000)));
         csi->Set(
           v8::String::New("pageT"),
           v8::Number::New(page.InMillisecondsF()));
         csi->Set(
             v8::String::New("tran"),
             v8::Number::New(
                 GetCSITransitionType(data_source->navigationType())));
 
         return csi;
       }
     }
     return v8::Null();
   }
 };
 
 v8::Extension* LoadTimesExtension::Get() {
   return new LoadTimesExtensionWrapper();
 }
 
 }  // namespace extensions_v8