[Resource Timing] Populate responseEnd on cross-origin requests
authorsimonjam@chromium.org <simonjam@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 18 Dec 2012 20:14:09 +0000 (20:14 +0000)
committersimonjam@chromium.org <simonjam@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 18 Dec 2012 20:14:09 +0000 (20:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=105231

Reviewed by Tony Gentilcore.

No new tests. Covered by existing cross-origin test, but masked by iframe bug.

* page/PerformanceResourceTiming.cpp:
(WebCore::PerformanceResourceTiming::responseEnd):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@138051 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/page/PerformanceResourceTiming.cpp

index 9094bae..b0b8b30 100644 (file)
@@ -1,3 +1,15 @@
+2012-12-18  James Simonsen  <simonjam@chromium.org>
+
+        [Resource Timing] Populate responseEnd on cross-origin requests
+        https://bugs.webkit.org/show_bug.cgi?id=105231
+
+        Reviewed by Tony Gentilcore.
+
+        No new tests. Covered by existing cross-origin test, but masked by iframe bug.
+
+        * page/PerformanceResourceTiming.cpp:
+        (WebCore::PerformanceResourceTiming::responseEnd):
+
 2012-12-18  Tim Horton  <timothy_horton@apple.com>
 
         Unreviewed build fix after http://trac.webkit.org/changeset/138041
index f9d7de2..539878f 100644 (file)
@@ -203,10 +203,7 @@ double PerformanceResourceTiming::responseStart() const
 
 double PerformanceResourceTiming::responseEnd() const
 {
-    if (!m_shouldReportDetails)
-        return 0.0;
-
-    if (!m_timing)
+    if (!m_finishTime)
         return responseStart();
 
     return monotonicTimeToDocumentMilliseconds(m_requestingDocument.get(), m_finishTime);