Changeset 280956 in webkit

Timestamp:

Aug 11, 2021, 9:29:25 PM (5 years ago)

Author:

achristensen@apple.com

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.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/PerformanceTiming.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2021-08-11  Alex Christensen  <achristensen@webkit.org>
+
+        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  <cdumez@apple.com>
 

trunk/Source/WebCore/page/PerformanceTiming.cpp

@@ -146 +146 @@
     }
 
-    // Like PerformanceResourceTiming::fetchStart, fetchStart is a required property
-    ASSERT(m_fetchStart);
     return m_fetchStart;
 }