Move FilePlayer and FileRecorder to Voice Engine

webrtc/voice_engine/BUILD.gn

 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
 #
 # Use of this source code is governed by a BSD-style license
 # that can be found in the LICENSE file in the root of the source
 # tree. An additional intellectual property rights grant can be found
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
 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 = [
+    "../modules/audio_coding:audio_coding",
+    "../modules/audio_coding:builtin_audio_decoder_factory",
+    "../modules/audio_coding:rent_a_codec",
+    "..:webrtc_common",
+  ]
+
+  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 = [
+    "../common_audio:common_audio",
+    "../modules/media_file:media_file",
+    "../system_wrappers:system_wrappers",
+    "..:webrtc_common",
+    ":audio_coder",
+  ]
+
+  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" ]

[kjellander_webrtc, 2016/08/15 11:22:27]

It would be great if these warnings could be fixed instead of surpressed, but
maybe it's too much work to ask for in this CL?

[kwiberg-webrtc, 2016/08/15 11:26:32]

This is CL ~#3 of a series of ~7 that cleans these files up. I'll add another
one at the end of the series that does this.

+  }
+}
+
+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 = [
+    "../base:rtc_base_approved",
+    "../common_audio:common_audio",
+    "../modules/media_file:media_file",
+    "../system_wrappers:system_wrappers",
+    "..:webrtc_common",
+    ":audio_coder",
+  ]
+
+  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",
     "channel.h",
     "channel_manager.cc",
     "channel_manager.h",
     "channel_proxy.cc",
     "channel_proxy.h",
     "include/voe_audio_processing.h",
     "include/voe_base.h",
@@ skipping 60 matching lines @@
   configs += [ "..:common_config" ]
   public_configs = [ "..:common_inherited_config" ]
 
   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" ]
   }
 
   deps = [
+    ":file_player",
+    ":file_recorder",
     ":level_indicator",
     "..:rtc_event_log",
     "..:webrtc_common",
     "../base:rtc_base_approved",
     "../common_audio",
     "../modules/audio_coding",
     "../modules/audio_conference_mixer",
     "../modules/audio_device",
     "../modules/audio_processing",
     "../modules/bitrate_controller",
@@ skipping 20 matching lines @@
     "../common_audio",
   ]
 }
 
 if (rtc_include_tests) {
   test("voice_engine_unittests") {
     deps = [
       ":voice_engine",
       "//testing/gmock",
       "//testing/gtest",
+      "//third_party/gflags",
       "//webrtc/common_audio",
       "//webrtc/modules/audio_coding",
       "//webrtc/modules/audio_conference_mixer",
       "//webrtc/modules/audio_device",
       "//webrtc/modules/audio_processing",
       "//webrtc/modules/media_file",
       "//webrtc/modules/rtp_rtcp",
       "//webrtc/modules/utility",
       "//webrtc/system_wrappers",
       "//webrtc/test:test_support_main",
     ]
 
     if (is_android) {
       deps += [ "//testing/android/native_test:native_test_native_code" ]
+      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",
       "voe_audio_processing_unittest.cc",
       "voe_base_unittest.cc",
       "voe_codec_unittest.cc",
       "voe_network_unittest.cc",
       "voice_engine_fixture.cc",
       "voice_engine_fixture.h",
     ]
@@ skipping 101 matching lines @@
       }
 
       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" ]
       }
     }
   }
 }