Bug 228571 – [ Mac ] fast/dom/webtiming-document-open.html is a flaky failure

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Eric Hutchison]

fast/dom/webtiming-document-open.html 

is a flaky failure.

History: https://results.webkit.org/?platform=mac&suite=layout-tests&test=fast%2Fdom%2Fwebtiming-document-open.html

Results: https://build.webkit.org/results/Apple-BigSur-Debug-WK2-Tests/r280347%20(2979)/results.html

Diff: 
--- /Volumes/Data/worker/bigsur-debug-tests-wk2/build/layout-test-results/fast/dom/webtiming-document-open-expected.txt
+++ /Volumes/Data/worker/bigsur-debug-tests-wk2/build/layout-test-results/fast/dom/webtiming-document-open-actual.txt
@@ -13,7 +13,7 @@
 PASS timing.domainLookupEnd is originalTiming.domainLookupEnd
 PASS timing.domainLookupStart is originalTiming.domainLookupStart
 PASS timing.fetchStart is originalTiming.fetchStart
-PASS timing.loadEventEnd is originalTiming.loadEventEnd
+FAIL timing.loadEventEnd should be 1627422346940. Was 1627422346941.
 PASS timing.loadEventStart is originalTiming.loadEventStart
 PASS timing.navigationStart is originalTiming.navigationStart
 PASS timing.redirectEnd is originalTiming.redirectEnd

Reproduced on r280380 using run-webkit-tests  -f --iterations 500 --exit-after-n-failures 1 --clobber-old-results fast/dom/webtiming-document-open.html.

[Radar WebKit Bug Importer]

<rdar://problem/81240846>

[Eric Hutchison]

Updated test expectations at https://trac.webkit.org/changeset/280408/webkit.

[Alex Christensen]

Created attachment 434532 [details]
Patch

[Chris Dumez]

Comment on attachment 434532 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=434532&action=review

> Source/WebCore/page/PerformanceTiming.cpp:58
>          return 0;

Is it intentional in all these getters that we're not caching in early return cases?

> Source/WebCore/page/PerformanceTiming.cpp:80
> +    m_unloadEventStart =  monotonicTimeToIntegerMilliseconds(timing->unloadEventStart());

nit: extra space

[Alex Christensen]

Comment on attachment 434532 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=434532&action=review

>> Source/WebCore/page/PerformanceTiming.cpp:58
>>          return 0;
> 
> Is it intentional in all these getters that we're not caching in early return cases?

All those early returns are unlikely to be hit, and if we do and the website queries multiple times then we'll early return multiple times.
This whole API has been replaced, MDN says it's deprecated and not to use it.  It will be removed a bit after bug 227336 is fixed.

[Alex Christensen]

http://trac.webkit.org/r280454