Title: [136920] trunk
Revision
136920
Author
simon...@chromium.org
Date
2012-12-06 21:08:25 -0800 (Thu, 06 Dec 2012)

Log Message

[Resource Timing] Allow detailed timing on same origin sites
https://bugs.webkit.org/show_bug.cgi?id=104328

Reviewed by Tony Gentilcore.

Source/WebCore:

This block was lost when I moved it to PerformanceResourceTiming.cpp.

Test: http/tests/w3c/webperf/submission/resource-timing/html/test_resource_attribute_order.html

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

LayoutTests:

* platform/chromium/TestExpectations: This was mismarked as an iframe failure.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (136919 => 136920)


--- trunk/LayoutTests/ChangeLog	2012-12-07 03:07:24 UTC (rev 136919)
+++ trunk/LayoutTests/ChangeLog	2012-12-07 05:08:25 UTC (rev 136920)
@@ -1,3 +1,12 @@
+2012-12-06  James Simonsen  <simon...@chromium.org>
+
+        [Resource Timing] Allow detailed timing on same origin sites
+        https://bugs.webkit.org/show_bug.cgi?id=104328
+
+        Reviewed by Tony Gentilcore.
+
+        * platform/chromium/TestExpectations: This was mismarked as an iframe failure.
+
 2012-12-06  Rick Byers  <rby...@chromium.org>
 
         CSS cursor property should support webkit-image-set

Modified: trunk/LayoutTests/platform/chromium/TestExpectations (136919 => 136920)


--- trunk/LayoutTests/platform/chromium/TestExpectations	2012-12-07 03:07:24 UTC (rev 136919)
+++ trunk/LayoutTests/platform/chromium/TestExpectations	2012-12-07 05:08:25 UTC (rev 136920)
@@ -4166,7 +4166,6 @@
 
 webkit.org/b/102724 svg/carto.net/colourpicker.svg [ ImageOnlyFailure ]
 
-webkit.org/b/103927 http/tests/w3c/webperf/submission/resource-timing/html/test_resource_attribute_order.html [ Text ]
 webkit.org/b/103927 http/tests/w3c/webperf/submission/resource-timing/html/test_resource_dynamic_insertion.html [ Text ]
 webkit.org/b/103927 http/tests/w3c/webperf/submission/resource-timing/html/test_resource_frame_initiator_type.html [ Text ]
 webkit.org/b/103927 http/tests/w3c/webperf/submission/resource-timing/html/test_resource_initiator_types.html [ Text ]

Modified: trunk/Source/WebCore/ChangeLog (136919 => 136920)


--- trunk/Source/WebCore/ChangeLog	2012-12-07 03:07:24 UTC (rev 136919)
+++ trunk/Source/WebCore/ChangeLog	2012-12-07 05:08:25 UTC (rev 136920)
@@ -1,3 +1,17 @@
+2012-12-06  James Simonsen  <simon...@chromium.org>
+
+        [Resource Timing] Allow detailed timing on same origin sites
+        https://bugs.webkit.org/show_bug.cgi?id=104328
+
+        Reviewed by Tony Gentilcore.
+
+        This block was lost when I moved it to PerformanceResourceTiming.cpp.
+
+        Test: http/tests/w3c/webperf/submission/resource-timing/html/test_resource_attribute_order.html
+
+        * page/PerformanceResourceTiming.cpp:
+        (WebCore::passesTimingAllowCheck):
+
 2012-12-06  Rick Byers  <rby...@chromium.org>
 
         CSS cursor property should support webkit-image-set

Modified: trunk/Source/WebCore/page/PerformanceResourceTiming.cpp (136919 => 136920)


--- trunk/Source/WebCore/page/PerformanceResourceTiming.cpp	2012-12-07 03:07:24 UTC (rev 136919)
+++ trunk/Source/WebCore/page/PerformanceResourceTiming.cpp	2012-12-07 05:08:25 UTC (rev 136920)
@@ -55,6 +55,10 @@
 {
     AtomicallyInitializedStatic(AtomicString&, timingAllowOrigin = *new AtomicString("timing-allow-origin"));
 
+    RefPtr<SecurityOrigin> resourceOrigin = SecurityOrigin::create(response.url());
+    if (resourceOrigin->isSameSchemeHostPort(requestingDocument->securityOrigin()))
+        return true;
+
     const String& timingAllowOriginString = response.httpHeaderField(timingAllowOrigin);
     if (timingAllowOriginString.isEmpty() || equalIgnoringCase(timingAllowOriginString, "null"))
         return false;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to