Bug 190202 - REGRESSION(r236154) Page loads are not drawn when loaded in shipped Safari11/12
Summary: REGRESSION(r236154) Page loads are not drawn when loaded in shipped Safari11/12
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alex Christensen
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-10-02 08:50 PDT by youenn fablet
Modified: 2018-10-05 15:27 PDT (History)
7 users (show)

See Also:


Attachments
Patch (1.76 KB, patch)
2018-10-05 14:15 PDT, Alex Christensen
youennf: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2018-10-02 08:50:50 PDT
After this revision, page loads are hanging on previous Safari version.
This was reported for Safari high Sierra and Safari Mojave.
I bissected the issue with WebKit Nightlies to  http://trac.webkit.org/r236153
Comment 1 Rob Buis 2018-10-02 08:55:29 PDT
I have this problem on High Sierra and I am seeing:
./Tools/Scripts/run-safari --no-saved-state
Starting SafariForWebKitDevelopment with DYLD_FRAMEWORK_PATH set to point to built WebKit in /Users/rwlbuis/WebKit/WebKitBuild/Release.
Failed to acquire exclusive access to AutoFill corrections SQLite store at AutoFillCorrections.db.
Failed to acquire exclusive access to AutoFill corrections SQLite store at CloudAutoFillCorrections.db.
Failed to acquire exclusive access to AutoFill corrections SQLite store at AutoFillCorrections.db.
Failed to acquire exclusive access to AutoFill corrections SQLite store at CloudAutoFillCorrections.db.
2018-10-02 17:55:14.088 SafariForWebKitDevelopment[50142:289352] It's not legal to call -layoutSubtreeIfNeeded on a view which is already being laid out.  If you are implementing the view's -layout method, you can call -[super layout] instead. Break on void _NSDetectedLayoutRecursion(void) to debug.  This will be logged only once.  This may break in the future.
Failed to acquire exclusive access to AutoFill corrections SQLite store at AutoFillCorrections.db.
Failed to acquire exclusive access to AutoFill corrections SQLite store at CloudAutoFillCorrections.db.
Failed to acquire exclusive access to AutoFill corrections SQLite store at AutoFillCorrections.db.
Failed to acquire exclusive access to AutoFill corrections SQLite store at CloudAutoFillCorrections.db.
2018-10-02 17:55:14.509 SafariForWebKitDevelopment[50142:289430] AOSKit ERROR: XPC CLIENT: Unauthorized client error received
Comment 2 Rob Buis 2018-10-02 08:57:06 PDT
Also on my system are stock Safari Version 11.1.2 (13605.3.8) and Xcode Version 9.4.1 (9F2000).
Comment 3 Alex Christensen 2018-10-02 09:32:42 PDT
Interesting.  Can this reproduce with the latest?
Comment 4 youenn fablet 2018-10-02 09:35:32 PDT
(In reply to Alex Christensen from comment #3)
> Interesting.  Can this reproduce with the latest?

It was reported in Mojave latest Safari.
Comment 5 Alex Christensen 2018-10-02 09:36:32 PDT
I mean with the latest WebKit.
Also, can it reproduce with local builds of WebKit?
Comment 6 Chris Dumez 2018-10-02 09:37:35 PDT
(In reply to Alex Christensen from comment #5)
> I mean with the latest WebKit.
> Also, can it reproduce with local builds of WebKit?

This is the bug, it happens if you use a recent locally built WebKit with the System's Safari.
Comment 7 Chris Dumez 2018-10-02 09:46:34 PDT
Does it require building WebKit with an external SDK?
Comment 8 Rob Buis 2018-10-02 09:53:53 PDT
(In reply to Alex Christensen from comment #5)
> I mean with the latest WebKit.
> Also, can it reproduce with local builds of WebKit?

Yes, I built r236716 today for my test.
Comment 9 Alex Christensen 2018-10-05 14:15:08 PDT
Created attachment 351698 [details]
Patch
Comment 10 youenn fablet 2018-10-05 14:20:10 PDT
Comment on attachment 351698 [details]
Patch

Are we missing some API tests to catch these kind of regressions?
Comment 11 Alex Christensen 2018-10-05 14:40:53 PDT
(In reply to youenn fablet from comment #10)
> Comment on attachment 351698 [details]
> Patch
> 
> Are we missing some API tests to catch these kind of regressions?
Technically yes, but we are in the process of removing all this SPI.

http://trac.webkit.org/r236886
Comment 12 Radar WebKit Bug Importer 2018-10-05 14:41:29 PDT
<rdar://problem/45055789>
Comment 13 Chris Dumez 2018-10-05 15:27:23 PDT
Comment on attachment 351698 [details]
Patch

Seems like we could have very easily written an API test for this.