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

Unified Diff: .gn

Issue 2234293002: Added new mixer to |check_targets| in .gn and fixed include/depend errors. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@mixer_dir_structure_comments
Patch Set: Created 4 years, 4 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_mixer/BUILD.gn » ('j') | webrtc/modules/audio_mixer/BUILD.gn » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: .gn
diff --git a/.gn b/.gn
index e151f7cdc3099b3d98b81aefaedab742fe564971..8a173ba4955bc54394a2aa0a14b60a85d980dcd4 100644
--- a/.gn
+++ b/.gn
@@ -19,7 +19,10 @@ secondary_source = "//build/secondary/"
# their includes checked for proper dependencies when you run either
# "gn check" or "gn gen --check".
# TODO(kjellander): Keep adding paths to this list as work in webrtc:5589 is done.
-check_targets = [ "//webrtc/voice_engine:level_indicator" ]
+check_targets = [
+ "//webrtc/voice_engine:level_indicator",
+ "//webrtc/modules/audio_mixer:audio_conference_mixer",
+]
# These are the list of GN files that run exec_script. This whitelist exists
# to force additional review for new uses of exec_script, which is strongly
« no previous file with comments | « no previous file | webrtc/modules/audio_mixer/BUILD.gn » ('j') | webrtc/modules/audio_mixer/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698