Moved the GainControlForNewAGC class to a separate file.

webrtc/modules/audio_processing/gain_control_for_experimental_agc.cc

+/*
+ *  Copyright (c) 2016 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.
+ */
+
+#include "webrtc/modules/audio_processing/gain_control_for_experimental_agc.h"
+
+#include "webrtc/base/checks.h"
+#include "webrtc/modules/audio_processing/include/audio_processing.h"
+
+namespace webrtc {
+
+GainControlForExperimentalAgc::GainControlForExperimentalAgc(
+    GainControl* gain_control)
+    : real_gain_control_(gain_control), volume_(0) {
+  thread_checker.DetachFromThread();

[the sun, 2016/02/09 11:49:55]

Really needed?

[peah-webrtc, 2016/02/10 09:33:37]

With the current locking test, it will require quite some work to remove it, and
I have no idea whether it will work on the trybots. So I'd rather keep it for
now. It does not hurt the module if it is created from another thread.

+}
+
+int GainControlForExperimentalAgc::Enable(bool enable) {
+  return real_gain_control_->Enable(enable);
+}
+
+bool GainControlForExperimentalAgc::is_enabled() const {
+  return real_gain_control_->is_enabled();
+}
+
+int GainControlForExperimentalAgc::set_stream_analog_level(int level) {
+  RTC_CHECK(thread_checker.CalledOnValidThread());
+  volume_ = level;
+  return AudioProcessing::kNoError;
+}
+
+int GainControlForExperimentalAgc::stream_analog_level() {
+  RTC_CHECK(thread_checker.CalledOnValidThread());

[the sun, 2016/02/09 11:49:55]

Thank you!

+  return volume_;
+}
+
+int GainControlForExperimentalAgc::set_mode(Mode mode) {
+  return AudioProcessing::kNoError;

[the sun, 2016/02/09 11:49:55]

(Not possible to add TCs to the other methods?)

[peah-webrtc, 2016/02/10 09:33:37]

I tried and it is a bit tricky as it requires the locking test to be rewritten.
I'd rather keep it as it is for now, as the added thread checkers now protect
the volume_ variable. The rest of the methods access the methods of the
gain_control object which resources are protected by its own lock.

+}
+
+GainControl::Mode GainControlForExperimentalAgc::mode() const {
+  return GainControl::kAdaptiveAnalog;
+}
+
+int GainControlForExperimentalAgc::set_target_level_dbfs(int level) {
+  return AudioProcessing::kNoError;
+}
+
+int GainControlForExperimentalAgc::target_level_dbfs() const {
+  return real_gain_control_->target_level_dbfs();
+}
+
+int GainControlForExperimentalAgc::set_compression_gain_db(int gain) {
+  return AudioProcessing::kNoError;
+}
+
+int GainControlForExperimentalAgc::compression_gain_db() const {
+  return real_gain_control_->compression_gain_db();
+}
+
+int GainControlForExperimentalAgc::enable_limiter(bool enable) {
+  return AudioProcessing::kNoError;
+}
+
+bool GainControlForExperimentalAgc::is_limiter_enabled() const {
+  return real_gain_control_->is_limiter_enabled();
+}
+
+int GainControlForExperimentalAgc::set_analog_level_limits(int minimum,
+                                                           int maximum) {
+  return AudioProcessing::kNoError;
+}
+
+int GainControlForExperimentalAgc::analog_level_minimum() const {
+  return real_gain_control_->analog_level_minimum();
+}
+
+int GainControlForExperimentalAgc::analog_level_maximum() const {
+  return real_gain_control_->analog_level_maximum();
+}
+
+bool GainControlForExperimentalAgc::stream_is_saturated() const {
+  return real_gain_control_->stream_is_saturated();
+}
+
+void GainControlForExperimentalAgc::SetMicVolume(int volume) {
+  RTC_CHECK(thread_checker.CalledOnValidThread());
+  volume_ = volume;
+}
+
+int GainControlForExperimentalAgc::GetMicVolume() {
+  RTC_CHECK(thread_checker.CalledOnValidThread());
+  return volume_;
+}
+
+}  // namespace webrtc