Revert of Cleanup of the AudioDeviceBuffer class

webrtc/modules/audio_device/audio_device_buffer.h

 /*
  *  Copyright (c) 2012 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.
  */
 
 #ifndef WEBRTC_MODULES_AUDIO_DEVICE_AUDIO_DEVICE_BUFFER_H_
 #define WEBRTC_MODULES_AUDIO_DEVICE_AUDIO_DEVICE_BUFFER_H_
 
 #include "webrtc/base/criticalsection.h"
 #include "webrtc/base/task_queue.h"
 #include "webrtc/base/thread_checker.h"
 #include "webrtc/modules/audio_device/include/audio_device.h"
 #include "webrtc/system_wrappers/include/file_wrapper.h"
 #include "webrtc/typedefs.h"
 
 namespace webrtc {
 class CriticalSectionWrapper;
 
+const uint32_t kPulsePeriodMs = 1000;
+const size_t kMaxBufferSizeBytes = 3840;  // 10ms in stereo @ 96kHz
 // Delta times between two successive playout callbacks are limited to this
 // value before added to an internal array.
 const size_t kMaxDeltaTimeInMs = 500;
 
 class AudioDeviceObserver;
 
 class AudioDeviceBuffer {
  public:
   AudioDeviceBuffer();
   virtual ~AudioDeviceBuffer();
 
   void SetId(uint32_t id) {};
-  int32_t RegisterAudioCallback(AudioTransport* audio_callback);
+  int32_t RegisterAudioCallback(AudioTransport* audioCallback);
 
   int32_t InitPlayout();
   int32_t InitRecording();
 
-  int32_t SetRecordingSampleRate(uint32_t fsHz);
-  int32_t SetPlayoutSampleRate(uint32_t fsHz);
+  virtual int32_t SetRecordingSampleRate(uint32_t fsHz);
+  virtual int32_t SetPlayoutSampleRate(uint32_t fsHz);
   int32_t RecordingSampleRate() const;
   int32_t PlayoutSampleRate() const;
 
-  int32_t SetRecordingChannels(size_t channels);
-  int32_t SetPlayoutChannels(size_t channels);
+  virtual int32_t SetRecordingChannels(size_t channels);
+  virtual int32_t SetPlayoutChannels(size_t channels);
   size_t RecordingChannels() const;
   size_t PlayoutChannels() const;
   int32_t SetRecordingChannel(const AudioDeviceModule::ChannelType channel);
   int32_t RecordingChannel(AudioDeviceModule::ChannelType& channel) const;
 
-  virtual int32_t SetRecordedBuffer(const void* audio_buffer,
-                                    size_t num_samples);
+  virtual int32_t SetRecordedBuffer(const void* audioBuffer, size_t nSamples);
   int32_t SetCurrentMicLevel(uint32_t level);
-  virtual void SetVQEData(int play_delay_ms, int rec_delay_ms, int clock_drift);
+  virtual void SetVQEData(int playDelayMS, int recDelayMS, int clockDrift);
   virtual int32_t DeliverRecordedData();
   uint32_t NewMicLevel() const;
 
-  virtual int32_t RequestPlayoutData(size_t num_samples);
-  virtual int32_t GetPlayoutData(void* audio_buffer);
+  virtual int32_t RequestPlayoutData(size_t nSamples);
+  virtual int32_t GetPlayoutData(void* audioBuffer);
 
-  // TODO(henrika): these methods should not be used and does not contain any
-  // valid implementation. Investigate the possibility to either remove them
-  // or add a proper implementation if needed.
   int32_t StartInputFileRecording(const char fileName[kAdmMaxFileNameSize]);
   int32_t StopInputFileRecording();
   int32_t StartOutputFileRecording(const char fileName[kAdmMaxFileNameSize]);
   int32_t StopOutputFileRecording();
 
-  int32_t SetTypingStatus(bool typing_status);
+  int32_t SetTypingStatus(bool typingStatus);
 
  private:
-  void AllocatePlayoutBufferIfNeeded();
-  void AllocateRecordingBufferIfNeeded();
-
   // Posts the first delayed task in the task queue and starts the periodic
   // timer.
   void StartTimer();
 
   // Called periodically on the internal thread created by the TaskQueue.
   void LogStats();
 
   // Updates counters in each play/record callback but does it on the task
   // queue to ensure that they can be read by LogStats() without any locks since
   // each task is serialized by the task queue.
   void UpdateRecStats(size_t num_samples);
   void UpdatePlayStats(size_t num_samples);
 
   // Ensures that methods are called on the same thread as the thread that
   // creates this object.
   rtc::ThreadChecker thread_checker_;
 
-  // Raw pointer to AudioTransport instance. Supplied to RegisterAudioCallback()
-  // and it must outlive this object.
-  AudioTransport* audio_transport_cb_;
-
-  // TODO(henrika): given usage of thread checker, it should be possible to
-  // remove all locks in this class.
   rtc::CriticalSection _critSect;
   rtc::CriticalSection _critSectCb;
 
+  AudioTransport* _ptrCbAudioTransport;
+
   // Task queue used to invoke LogStats() periodically. Tasks are executed on a
   // worker thread but it does not necessarily have to be the same thread for
   // each task.
   rtc::TaskQueue task_queue_;
 
   // Ensures that the timer is only started once.
   bool timer_has_started_;
 
-  // Sample rate in Hertz.
-  uint32_t rec_sample_rate_;
-  uint32_t play_sample_rate_;
+  uint32_t _recSampleRate;
+  uint32_t _playSampleRate;
 
-  // Number of audio channels.
-  size_t rec_channels_;
-  size_t play_channels_;
+  size_t _recChannels;
+  size_t _playChannels;
 
   // selected recording channel (left/right/both)
-  AudioDeviceModule::ChannelType rec_channel_;
+  AudioDeviceModule::ChannelType _recChannel;
 
-  // Number of bytes per audio sample (2 or 4).
-  size_t rec_bytes_per_sample_;
-  size_t play_bytes_per_sample_;
+  // 2 or 4 depending on mono or stereo
+  size_t _recBytesPerSample;
+  size_t _playBytesPerSample;
 
-  // Number of audio samples/bytes per 10ms.
-  size_t rec_samples_per_10ms_;
-  size_t rec_bytes_per_10ms_;
-  size_t play_samples_per_10ms_;
-  size_t play_bytes_per_10ms_;
+  // 10ms in stereo @ 96kHz
+  int8_t _recBuffer[kMaxBufferSizeBytes];
 
-  // Buffer used for recorded audio samples. Size is given by
-  // |rec_bytes_per_10ms_| and the buffer is allocated in InitRecording() on the
-  // main/creating thread.
-  std::unique_ptr<int8_t[]> rec_buffer_;
+  // one sample <=> 2 or 4 bytes
+  size_t _recSamples;
+  size_t _recSize;  // in bytes
 
-  // Buffer used for audio samples to be played out. Size is given by
-  // |play_bytes_per_10ms_| and the buffer is allocated in InitPlayout() on the
-  // main/creating thread.
-  std::unique_ptr<int8_t[]> play_buffer_;
+  // 10ms in stereo @ 96kHz
+  int8_t _playBuffer[kMaxBufferSizeBytes];
 
-  // AGC parameters.
-  uint32_t current_mic_level_;
-  uint32_t new_mic_level_;
+  // one sample <=> 2 or 4 bytes
+  size_t _playSamples;
+  size_t _playSize;  // in bytes
 
-  // Contains true of a key-press has been detected.
-  bool typing_status_;
+  FileWrapper& _recFile;
+  FileWrapper& _playFile;
 
-  // Delay values used by the AEC.
-  int play_delay_ms_;
-  int rec_delay_ms_;
+  uint32_t _currentMicLevel;
+  uint32_t _newMicLevel;
 
-  // Contains a clock-drift measurement.
-  int clock_drift_;
+  bool _typingStatus;
+
+  int _playDelayMS;
+  int _recDelayMS;
+  int _clockDrift;
+  int high_delay_counter_;
 
   // Counts number of times LogStats() has been called.
   size_t num_stat_reports_;
 
   // Total number of recording callbacks where the source provides 10ms audio
   // data each time.
   uint64_t rec_callbacks_;
 
   // Total number of recording callbacks stored at the last timer task.
   uint64_t last_rec_callbacks_;
@@ skipping 27 matching lines @@
   // milliseconds) between two successive playout callbacks, and the stored
   // value is the number of times a given time difference was found.
   // Writing to the array is done without a lock since it is only read once at
   // destruction when no audio is running.
   uint32_t playout_diff_times_[kMaxDeltaTimeInMs + 1] = {0};
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_AUDIO_DEVICE_AUDIO_DEVICE_BUFFER_H_