Added volume adjust sound behind the flag.

media/audio/sounds/wav_audio_handler.h

 // 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.
 
 #ifndef MEDIA_AUDIO_SOUNDS_WAV_AUDIO_HANDLER_H_
 #define MEDIA_AUDIO_SOUNDS_WAV_AUDIO_HANDLER_H_
 
 #include "base/strings/string_piece.h"
+#include "base/time/time.h"
+#include "media/audio/audio_parameters.h"
 #include "media/base/media_export.h"
 
 namespace media {
 
 class AudioBus;
 
 // This class provides the input from wav file format.  See
 // https://ccrma.stanford.edu/courses/422/projects/WaveFormat/
 class MEDIA_EXPORT WavAudioHandler {
  public:
   explicit WavAudioHandler(const base::StringPiece& wav_data);
   virtual ~WavAudioHandler();
 
   // Returns true when cursor points to the end of the track.
   bool AtEnd(size_t cursor) const;
 
   // Copies the audio data to |bus| starting from the |cursor| and in
   // the case of success stores the number of written bytes in
   // |bytes_written|. |bytes_written| should not be NULL.
   bool CopyTo(AudioBus* bus, size_t cursor, size_t* bytes_written) const;
 
-  int size() const { return data_.size(); }
-  uint16 num_channels() const { return num_channels_; }
-  uint32 sample_rate() const { return sample_rate_; }
-  uint32 byte_rate() const { return byte_rate_; }
-  uint16 bits_per_sample() const { return bits_per_sample_; }
+  int GetByteRate() const;

[DaleCurtis, 2013/12/18 21:20:21]

Looks unused? Does this end up being different than
AudioParameters::GetBytesPerSecond() ?

[ygorshenin1, 2013/12/19 15:42:29]

Done.

+  const AudioParameters& params() const { return params_; }
+  const base::TimeDelta& duration() const { return duration_; }

[DaleCurtis, 2013/12/18 21:20:21]

TimeDelta can be passed by value efficiently, so no reference here. Also, does
AudioParameters::GetBufferDuration() not work here?

[ygorshenin1, 2013/12/19 15:42:29]

Fixed to return by value now.

+  const base::StringPiece& data() const { return data_; }

[DaleCurtis, 2013/12/18 21:20:21]

I don't think this should be exposed. I don't see where this is used either?

[ygorshenin1, 2013/12/19 15:42:29]

It's used once in wav_audio_handler_unittest.cc, for testing CopyTo().

On 2013/12/18 21:20:21, DaleCurtis wrote:

 
  private:
   // Parses a chunk of wav format data. Returns the length of the chunk.
   int ParseSubChunk(const base::StringPiece& data);
 
   // Parses the 'fmt' section chunk and stores |params_|.
   bool ParseFmtChunk(const base::StringPiece& data);
 
   // Parses the 'data' section chunk and stores |data_|.
   bool ParseDataChunk(const base::StringPiece& data);
 
   // Data part of the |wav_data_|.
   base::StringPiece data_;
 
-  uint16 num_channels_;
-  uint32 sample_rate_;
-  uint32 byte_rate_;
-  uint16 bits_per_sample_;
-  int bytes_per_sample_;
-  int bytes_per_frame_;
+  base::TimeDelta duration_;
+  AudioParameters params_;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_AUDIO_SOUNDS_WAV_AUDIO_HANDLER_H_