Bug 220101 - Remove non-inclusive language from libwebrtc
Summary: Remove non-inclusive language from libwebrtc
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebRTC (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords: InRadar
Depends on:
Blocks: 213092
  Show dependency treegraph
 
Reported: 2020-12-22 13:22 PST by Beth Dakin
Modified: 2021-01-07 00:10 PST (History)
12 users (show)

See Also:


Attachments
Patch (2.52 MB, patch)
2021-01-05 03:20 PST, youenn fablet
no flags Details | Formatted Diff | Diff
Patch (4.80 KB, patch)
2021-01-06 02:27 PST, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Beth Dakin 2020-12-22 13:22:36 PST
See results of 'report-non-inclusive-language --verbose' inside Source/ThirdParty/libwebrtc/Source/webrtc below. I took a look at https://webrtc.googlesource.com/src, and it's only marginally improved, with 84 occurrences of non-inclusive terms down from 93, so updating to a newer libwebrtc will only get us so far. We should try to make changes upstream as well.

.gn: whitelist: 2
tools_webrtc/vim/webrtc.ycm_extra_conf.py: whitelist: 1
tools_webrtc/msan/suppressions.txt: blacklist: 2
tools_webrtc/ubsan/vptr_suppressions.txt: blacklist: 5
tools_webrtc/ubsan/suppressions.txt: blacklist: 2
rtc_base/message_digest.h: whitelist: 1
rtc_base/win/windows_version.h: blacklist: 1
build_overrides/build.gni: blacklist: 3
sdk/android/api/org/webrtc/MediaCodecVideoEncoder.java: blacklist: 1
sdk/android/api/org/webrtc/HardwareVideoDecoderFactory.java: blacklist: 2
sdk/android/api/org/webrtc/PlatformSoftwareVideoDecoderFactory.java: whitelist: 2
sdk/android/src/java/org/webrtc/audio/WebRtcAudioEffects.java: blacklist: 2
sdk/android/src/jni/audio_device/audio_device_module.cc: blacklist: 2
modules/audio_device/android/audio_manager.h: blacklist: 3
modules/audio_device/android/audio_manager.cc: blacklist: 7
modules/audio_device/android/audio_device_template.h: blacklist: 3
modules/audio_device/android/java/src/org/webrtc/voiceengine/WebRtcAudioEffects.java: blacklist: 16
modules/audio_device/android/java/src/org/webrtc/voiceengine/WebRtcAudioUtils.java: blacklist: 16
modules/audio_device/android/java/src/org/webrtc/voiceengine/WebRtcAudioManager.java: blacklist: 18
logging/rtc_event_log/encoder/rtc_event_log_encoder_new_format.cc: whitelist: 4
Total: 93 (blacklist: 83, whitelist: 10, slave: 0)
Comment 1 Radar WebKit Bug Importer 2020-12-29 13:23:11 PST
<rdar://problem/72727968>
Comment 2 youenn fablet 2021-01-04 07:57:03 PST
We can remove the sdk/android and modules/audio_device/android folders
Comment 3 Beth Dakin 2021-01-04 13:54:46 PST
(In reply to youenn fablet from comment #2)
> We can remove the sdk/android and modules/audio_device/android folders

Oh, that's a good idea!
Comment 4 youenn fablet 2021-01-05 03:19:49 PST
Related upstream issue is https://bugs.chromium.org/p/webrtc/issues/detail?id=11680
Comment 5 youenn fablet 2021-01-05 03:20:06 PST
Created attachment 416986 [details]
Patch
Comment 6 youenn fablet 2021-01-05 04:56:03 PST
Patch removes corresponding files except for rtc_base/message_digest.h.
I added a comment about this file in https://bugs.chromium.org/p/webrtc/issues/detail?id=11680
Comment 7 Beth Dakin 2021-01-05 09:35:58 PST
Comment on attachment 416986 [details]
Patch

All red patch, how exciting! Thanks Youenn!
Comment 8 youenn fablet 2021-01-05 09:44:53 PST
Comment on attachment 416986 [details]
Patch

Last remaining instance is being fixed upstream. I'll cherry pick the change when it lands upstream
Comment 9 EWS 2021-01-05 09:55:54 PST
Committed r271150: <https://trac.webkit.org/changeset/271150>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 416986 [details].
Comment 10 Simon Fraser (smfr) 2021-01-05 10:15:21 PST
The patch that landed had some additional new logging in it.
Comment 11 youenn fablet 2021-01-05 10:31:50 PST
(In reply to Simon Fraser (smfr) from comment #10)
> The patch that landed had some additional new logging in it.

Thanks, I will revert them.
Comment 12 youenn fablet 2021-01-06 02:27:51 PST
Reopening to attach new patch.
Comment 13 youenn fablet 2021-01-06 02:27:54 PST
Created attachment 417076 [details]
Patch
Comment 14 EWS 2021-01-07 00:10:51 PST
Committed r271227: <https://trac.webkit.org/changeset/271227>

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