Fix ResourceLoadTiming resolution lose issue.

Source/core/inspector/InspectorResourceAgent.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 90 matching lines @@
 {
     RefPtr<InspectorObject> headersObject = InspectorObject::create();
     HTTPHeaderMap::const_iterator end = headers.end();
     for (HTTPHeaderMap::const_iterator it = headers.begin(); it != end; ++it)
         headersObject->setString(it->key.string(), it->value);
     return headersObject;
 }
 
 static PassRefPtr<TypeBuilder::Network::ResourceTiming> buildObjectForTiming(const ResourceLoadTiming& timing, DocumentLoader* loader)
 {
+#ifdef ENABLE_DOUBLE_RESOURCE_LOAD_TIMING
+    double requestTime = timing.requestTime;
+    return TypeBuilder::Network::ResourceTiming::create()
+        .setRequestTime(loader->timing()->monotonicTimeToPseudoWallTime(requestTime))
+        .setProxyStart((timing.proxyStart - requestTime) * 1000)
+        .setProxyEnd((timing.proxyEnd - requestTime) * 1000)
+        .setDnsStart((timing.dnsStart - requestTime) * 1000)
+        .setDnsEnd((timing.dnsEnd - requestTime) * 1000)
+        .setConnectStart((timing.connectStart - requestTime) * 1000)
+        .setConnectEnd((timing.connectEnd - requestTime) * 1000)
+        .setSslStart((timing.sslStart - requestTime) * 1000)
+        .setSslEnd((timing.sslEnd - requestTime) * 1000)
+        .setSendStart((timing.sendStart - requestTime) * 1000)
+        .setSendEnd((timing.sendEnd - requestTime) * 1000)
+        .setReceiveHeadersEnd((timing.receiveHeadersEnd - requestTime) * 1000)

[abarth-chromium, 2013/05/22 18:19:05]

Should we create an inline helper function to do this conversion so we don't
need to do it so many times?

[James Simonsen, 2013/05/22 21:36:59]

Alternatively, why not just store the absolute values everywhere instead of
converting them to deltas (which will then be converted back to absolute values
in window.performance)?

+        .release();
+#else
     return TypeBuilder::Network::ResourceTiming::create()
         .setRequestTime(loader->timing()->monotonicTimeToPseudoWallTime(timing.convertResourceLoadTimeToMonotonicTime(0)))
         .setProxyStart(timing.proxyStart)
         .setProxyEnd(timing.proxyEnd)
         .setDnsStart(timing.dnsStart)
         .setDnsEnd(timing.dnsEnd)
         .setConnectStart(timing.connectStart)
         .setConnectEnd(timing.connectEnd)
         .setSslStart(timing.sslStart)
         .setSslEnd(timing.sslEnd)
         .setSendStart(timing.sendStart)
         .setSendEnd(timing.sendEnd)
         .setReceiveHeadersEnd(timing.receiveHeadersEnd)
         .release();
+#endif
 }
 
 static PassRefPtr<TypeBuilder::Network::Request> buildObjectForResourceRequest(const ResourceRequest& request)
 {
     RefPtr<TypeBuilder::Network::Request> requestObject = TypeBuilder::Network::Request::create()
         .setUrl(request.url().string())
         .setMethod(request.httpMethod())
         .setHeaders(buildObjectForHeaders(request.httpHeaderFields()));
     if (request.httpBody() && !request.httpBody()->isEmpty())
         requestObject->setPostData(request.httpBody()->flattenToString());
@@ skipping 538 matching lines @@
     , m_client(client)
     , m_frontend(0)
     , m_resourcesData(adoptPtr(new NetworkResourcesData()))
     , m_loadingXHRSynchronously(false)
     , m_isRecalculatingStyle(false)
 {
 }
 
 } // namespace WebCore