Bug 220109

Summary: Remove non-inclusive language from JetStream 2.0
Product: WebKit Reporter: Beth Dakin <bdakin>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, bdakin, cdumez, ews-watchlist, ggaren, jbedard, rniwa, saam, sam, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 213092    
Attachments:
Description Flags
Patch
none
Patch
sam: review+
Patch
none
Patch
none
Patch
none
Patch none

Beth Dakin
Reported 2020-12-22 16:45:00 PST
Running 'report-non-inclusive-language --verbose' inside Websites/browserbench.org/JetStream2.0/ finds a number of non-inclusive terms that we should remove.
Attachments
Patch (1.33 MB, patch)
2020-12-22 16:49 PST, Beth Dakin 		no flags
DetailsFormatted DiffDiff
Patch (2.00 MB, patch)
2020-12-23 21:15 PST, Beth Dakin 		sam: review+
DetailsFormatted DiffDiff
Patch (1.33 MB, patch)
2021-01-02 15:00 PST, Beth Dakin 		no flags
DetailsFormatted DiffDiff
Patch (1.33 MB, patch)
2021-01-02 15:05 PST, Beth Dakin 		no flags
DetailsFormatted DiffDiff
Patch (1.33 MB, patch)
2021-01-02 15:10 PST, Beth Dakin 		no flags
DetailsFormatted DiffDiff
Patch (1.33 MB, patch)
2021-01-02 15:11 PST, Beth Dakin 		no flags
DetailsFormatted DiffDiff
Show Obsolete (5) View All Add attachment proposed patch, testcase, etc.
Beth Dakin
Comment 1 2020-12-22 16:49:04 PST
Created attachment 416698 [details] Patch Here's an attempt. I'm not sure how to tell if this breaks things other than running JetStream locally. It ran with and without this change, and the result was similar but slightly better with the change.
Beth Dakin
Comment 2 2020-12-23 21:15:30 PST
Created attachment 416738 [details] Patch Not sure why this patch is so angry, trying again.
Sam Weinig
Comment 3 2020-12-25 10:42:46 PST
Comment on attachment 416738 [details] Patch rs=me! (Assuming the bots get less angry eventually). Thanks for continuing to push this!
Radar WebKit Bug Importer
Comment 4 2020-12-29 16:45:59 PST
<rdar://problem/72730133>
Beth Dakin
Comment 5 2021-01-02 15:00:15 PST
Created attachment 416901 [details] Patch Thanks Sam! Trying again.
Beth Dakin
Comment 6 2021-01-02 15:05:59 PST
Created attachment 416902 [details] Patch
Beth Dakin
Comment 7 2021-01-02 15:10:07 PST
Created attachment 416903 [details] Patch
Beth Dakin
Comment 8 2021-01-02 15:11:48 PST
Created attachment 416904 [details] Patch
EWS
Comment 9 2021-01-03 10:43:52 PST
Committed r271126: <https://trac.webkit.org/changeset/271126> All reviewed patches have been landed. Closing bug and clearing flags on attachment 416904 [details].
Note You need to log in before you can comment on or make changes to this bug.