Bug 232223

Summary: WTR / DRT should rely on SecurityOrigin::isLocalHostOrLoopbackIPAddress()
Product: WebKit Reporter: Alexey Shvayka <ashvayka>
Component: Tools / TestsAssignee: Alexey Shvayka <ashvayka>
Status: RESOLVED WONTFIX    
Severity: Enhancement CC: clopez, eric.carlson, ews-watchlist, ggaren, glenn, jer.noble, philipj, sergio, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
See Also: https://bugs.webkit.org/show_bug.cgi?id=236049
https://bugs.webkit.org/show_bug.cgi?id=127676
https://bugs.webkit.org/show_bug.cgi?id=160504
Attachments:
Description Flags
Patch
none
Patch
ews-feeder: commit-queue-
Patch
none
Patch ews-feeder: commit-queue-

Description Alexey Shvayka 2021-10-24 16:58:09 PDT 
WTR / DTR should rely on SecurityOrigin::isLocalHostOrLoopbackIPAddress()
Comment 1 Alexey Shvayka 2021-10-24 16:59:13 PDT 
Created attachment 442331 [details]
Patch
Comment 2 Alexey Shvayka 2021-10-24 17:05:55 PDT 
Created attachment 442332 [details]
Patch
Comment 3 EWS Watchlist 2021-10-24 17:06:56 PDT 
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Comment 4 Alexey Shvayka 2021-10-24 17:30:27 PDT 
Created attachment 442333 [details]
Patch
Comment 5 Alexey Shvayka 2021-10-24 21:55:05 PDT 
Created attachment 442337 [details]
Patch
Comment 6 Geoffrey Garen 2021-10-27 10:50:05 PDT 
Comment on attachment 442337 [details]
Patch

Seems like the right approach. EWS is still failing.
Comment 7 Radar WebKit Bug Importer 2021-10-31 16:59:16 PDT 
<rdar://problem/84862800>
Comment 8 Alexey Shvayka 2022-08-05 18:29:50 PDT 
Closing this in favour of https://github.com/WebKit/WebKit/pull/2489, and also since my solution doesn't work on EWS.