Unable to run system Safari with trunk WebKit

[Chris Dumez]

Unable to run system Safari with trunk WebKit:
SafariForWebKitDevelopment[644:9602] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Related web view <WKWebView: 0x7fbc8752ce00> has data store <WKWebsiteDataStore: 0x7fbc872856e0> but configuration specifies a different data store <WKWebsiteDataStore: 0x7fbc87562590>'

[Radar WebKit Bug Importer]

<rdar://problem/49784574>

[Chris Dumez]

Created attachment 367147 [details]
Patch

[Alex Christensen]

Comment on attachment 367147 [details]
Patch

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

> Source/WebKit/UIProcess/Cocoa/VersionChecks.h:46
> +#define DYLD_MACOS_VERSION_FIRST_WITH_EXCEPTIONS_FOR_RELATED_WEBVIEWS_USING_DIFFERENT_DATA_STORES 0

I don't think this will fix our open source friends' problem.  We need to define the last version and use DYLD_MACOSX_VERSION_10_14

> Source/WebKit/UIProcess/Cocoa/VersionChecks.h:86
> +enum class AssumeSafariIsAlwaysLinkedOnAfter { No, Yes };

: bool

[Chris Dumez]

(In reply to Alex Christensen from comment #3)
> Comment on attachment 367147 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=367147&action=review
> 
> > Source/WebKit/UIProcess/Cocoa/VersionChecks.h:46
> > +#define DYLD_MACOS_VERSION_FIRST_WITH_EXCEPTIONS_FOR_RELATED_WEBVIEWS_USING_DIFFERENT_DATA_STORES 0
> 
> I don't think this will fix our open source friends' problem.  We need to
> define the last version and use DYLD_MACOSX_VERSION_10_14

I believe it works because of the change I made to linkedOnAfter which special-cases 0 as sdkVersion.

> 
> > Source/WebKit/UIProcess/Cocoa/VersionChecks.h:86
> > +enum class AssumeSafariIsAlwaysLinkedOnAfter { No, Yes };
> 
> : bool

[Chris Dumez]

(In reply to Chris Dumez from comment #4)
> (In reply to Alex Christensen from comment #3)
> > Comment on attachment 367147 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=367147&action=review
> > 
> > > Source/WebKit/UIProcess/Cocoa/VersionChecks.h:46
> > > +#define DYLD_MACOS_VERSION_FIRST_WITH_EXCEPTIONS_FOR_RELATED_WEBVIEWS_USING_DIFFERENT_DATA_STORES 0
> > 
> > I don't think this will fix our open source friends' problem.  We need to
> > define the last version and use DYLD_MACOSX_VERSION_10_14
> 
> I believe it works because of the change I made to linkedOnAfter which
> special-cases 0 as sdkVersion.

As it is working for me without internal changes.

[Chris Dumez]

Created attachment 367149 [details]
Patch

[Chris Dumez]

Comment on attachment 367149 [details]
Patch

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

> Source/WebKit/ChangeLog:20
> +        - In the header, DYLD_IOS_VERSION_* / DYLD_MACOS_VERSION_* constants are 0 when building with the

I explained this in the Changelog here.

[WebKit Commit Bot]

Comment on attachment 367149 [details]
Patch

Clearing flags on attachment: 367149

Committed r244146: <https://trac.webkit.org/changeset/244146>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Chris Dumez]

Reverted r244146 for reason:

Caused a few DeviceOrientation API test failures on iOS

Committed r244170: <https://trac.webkit.org/changeset/244170>

[Chris Dumez]

Created attachment 367196 [details]
Patch

[Chris Dumez]

Created attachment 367199 [details]
Patch

[Chris Dumez]

Created attachment 367202 [details]
Patch

[EWS Watchlist]

Comment on attachment 367202 [details]
Patch

Attachment 367202 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/11838498

New failing tests:
fast/device-orientation/device-orientation-request-permission-granted.html
fast/device-orientation/device-motion-request-permission-denied.html
fast/device-orientation/device-motion-request-permission-granted.html
fast/device-orientation/device-orientation-request-permission-user-gesture.html
http/tests/events/device-orientation-motion-secure-context.html
fast/device-orientation/device-orientation-request-permission-denied.html
fast/device-orientation/device-motion-request-permission-user-gesture.html

[EWS Watchlist]

Created attachment 367208 [details]
Archive of layout-test-results from ews122 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews122  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6

[Chris Dumez]

Created attachment 367246 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 367246 [details]
Patch

Clearing flags on attachment: 367246

Committed r244203: <https://trac.webkit.org/changeset/244203>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.