Broke out 'level_indicator' of the voice_engine GN target.

webrtc/voice_engine/BUILD.gn

Index: webrtc/voice_engine/BUILD.gn
diff --git a/webrtc/voice_engine/BUILD.gn b/webrtc/voice_engine/BUILD.gn
index b587d04073fb778585174cb4a694a45bd0440da5..0492919740e2be252e167acb1b56e8a9b3848c32 100644
--- a/webrtc/voice_engine/BUILD.gn
+++ b/webrtc/voice_engine/BUILD.gn
@@ -29,8 +29,6 @@ source_set("voice_engine") {
     "include/voe_rtp_rtcp.h",
     "include/voe_video_sync.h",
     "include/voe_volume_control.h",
-    "level_indicator.cc",
-    "level_indicator.h",
     "monitor_module.cc",
     "monitor_module.h",
     "network_predictor.cc",
@@ -106,6 +104,20 @@ source_set("voice_engine") {
     "../modules/utility",
     "../system_wrappers",
   ]
+
+  public_deps = [
+    ":level_indicator",

[kjellander_webrtc, 2016/08/09 21:25:17]

does it need to be public?

[aleloi, 2016/08/10 09:08:04]

No, it doesn't. I thought it needed to be public because it's used in
TransmitMixer, Channel and OutputMixer. All those targets are in voice_engine,
so level_indicator can stay private.

+  ]
+}
+
+source_set("level_indicator") {
+  sources = [
+    "level_indicator.cc",
+    "level_indicator.h",
+  ]
+

[kjellander_webrtc, 2016/08/09 21:25:17]

Add a dependency on "../common_audio", as well, since it's used from the .cc.
"webrtc/modules/include/module_common_types.h" is unfortunately not listed in
any GN target yet, so that has to wait.

[kjellander_webrtc, 2016/08/09 21:25:17]

Please add dependency on "../base:rtc_base_approved" since that header is
obviously referenced.
Try make it pass gn gen --check (bonus if the target can be added to
check_targets in
https://chromium.googlesource.com/external/webrtc/+/master/.gn).

[aleloi, 2016/08/10 09:08:04]

Done, and also webrtc_common from webrtc/typedefs.h. It passes the check tests
now.

+  configs += [ "..:common_config" ]
+  public_configs = [ "..:common_inherited_config" ]
 }
 
 if (rtc_include_tests) {