Added volume adjust sound behind the flag.

media/audio/sounds/audio_stream_handler.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/audio/sounds/audio_stream_handler.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "base/single_thread_task_runner.h"
+#include "base/synchronization/lock.h"
 #include "media/audio/audio_manager.h"
 #include "media/audio/audio_manager_base.h"
 #include "media/base/channel_layout.h"
 
 namespace media {
 
 namespace {
 
 // Volume percent.
 const double kOutputVolumePercent = 0.8;
 
 // The number of frames each OnMoreData() call will request.
 const int kDefaultFrameCount = 1024;
 
 AudioStreamHandler::TestObserver* g_observer_for_testing = NULL;
 AudioOutputStream::AudioSourceCallback* g_audio_source_for_testing = NULL;
 
 }  // namespace
 
 class AudioStreamHandler::AudioStreamContainer
     : public AudioOutputStream::AudioSourceCallback {
  public:
-  AudioStreamContainer(const WavAudioHandler& wav_audio,
-                       const AudioParameters& params)
+  AudioStreamContainer(const WavAudioHandler& wav_audio)
       : stream_(NULL),
         wav_audio_(wav_audio),
-        params_(params),
-        cursor_(0) {
+        cursor_(0),
+        can_start_(true) {
   }
 
   virtual ~AudioStreamContainer() {
     DCHECK(AudioManager::Get()->GetTaskRunner()->BelongsToCurrentThread());
   }
 
   void Play() {
     DCHECK(AudioManager::Get()->GetTaskRunner()->BelongsToCurrentThread());
 
     if (!stream_) {
-      stream_ = AudioManager::Get()->MakeAudioOutputStreamProxy(params_,
-                                                                std::string(),
-                                                                std::string());
+      const AudioParameters& p = wav_audio_.params();
+      const AudioParameters params(AudioParameters::AUDIO_PCM_LOW_LATENCY,
+                                   p.channel_layout(),
+                                   p.sample_rate(),
+                                   p.bits_per_sample(),
+                                   kDefaultFrameCount);
+      stream_ = AudioManager::Get()->MakeAudioOutputStreamProxy(
+          params, std::string(), std::string());
       if (!stream_ || !stream_->Open()) {
         LOG(ERROR) << "Failed to open an output stream.";
         return;
       }
       stream_->SetVolume(kOutputVolumePercent);
-    } else {
-      // TODO (ygorshenin@): implement smart stream rewind.
-      stream_->Stop();
+    } else if (!CanStart()) {

[DaleCurtis, 2014/01/09 19:58:26]

You can simplify this by tracking "started_" instead of "can_start_" and using a
base::CancelableClosure().  Doing so allows you to remove the extra lock and
avoid the explosion of Get/Set/Can methods.  Replace this else with:

<acquire lock_>
stop_callback_.Reset(base::Bind(
    &AudioStreamContainer::StopStream, base::Unretained(this)));
if (started_ && wav_audio_.AtEnd(cursor_)) {
  cursor_ = 0;
  return;
}
started_ = true;

Down below inside OnMoreData() change the PostTask() to a PostDelayedTask() with
a timeout of whatever you want for your keep alive duration (XXX):

<acquire lock_>
AudioManager::Get()->GetTaskRunner()->PostDelayedTask(
    FROM_HERE, stop_callback_, base::TimeDelta::FromSeconds(XXX));

Inside StopStream() just add a "started_ = false";

[ygorshenin1, 2014/01/13 10:00:55]

Done.

[ygorshenin1, 2014/01/13 10:00:55]

Done.

+      return;
     }
 
-    cursor_ = 0;
+    SetCursor(0);
+    SetCanStart(false);
     if (g_audio_source_for_testing)
       stream_->Start(g_audio_source_for_testing);
     else
       stream_->Start(this);
 
     if (g_observer_for_testing)
       g_observer_for_testing->OnPlay();
   }
 
   void Stop() {
     DCHECK(AudioManager::Get()->GetTaskRunner()->BelongsToCurrentThread());
-    if (!stream_)
-      return;
-    stream_->Stop();
-    stream_->Close();
+    StopStream();
+    if (stream_)
+      stream_->Close();
     stream_ = NULL;
-
-    if (g_observer_for_testing)
-      g_observer_for_testing->OnStop(cursor_);
   }
 
  private:
   // AudioOutputStream::AudioSourceCallback overrides:
   // Following methods could be called from *ANY* thread.
   virtual int OnMoreData(AudioBus* dest,
                          AudioBuffersState /* state */) OVERRIDE {
     size_t bytes_written = 0;
-    if (wav_audio_.AtEnd(cursor_) ||
-        !wav_audio_.CopyTo(dest, cursor_, &bytes_written)) {
+    size_t cursor = GetCursor();
+    if (wav_audio_.AtEnd(cursor) ||
+        !wav_audio_.CopyTo(dest, cursor, &bytes_written)) {
       AudioManager::Get()->GetTaskRunner()->PostTask(
           FROM_HERE,
-          base::Bind(&AudioStreamContainer::Stop, base::Unretained(this)));
+          base::Bind(&AudioStreamContainer::StopStream,
+                     base::Unretained(this)));
       return 0;
     }
-    cursor_ += bytes_written;
+    SetCursor(cursor + bytes_written);
 
     return dest->frames();
   }
 
   virtual int OnMoreIOData(AudioBus* /* source */,
                            AudioBus* dest,
                            AudioBuffersState state) OVERRIDE {
     return OnMoreData(dest, state);
   }
 
   virtual void OnError(AudioOutputStream* /* stream */) OVERRIDE {
     LOG(ERROR) << "Error during system sound reproduction.";
   }
 
+  void StopStream() {
+    if (stream_)
+      stream_->Stop();
+    if (g_observer_for_testing)
+      g_observer_for_testing->OnStop(GetCursor());
+    SetCanStart(true);
+  }
+
+  size_t GetCursor() {
+    base::AutoLock al(cursor_lock_);
+    return cursor_;
+  }
+
+  void SetCursor(size_t cursor) {
+    base::AutoLock al(cursor_lock_);
+    cursor_ = cursor;
+  }
+
+  bool CanStart() {
+    base::AutoLock al(can_start_lock_);
+    return can_start_;
+  }
+
+  void SetCanStart(bool can_start) {
+    base::AutoLock al(can_start_lock_);
+    can_start_ = can_start;
+  }
+
   AudioOutputStream* stream_;
 
   const WavAudioHandler wav_audio_;
-  const AudioParameters params_;
 
+  base::Lock cursor_lock_;
   size_t cursor_;
 
+  base::Lock can_start_lock_;
+  bool can_start_;
+
   DISALLOW_COPY_AND_ASSIGN(AudioStreamContainer);
 };
 
 AudioStreamHandler::AudioStreamHandler(const base::StringPiece& wav_data)
     : wav_audio_(wav_data),
       initialized_(false) {
   AudioManager* manager = AudioManager::Get();
   if (!manager) {
     LOG(ERROR) << "Can't get access to audio manager.";
     return;
   }
-  AudioParameters params(AudioParameters::AUDIO_PCM_LOW_LATENCY,
-                         GuessChannelLayout(wav_audio_.num_channels()),
-                         wav_audio_.sample_rate(),
-                         wav_audio_.bits_per_sample(),
-                         kDefaultFrameCount);
-  if (!params.IsValid()) {
+  if (!wav_audio_.params().IsValid()) {
     LOG(ERROR) << "Audio params are invalid.";
     return;
   }
-  stream_.reset(new AudioStreamContainer(wav_audio_, params));
+  stream_.reset(new AudioStreamContainer(wav_audio_));
   initialized_ = true;
 }
 
 AudioStreamHandler::~AudioStreamHandler() {
   DCHECK(CalledOnValidThread());
   AudioManager::Get()->GetTaskRunner()->PostTask(
       FROM_HERE,
       base::Bind(&AudioStreamContainer::Stop, base::Unretained(stream_.get())));
   AudioManager::Get()->GetTaskRunner()->DeleteSoon(FROM_HERE,
                                                     stream_.release());
@@ skipping 29 matching lines @@
   g_observer_for_testing = observer;
 }
 
 // static
 void AudioStreamHandler::SetAudioSourceForTesting(
     AudioOutputStream::AudioSourceCallback* source) {
   g_audio_source_for_testing = source;
 }
 
 }  // namespace media