DevTools: Network requests are timed with 1.6 days.

chrome/browser/net/load_timing_observer.cc

 // Copyright (c) 2010 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/browser/net/load_timing_observer.h"
 
 #include "base/time.h"
 #include "chrome/browser/browser_thread.h"
 #include "chrome/browser/net/chrome_net_log.h"
 #include "chrome/common/resource_response.h"
 #include "net/base/load_flags.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_netlog_params.h"
 
 using base::Time;
 using base::TimeTicks;
 using webkit_glue::ResourceLoaderBridge;
 using webkit_glue::ResourceLoadTimingInfo;
 
 const size_t kMaxNumEntries = 1000;
 
 namespace {
 
+const int64 kSyncPeriod = 1000 * 1000 * 10;

[tonyg, 2010/12/16 22:21:03]

I'd recommend adding the units to the name: kSyncPeriodMicroseconds.

+
 // We know that this conversion is not solid and suffers from world clock
-// changes, but it should be good enough for the load timing info.
+// changes, but given that we sync clock every 10 seconds, it should be good
+// enough for the load timing info.
 static Time TimeTicksToTime(const TimeTicks& time_ticks) {
   static int64 tick_to_time_offset;
-  static bool tick_to_time_offset_available = false;
-  if (!tick_to_time_offset_available) {
+  static int64 last_sync_ticks = 0;
+  if (time_ticks.ToInternalValue() - last_sync_ticks > kSyncPeriod) {
     int64 cur_time = (Time::Now() - Time()).InMicroseconds();
     int64 cur_time_ticks = (TimeTicks::Now() - TimeTicks()).InMicroseconds();
     // If we add this number to a time tick value, it gives the timestamp.
     tick_to_time_offset = cur_time - cur_time_ticks;
-    tick_to_time_offset_available = true;
+    last_sync_ticks = time_ticks.ToInternalValue();
   }
   return Time::FromInternalValue(time_ticks.ToInternalValue() +
                                  tick_to_time_offset);
 }
 
 static int32 TimeTicksToOffset(
     const TimeTicks& time_ticks,
     LoadTimingObserver::URLRequestRecord* record) {
   return static_cast<int32>(
       (time_ticks - record->base_ticks).InMillisecondsRoundedUp());
@@ skipping 245 matching lines @@
   if (it == socket_to_record_.end())
     return;
 
   if (type == net::NetLog::TYPE_SSL_CONNECT) {
     if (is_begin)
       it->second.ssl_start = time;
     else if (is_end)
       it->second.ssl_end = time;
   }
 }