Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(206)

Unified Diff: webrtc/modules/audio_device/BUILD.gn

Issue 2790883002: remove module-wide warning suppression. (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | webrtc/modules/audio_device/ios/objc/RTCAudioSession.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/audio_device/BUILD.gn
diff --git a/webrtc/modules/audio_device/BUILD.gn b/webrtc/modules/audio_device/BUILD.gn
index 9c776d38913f0610e64cbeb39ce0901137930b39..f9b7f8b6238c9140552c858cb687fd945154fae9 100644
--- a/webrtc/modules/audio_device/BUILD.gn
+++ b/webrtc/modules/audio_device/BUILD.gn
@@ -19,14 +19,6 @@ config("audio_device_config") {
"include",
"dummy", # Contains dummy audio device implementations.
]
- if (is_ios) {
- # GN orders flags on a target before flags from configs. In order to be able
- # suppress the -Wthread-safety-analysis warning, it has come from a config
- # and can't be on the target directly.
- # TODO(tkchin): Remove after fixing
- # https://bugs.chromium.org/p/webrtc/issues/detail?id=5748
- cflags = [ "-Wno-thread-safety-analysis" ]
- }
}
config("audio_device_warnings_config") {
« no previous file with comments | « no previous file | webrtc/modules/audio_device/ios/objc/RTCAudioSession.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698