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

Unified Diff: webrtc/voice_engine/BUILD.gn

Issue 2247033003: Move FilePlayer and FileRecorder to Voice Engine (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: rebase 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 | « webrtc/modules/utility/utility.gypi ('k') | webrtc/voice_engine/channel.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/voice_engine/BUILD.gn
diff --git a/webrtc/voice_engine/BUILD.gn b/webrtc/voice_engine/BUILD.gn
index e330bab4cc304bf31d3a2b874db1a3d810d732be..ed34637e3896279e34136c056d1f04bcbcf1e38d 100644
--- a/webrtc/voice_engine/BUILD.gn
+++ b/webrtc/voice_engine/BUILD.gn
@@ -9,6 +9,74 @@
import("../build/webrtc.gni")
import("//testing/test.gni")
+source_set("audio_coder") {
+ sources = [
+ "coder.cc",
+ "coder.h",
+ ]
+ configs += [ "..:common_config" ]
+ public_configs = [ "..:common_inherited_config" ]
+ deps = [
+ "..:webrtc_common",
+ "../modules/audio_coding:audio_coding",
+ "../modules/audio_coding:builtin_audio_decoder_factory",
+ "../modules/audio_coding:rent_a_codec",
+ ]
+
+ if (is_clang) {
+ # Suppress warnings from Chrome's Clang plugins.
+ # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
+ configs -= [ "//build/config/clang:find_bad_constructs" ]
+ }
+}
+
+source_set("file_player") {
+ sources = [
+ "file_player.h",
+ "file_player_impl.cc",
+ "file_player_impl.h",
+ ]
+ configs += [ "..:common_config" ]
+ public_configs = [ "..:common_inherited_config" ]
+ deps = [
+ ":audio_coder",
+ "..:webrtc_common",
+ "../common_audio:common_audio",
+ "../modules/media_file:media_file",
+ "../system_wrappers:system_wrappers",
+ ]
+
+ if (is_clang) {
+ # Suppress warnings from Chrome's Clang plugins.
+ # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
+ configs -= [ "//build/config/clang:find_bad_constructs" ]
+ }
+}
+
+source_set("file_recorder") {
+ sources = [
+ "file_recorder.h",
+ "file_recorder_impl.cc",
+ "file_recorder_impl.h",
+ ]
+ configs += [ "..:common_config" ]
+ public_configs = [ "..:common_inherited_config" ]
+ deps = [
+ ":audio_coder",
+ "..:webrtc_common",
+ "../base:rtc_base_approved",
+ "../common_audio:common_audio",
+ "../modules/media_file:media_file",
+ "../system_wrappers:system_wrappers",
+ ]
+
+ if (is_clang) {
+ # Suppress warnings from Chrome's Clang plugins.
+ # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
+ configs -= [ "//build/config/clang:find_bad_constructs" ]
+ }
+}
+
source_set("voice_engine") {
sources = [
"channel.cc",
@@ -89,6 +157,8 @@ source_set("voice_engine") {
}
deps = [
+ ":file_player",
+ ":file_recorder",
":level_indicator",
"..:rtc_event_log",
"..:webrtc_common",
@@ -129,6 +199,7 @@ if (rtc_include_tests) {
":voice_engine",
"//testing/gmock",
"//testing/gtest",
+ "//third_party/gflags",
"//webrtc/common_audio",
"//webrtc/modules/audio_coding",
"//webrtc/modules/audio_conference_mixer",
@@ -144,10 +215,15 @@ if (rtc_include_tests) {
if (is_android) {
deps += [ "//testing/android/native_test:native_test_native_code" ]
shard_timeout = 900
+ data = [
+ "//resources/utility/encapsulated_pcm16b_8khz.wav",
+ "//resources/utility/encapsulated_pcmu_8khz.wav",
+ ]
}
sources = [
"channel_unittest.cc",
+ "file_player_unittests.cc",
"network_predictor_unittest.cc",
"transmit_mixer_unittest.cc",
"utility_unittest.cc",
« no previous file with comments | « webrtc/modules/utility/utility.gypi ('k') | webrtc/voice_engine/channel.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698