ResourceTiming should not depend on Document

Source/core/timing/PerformanceResourceTiming.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2012 Intel 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.
@@ skipping 14 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/timing/PerformanceResourceTiming.h"
 
-#include "core/dom/Document.h"
-#include "core/loader/DocumentLoadTiming.h"
-#include "core/loader/DocumentLoader.h"
 #include "core/timing/ResourceTimingInfo.h"
 #include "platform/network/ResourceRequest.h"
 #include "platform/network/ResourceResponse.h"
 
 namespace blink {
 
-static double monotonicTimeToDocumentMilliseconds(Document* document, double seconds)
+static double monotonicTimeToDOMHighResTimeStamp(double timeOrigin, double seconds)
 {
     ASSERT(seconds >= 0.0);
-    return document->loader()->timing().monotonicTimeToZeroBasedDocumentTime(seconds) * 1000.0;
+    if (!seconds)
+        return 0.0;
+    return (seconds - timeOrigin) * 1000.0;
 }
 
-PerformanceResourceTiming::PerformanceResourceTiming(const ResourceTimingInfo& info, Document* requestingDocument, double startTime, double lastRedirectEndTime, bool allowTimingDetails, bool allowRedirectDetails)
-    : PerformanceEntry(info.initialRequest().url().string(), "resource", monotonicTimeToDocumentMilliseconds(requestingDocument, startTime), monotonicTimeToDocumentMilliseconds(requestingDocument, info.loadFinishTime()))
+PerformanceResourceTiming::PerformanceResourceTiming(const ResourceTimingInfo& info, double timeOrigin, double startTime, double lastRedirectEndTime, bool allowTimingDetails, bool allowRedirectDetails)
+    : PerformanceEntry(info.initialRequest().url().string(), "resource", monotonicTimeToDOMHighResTimeStamp(timeOrigin, startTime), monotonicTimeToDOMHighResTimeStamp(timeOrigin, info.loadFinishTime()))
     , m_initiatorType(info.initiatorType())
-    , m_requestingDocument(requestingDocument)
+    , m_timeOrigin(timeOrigin)
     , m_timing(info.finalResponse().resourceLoadTiming())
     , m_lastRedirectEndTime(lastRedirectEndTime)
     , m_finishTime(info.loadFinishTime())
     , m_didReuseConnection(info.finalResponse().connectionReused())
     , m_allowTimingDetails(allowTimingDetails)
     , m_allowRedirectDetails(allowRedirectDetails)
 {
 }
 
 PerformanceResourceTiming::~PerformanceResourceTiming()
 {
 }
 
 AtomicString PerformanceResourceTiming::initiatorType() const
 {
     return m_initiatorType;
 }
 
 double PerformanceResourceTiming::workerStart() const
 {
     if (!m_timing || m_timing->workerStart() == 0.0)
         return 0.0;
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_timing->workerStart());
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_timing->workerStart());
 }
 
 double PerformanceResourceTiming::workerReady() const
 {
     if (!m_timing || m_timing->workerReady() == 0.0)
         return 0.0;
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_timing->workerReady());
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_timing->workerReady());
 }
 
 double PerformanceResourceTiming::redirectStart() const
 {
     if (!m_lastRedirectEndTime || !m_allowRedirectDetails)
         return 0.0;
 
     if (double workerReadyTime = workerReady())
         return workerReadyTime;
 
     return PerformanceEntry::startTime();
 }
 
 double PerformanceResourceTiming::redirectEnd() const
 {
     if (!m_lastRedirectEndTime || !m_allowRedirectDetails)
         return 0.0;
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_lastRedirectEndTime);
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_lastRedirectEndTime);
 }
 
 double PerformanceResourceTiming::fetchStart() const
 {
     if (m_lastRedirectEndTime) {
         // FIXME: ASSERT(m_timing) should be in constructor once timeticks of
         // AppCache is exposed from chrome network stack, crbug/251100
         ASSERT(m_timing);
-        return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_timing->requestTime());
+        return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_timing->requestTime());
     }
 
     if (double workerReadyTime = workerReady())
         return workerReadyTime;
 
     return PerformanceEntry::startTime();
 }
 
 double PerformanceResourceTiming::domainLookupStart() const
 {
     if (!m_allowTimingDetails)
         return 0.0;
 
     if (!m_timing || m_timing->dnsStart() == 0.0)
         return fetchStart();
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_timing->dnsStart());
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_timing->dnsStart());
 }
 
 double PerformanceResourceTiming::domainLookupEnd() const
 {
     if (!m_allowTimingDetails)
         return 0.0;
 
     if (!m_timing || m_timing->dnsEnd() == 0.0)
         return domainLookupStart();
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_timing->dnsEnd());
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_timing->dnsEnd());
 }
 
 double PerformanceResourceTiming::connectStart() const
 {
     if (!m_allowTimingDetails)
         return 0.0;
 
     // connectStart will be zero when a network request is not made.
     if (!m_timing || m_timing->connectStart() == 0.0 || m_didReuseConnection)
         return domainLookupEnd();
 
     // connectStart includes any DNS time, so we may need to trim that off.
     double connectStart = m_timing->connectStart();
     if (m_timing->dnsEnd() > 0.0)
         connectStart = m_timing->dnsEnd();
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), connectStart);
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, connectStart);
 }
 
 double PerformanceResourceTiming::connectEnd() const
 {
     if (!m_allowTimingDetails)
         return 0.0;
 
     // connectStart will be zero when a network request is not made.
     if (!m_timing || m_timing->connectEnd() == 0.0 || m_didReuseConnection)
         return connectStart();
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_timing->connectEnd());
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_timing->connectEnd());
 }
 
 double PerformanceResourceTiming::secureConnectionStart() const
 {
     if (!m_allowTimingDetails)
         return 0.0;
 
     if (!m_timing || m_timing->sslStart() == 0.0) // Secure connection not negotiated.
         return 0.0;
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_timing->sslStart());
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_timing->sslStart());
 }
 
 double PerformanceResourceTiming::requestStart() const
 {
     if (!m_allowTimingDetails)
         return 0.0;
 
     if (!m_timing)
         return connectEnd();
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_timing->sendStart());
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_timing->sendStart());
 }
 
 double PerformanceResourceTiming::responseStart() const
 {
     if (!m_allowTimingDetails)
         return 0.0;
 
     if (!m_timing)
         return requestStart();
 
     // FIXME: This number isn't exactly correct. See the notes in PerformanceTiming::responseStart().
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_timing->receiveHeadersEnd());
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_timing->receiveHeadersEnd());
 }
 
 double PerformanceResourceTiming::responseEnd() const
 {
     if (!m_finishTime)
         return responseStart();
 
-    return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_finishTime);
-}
-
-DEFINE_TRACE(PerformanceResourceTiming)
-{
-    visitor->trace(m_requestingDocument);
-    PerformanceEntry::trace(visitor);
+    return monotonicTimeToDOMHighResTimeStamp(m_timeOrigin, m_finishTime);
 }
 
 } // namespace blink