Bug 209621 - [ Mac wk1 ] fast/loader/child-frame-add-after-back-forward.html is flaky timing out.
Summary: [ Mac wk1 ] fast/loader/child-frame-add-after-back-forward.html is flaky tim...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Frames (show other bugs)
Version: WebKit Nightly Build
Hardware: Mac macOS 10.14
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-03-26 14:41 PDT by Jason Lawrence
Modified: 2020-04-02 11:12 PDT (History)
9 users (show)

See Also:


Attachments
Patch (2.62 KB, patch)
2020-04-02 09:58 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Lawrence 2020-03-26 14:41:54 PDT
fast/loader/child-frame-add-after-back-forward.html 

Description:
This test is flaky timing out on Mac wk1. The flaky timeouts are apparent through most of the visible history, with the oldest appearing on 12/31/2019.

History:
https://results.webkit.org/?suite=layout-tests&test=fast%2Floader%2Fchild-frame-add-after-back-forward.html&platform=mac&flavor=wk1&style=release&style=debug&limit=50000

Diff:
--- /Volumes/Data/slave/catalina-debug-tests-wk1/build/layout-test-results/fast/loader/child-frame-add-after-back-forward-expected.txt
+++ /Volumes/Data/slave/catalina-debug-tests-wk1/build/layout-test-results/fast/loader/child-frame-add-after-back-forward-actual.txt
@@ -1,8 +1,5 @@
 main frame - has 1 onunload handler(s)
-main frame - has 1 onunload handler(s)
-Go forward then back.  
+FAIL: Timed out waiting for notifyDone to be called
+This tests the parsing of charset in the meta tag. The following should be a Hebrew word:
 
---------
-Frame: '<!--frame1-->'
---------
-PASS
+עברית
Comment 1 Radar WebKit Bug Importer 2020-03-26 14:43:21 PDT
<rdar://problem/60937276>
Comment 2 Jason Lawrence 2020-03-26 14:51:21 PDT
I have marked this test as timing out while this issue is investigated.
https://trac.webkit.org/changeset/259082/webkit
Comment 3 Chris Dumez 2020-04-01 15:56:23 PDT
Looks like this could be a test harness issue, not an issue with the test in question. The output seems to be for another test:
fast/loader/charset-parse.html

Likely the preceding test.
Comment 4 Chris Dumez 2020-04-01 15:58:37 PDT
(In reply to Chris Dumez from comment #3)
> Looks like this could be a test harness issue, not an issue with the test in
> question. The output seems to be for another test:
> fast/loader/charset-parse.html
> 
> Likely the preceding test.

Or maybe  fast/loader/child-frame-add-after-back-forward.html ends up navigating too far back in some cases (back to the previous test which was loaded in the same view). I think this makes some sense.
Comment 5 Chris Dumez 2020-04-02 09:36:02 PDT
Unable to reproduce so far. If someone is, please let me know how.
Comment 6 Chris Dumez 2020-04-02 09:53:48 PDT
I don't see anything particularly wrong or racy with the test..
Comment 7 Chris Dumez 2020-04-02 09:58:08 PDT
Created attachment 395275 [details]
Patch
Comment 8 EWS 2020-04-02 11:12:50 PDT
Committed r259395: <https://trac.webkit.org/changeset/259395>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 395275 [details].