Bug 178418

Summary: Expose _boundInterfaceIdentifier and _allowsCellularAccess to WKWebsiteDataStorePrivate
Product: WebKit Reporter: Alex Christensen <achristensen>
Component: New BugsAssignee: Alex Christensen <achristensen>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, cdumez, cgarcia, commit-queue, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Archive of layout-test-results from ews126 for ios-simulator-wk2
none
Patch none

Description Alex Christensen 2017-10-17 17:01:00 PDT
Expose _boundInterfaceIdentifier and _allowsCellularAccess to WKWebsiteDataStorePrivate
Comment 1 Alex Christensen 2017-10-17 18:32:11 PDT
Created attachment 324077 [details]
Patch
Comment 2 Alex Christensen 2017-10-17 18:43:03 PDT
Created attachment 324079 [details]
Patch
Comment 3 Alex Christensen 2017-10-17 19:01:36 PDT
Created attachment 324083 [details]
Patch
Comment 4 Alex Christensen 2017-10-18 10:25:18 PDT
Created attachment 324130 [details]
Patch
Comment 5 Alex Christensen 2017-10-18 10:38:07 PDT
Created attachment 324132 [details]
Patch
Comment 6 Alex Christensen 2017-10-18 11:21:43 PDT
Created attachment 324139 [details]
Patch
Comment 7 Tim Horton 2017-10-18 11:46:39 PDT
Comment on attachment 324139 [details]
Patch

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

> Source/WebKit/UIProcess/API/Cocoa/_WKProcessPoolConfiguration.mm:203
> -    return _processPoolConfiguration->allowsCellularAccess();
> +    return YES;

The fact that there are no changes to the header to remove/deprecate/warn that these are lies is concerning.
Comment 8 Alex Christensen 2017-10-18 12:13:15 PDT
Created attachment 324145 [details]
Patch
Comment 9 Alex Christensen 2017-10-18 12:20:09 PDT
Created attachment 324147 [details]
Patch
Comment 10 Alex Christensen 2017-10-18 12:35:38 PDT
Created attachment 324149 [details]
Patch
Comment 11 Build Bot 2017-10-18 14:23:17 PDT
Comment on attachment 324149 [details]
Patch

Attachment 324149 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/4907596

New failing tests:
http/tests/resourceLoadStatistics/add-partitioning-to-redirect.html
http/tests/websocket/tests/hybi/secure-cookie-secure-connection.pl
http/tests/websocket/tests/hybi/secure-cookie-insecure-connection.pl
http/tests/cookies/third-party-cookie-relaxing.html
imported/w3c/web-platform-tests/html/semantics/scripting-1/the-script-element/script-crossorigin-network.html
http/tests/cookies/http-get-cookie-set-in-js.html
http/tests/websocket/tests/hybi/httponly-cookie.pl
http/tests/security/contentSecurityPolicy/report-and-enforce.php
http/tests/security/svg-image-leak.html
http/tests/cookies/private-cookie-storage.html
http/tests/cookies/sync-xhr-set-cookie-invalidates-cache.html
imported/w3c/web-platform-tests/fetch/api/cors/cors-cookies.any.html
imported/w3c/web-platform-tests/XMLHttpRequest/anonymous-mode-unsupported.htm
http/tests/resourceLoadStatistics/remove-partitioning-from-redirect.html
imported/w3c/web-platform-tests/fetch/api/cors/cors-cookies.any.worker.html
Comment 12 Build Bot 2017-10-18 14:23:20 PDT
Created attachment 324166 [details]
Archive of layout-test-results from ews126 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews126  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.6
Comment 13 Alex Christensen 2017-10-20 13:00:42 PDT
Created attachment 324434 [details]
Patch
Comment 14 WebKit Commit Bot 2017-10-20 13:39:37 PDT
Comment on attachment 324434 [details]
Patch

Clearing flags on attachment: 324434

Committed r223791: <https://trac.webkit.org/changeset/223791>
Comment 15 WebKit Commit Bot 2017-10-20 13:39:39 PDT
All reviewed patches have been landed.  Closing bug.
Comment 16 Alex Christensen 2017-10-20 18:11:33 PDT
http://trac.webkit.org/r223803
Comment 17 Radar WebKit Bug Importer 2017-11-15 13:10:42 PST
<rdar://problem/35568914>