Bug 201546

Summary: [WKTR] Drop TestRunner.setPrivateBrowsingEnabled_DEPRECATED()
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, beidson, commit-queue, ggaren, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=201730
Bug Depends on: 201723, 201475, 201547, 201574, 201590, 201596, 201648, 201691    
Bug Blocks:    
Attachments:
Description Flags
WIP Patch
none
WIP Patch
none
WIP Patch
none
Patch
none
Patch
none
Patch none

Description Chris Dumez 2019-09-06 09:21:43 PDT
Drop TestRunner.setPrivateBrowsingEnabled_DEPRECATED() in WebKitTestRunner since it does not do the right thing for WK2 and we have a better alternative.
Comment 1 Chris Dumez 2019-09-06 09:22:12 PDT
Created attachment 378198 [details]
WIP Patch
Comment 2 Chris Dumez 2019-09-06 18:55:05 PDT
Created attachment 378263 [details]
WIP Patch
Comment 3 Chris Dumez 2019-09-06 20:54:46 PDT
Failures:
http/tests/cache-storage/cache-persistency.https.html
http/tests/cookies/private-cookie-storage.html
http/tests/workers/service/serviceworker-private-browsing.https.html // Bug 201574.
imported/w3c/web-platform-tests/IndexedDB/fire-error-event-exception.html
http/tests/adClickAttribution/conversion-disabled-in-ephemeral-session.html
http/tests/workers/service/basic-register.html
storage/domstorage/localstorage/private-browsing-affects-storage.html // Bug 201547.
Comment 4 Chris Dumez 2019-09-09 20:30:50 PDT
Created attachment 378440 [details]
WIP Patch
Comment 5 Chris Dumez 2019-09-10 08:32:56 PDT
Remaining failures:
http/tests/cache-storage/cache-persistency.https.html
http/tests/cookies/private-cookie-storage.html
Comment 6 Chris Dumez 2019-09-11 15:52:06 PDT
Created attachment 378587 [details]
Patch
Comment 7 Chris Dumez 2019-09-11 15:52:53 PDT
Created attachment 378588 [details]
Patch
Comment 8 Chris Dumez 2019-09-12 08:55:36 PDT
Comment on attachment 378588 [details]
Patch

Looks like I need to update a couple more tests to make iOS-wk2 happy.
Comment 9 Chris Dumez 2019-09-12 09:27:29 PDT
*** Bug 201722 has been marked as a duplicate of this bug. ***
Comment 10 Chris Dumez 2019-09-12 09:30:24 PDT
Created attachment 378647 [details]
Patch
Comment 11 WebKit Commit Bot 2019-09-12 15:12:32 PDT
Comment on attachment 378647 [details]
Patch

Clearing flags on attachment: 378647

Committed r249817: <https://trac.webkit.org/changeset/249817>
Comment 12 WebKit Commit Bot 2019-09-12 15:12:34 PDT
All reviewed patches have been landed.  Closing bug.
Comment 13 Radar WebKit Bug Importer 2019-09-12 15:13:22 PDT
<rdar://problem/55319339>