Bug 230499 - Remove XSS Auditor: Part 2 (Remove engine support)
Summary: Remove XSS Auditor: Part 2 (Remove engine support)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Brent Fulgham
URL:
Keywords: InRadar
: 34061 35802 54110 57852 67818 112764 189551 214735 (view as bug list)
Depends on:
Blocks:
 
Reported: 2021-09-20 12:20 PDT by Brent Fulgham
Modified: 2021-09-21 16:02 PDT (History)
18 users (show)

See Also:

Attachments
Patch (58.31 KB, patch)
2021-09-20 13:45 PDT, Brent Fulgham 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (179.22 KB, patch)
2021-09-20 17:25 PDT, Brent Fulgham 		no flags Details | Formatted Diff | Diff
Patch (565.68 KB, patch)
2021-09-21 11:02 PDT, Brent Fulgham 		no flags Details | Formatted Diff | Diff
Patch (565.88 KB, patch)
2021-09-21 11:39 PDT, Brent Fulgham 		no flags Details | Formatted Diff | Diff
Show Obsolete (3) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2021-09-20 12:20:12 PDT 
As a second step in removing the XSS Auditor, let's remove the implementation of the feature in the engine, but leave the API in place so that client software doesn't see any change in interface.
Comment 1 Radar WebKit Bug Importer 2021-09-20 12:20:35 PDT 
<rdar://problem/83318883>
Comment 2 Brent Fulgham 2021-09-20 13:45:33 PDT 
Created attachment 438722 [details]
Patch
Comment 3 Brent Fulgham 2021-09-20 16:27:46 PDT 
Looks like the failures are all in XSS Auditor tests, which makes sense.
Comment 4 Brent Fulgham 2021-09-20 17:25:52 PDT 
Created attachment 438759 [details]
Patch
Comment 5 Brent Fulgham 2021-09-21 11:02:35 PDT 
Created attachment 438835 [details]
Patch
Comment 6 Brent Fulgham 2021-09-21 11:39:11 PDT 
Created attachment 438841 [details]
Patch
Comment 7 Brent Fulgham 2021-09-21 14:01:42 PDT 
*** Bug 35802 has been marked as a duplicate of this bug. ***
Comment 8 Brent Fulgham 2021-09-21 14:04:33 PDT 
*** Bug 54110 has been marked as a duplicate of this bug. ***
Comment 9 Brent Fulgham 2021-09-21 14:04:58 PDT 
*** Bug 57852 has been marked as a duplicate of this bug. ***
Comment 10 Brent Fulgham 2021-09-21 14:07:53 PDT 
*** Bug 34061 has been marked as a duplicate of this bug. ***
Comment 11 Yusuke Suzuki 2021-09-21 14:10:01 PDT 
Comment on attachment 438841 [details]
Patch

r=me
Comment 12 Brent Fulgham 2021-09-21 14:28:22 PDT 
*** Bug 112764 has been marked as a duplicate of this bug. ***
Comment 13 Brent Fulgham 2021-09-21 14:31:39 PDT 
*** Bug 189551 has been marked as a duplicate of this bug. ***
Comment 14 Brent Fulgham 2021-09-21 14:31:57 PDT 
Will land once the API tests complete.
Comment 15 Brent Fulgham 2021-09-21 14:33:07 PDT 
*** Bug 214735 has been marked as a duplicate of this bug. ***
Comment 16 Brent Fulgham 2021-09-21 14:45:15 PDT 
*** Bug 67818 has been marked as a duplicate of this bug. ***
Comment 17 EWS 2021-09-21 16:02:23 PDT 
Committed r282847 (241978@main): <https://commits.webkit.org/241978@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 438841 [details].