Title: [280956] trunk/Source/WebCore
Revision
280956
Author
[email protected]
Date
2021-08-11 21:29:25 -0700 (Wed, 11 Aug 2021)

Log Message

Remove optimistic assertion added in r280931
https://bugs.webkit.org/show_bug.cgi?id=229008

* page/PerformanceTiming.cpp:
(WebCore::PerformanceTiming::fetchStart const):
This assertion is hit in the test fast/dom/Window/property-access-on-cached-window-after-frame-removed.html
when accessing window.performance.fetchStart from a detached window object from a frame that has been removed from the DOM.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (280955 => 280956)


--- trunk/Source/WebCore/ChangeLog	2021-08-12 03:48:28 UTC (rev 280955)
+++ trunk/Source/WebCore/ChangeLog	2021-08-12 04:29:25 UTC (rev 280956)
@@ -1,3 +1,13 @@
+2021-08-11  Alex Christensen  <[email protected]>
+
+        Remove optimistic assertion added in r280931
+        https://bugs.webkit.org/show_bug.cgi?id=229008
+
+        * page/PerformanceTiming.cpp:
+        (WebCore::PerformanceTiming::fetchStart const):
+        This assertion is hit in the test fast/dom/Window/property-access-on-cached-window-after-frame-removed.html
+        when accessing window.performance.fetchStart from a detached window object from a frame that has been removed from the DOM.
+
 2021-08-11  Chris Dumez  <[email protected]>
 
         Add initial support for Cross-Origin-Embedder-Policy (COEP)

Modified: trunk/Source/WebCore/page/PerformanceTiming.cpp (280955 => 280956)


--- trunk/Source/WebCore/page/PerformanceTiming.cpp	2021-08-12 03:48:28 UTC (rev 280955)
+++ trunk/Source/WebCore/page/PerformanceTiming.cpp	2021-08-12 04:29:25 UTC (rev 280956)
@@ -145,8 +145,6 @@
             m_fetchStart = monotonicTimeToIntegerMilliseconds(timing->startTime());
     }
 
-    // Like PerformanceResourceTiming::fetchStart, fetchStart is a required property
-    ASSERT(m_fetchStart);
     return m_fetchStart;
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to