Improve input handling for WaveAudioHandler.

media/audio/simple_sources.cc

 // Copyright (c) 2012 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.
 // MSVC++ requires this to be set before any other includes to get M_PI.
 #define _USE_MATH_DEFINES
 #include <cmath>
 
 #include "media/audio/simple_sources.h"
 
 #include <algorithm>
 
 #include "base/files/file.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "media/audio/sounds/wav_audio_handler.h"
 #include "media/base/audio_bus.h"
 
 namespace media {
-
+namespace {
 // Opens |wav_filename|, reads it and loads it as a wav file. This function will
 // return a null pointer if we can't read the file or if it's malformed. The
 // caller takes ownership of the returned data. The size of the data is stored
 // in |read_length|.
-static scoped_ptr<uint8[]> ReadWavFile(const base::FilePath& wav_filename,
-                                       size_t* file_length) {
+scoped_ptr<char[]> ReadWavFile(const base::FilePath& wav_filename,
+                               size_t* read_length) {
   base::File wav_file(
       wav_filename, base::File::FLAG_OPEN | base::File::FLAG_READ);
   if (!wav_file.IsValid()) {
     LOG(ERROR) << "Failed to read " << wav_filename.value()
                << " as input to the fake device.";
     return nullptr;
   }
 
   size_t wav_file_length = wav_file.GetLength();
   if (wav_file_length == 0u) {
     LOG(ERROR) << "Input file to fake device is empty: "
                << wav_filename.value();
     return nullptr;
   }
 
-  uint8* wav_file_data = new uint8[wav_file_length];
-  size_t read_bytes = wav_file.Read(0, reinterpret_cast<char*>(wav_file_data),
-                                    wav_file_length);
+  scoped_ptr<char[]> data(new char[wav_file_length]);
+  size_t read_bytes = wav_file.Read(0, data.get(), wav_file_length);
   if (read_bytes != wav_file_length) {
     LOG(ERROR) << "Failed to read all bytes of " << wav_filename.value();
     return nullptr;
   }
-  *file_length = wav_file_length;
-  return scoped_ptr<uint8[]>(wav_file_data);
-}
-
-// Opens |wav_filename|, reads it and loads it as a wav file. This function will
-// bluntly trigger CHECKs if we can't read the file or if it's malformed.
-static scoped_ptr<WavAudioHandler> CreateWavAudioHandler(
-    const base::FilePath& wav_filename, const uint8* wav_file_data,
-    size_t wav_file_length, const AudioParameters& expected_params) {
-  base::StringPiece wav_data(reinterpret_cast<const char*>(wav_file_data),
-                             wav_file_length);
-  scoped_ptr<WavAudioHandler> wav_audio_handler(new WavAudioHandler(wav_data));
-  return wav_audio_handler.Pass();
+  *read_length = wav_file_length;
+  return data;
 }
 
 // These values are based on experiments for local-to-local
 // PeerConnection to demonstrate audio/video synchronization.
 static const int kBeepDurationMilliseconds = 20;
 static const int kBeepFrequency = 400;
 
 // Intervals between two automatic beeps.
 static const int kAutomaticBeepIntervalInMs = 500;
 
@@ skipping 23 matching lines @@
   }
 
  private:
   mutable base::Lock lock_;
   bool beep_once_;
   bool automatic_beep_;
 };
 
 static base::LazyInstance<BeepContext>::Leaky g_beep_context =
     LAZY_INSTANCE_INITIALIZER;
+}  // namespace
 
 //////////////////////////////////////////////////////////////////////////////
 // SineWaveAudioSource implementation.
 
 SineWaveAudioSource::SineWaveAudioSource(int channels,
                                          double freq, double sample_freq)
     : channels_(channels),
       f_(freq / sample_freq),
       time_state_(0),
       cap_(0),
@@ skipping 50 matching lines @@
 }
 
 FileSource::~FileSource() {
 }
 
 void FileSource::LoadWavFile(const base::FilePath& path_to_wav_file) {
   // Don't try again if we already failed.
   if (load_failed_)
     return;
 
-  // Read the file, and put its data in a scoped_ptr so it gets deleted later.
-  size_t file_length = 0;
-  wav_file_data_ = ReadWavFile(path_to_wav_file, &file_length);
-  if (!wav_file_data_) {
+  // Read the file, and put its data in a scoped_ptr so it gets deleted when
+  // this class destructs. This data must be valid for the lifetime of
+  // |wav_audio_handler_|.
+  size_t length = 0u;
+  raw_wav_data_ = ReadWavFile(path_to_wav_file, &length);
+  if (!raw_wav_data_) {
     load_failed_ = true;
     return;
   }
 
-  wav_audio_handler_ = CreateWavAudioHandler(
-      path_to_wav_file, wav_file_data_.get(), file_length, params_);
+  // Attempt to create a handler with this data. If the data is invalid, return.
+  wav_audio_handler_ =
+      WavAudioHandler::Create(base::StringPiece(raw_wav_data_.get(), length));
+  if (!wav_audio_handler_) {
+    LOG(ERROR) << "WAV data could be read but is not valid";
+    load_failed_ = true;
+    return;
+  }
 
   // Hook us up so we pull in data from the file into the converter. We need to
   // modify the wav file's audio parameters since we'll be reading small slices
   // of it at a time and not the whole thing (like 10 ms at a time).
   AudioParameters file_audio_slice(
       AudioParameters::AUDIO_PCM_LOW_LATENCY,
       GuessChannelLayout(wav_audio_handler_->num_channels()),
       wav_audio_handler_->sample_rate(), wav_audio_handler_->bits_per_sample(),
       params_.frames_per_buffer());
 
@@ skipping 101 matching lines @@
 }
 
 void BeepingSource::OnError(AudioOutputStream* stream) {
 }
 
 void BeepingSource::BeepOnce() {
   g_beep_context.Pointer()->SetBeepOnce(true);
 }
 
 }  // namespace media