Integrate HTMLMediaElement with Web Audio API's MediaElementAudioSourceNode

content/renderer/media/audio_device.cc

 // Copyright (c) 2011 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 "content/renderer/media/audio_device.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/message_loop.h"
 #include "base/time.h"
@@ skipping 42 matching lines @@
 }
 
 void AudioDevice::Initialize(size_t buffer_size,
                              int channels,
                              double sample_rate,
                              AudioParameters::Format latency_format,
                              RenderCallback* callback) {
   CHECK_EQ(0, stream_id_) <<
       "AudioDevice::Initialize() must be called before Start()";
 
+  DCHECK(!is_initialized_);

[acolwell GONE FROM CHROMIUM, 2011/12/21 22:53:56]

Should this be a CHECK() ? Do we really want to try to proceed  in release mode
if this gets called twice? 

[Chris Rogers, 2011/12/22 00:54:33]

Done.

+  if (is_initialized_)

[acolwell GONE FROM CHROMIUM, 2011/12/21 22:53:56]

I don't think this should be here. We should either support multiple
Initialize() calls or not. Returning early here seem like it could lead to
strange behavior especially since the caller has know way of knowing that all of
the parameters passed in are being completely ignored.

[Chris Rogers, 2011/12/22 00:54:33]

Done.

+    return;
+
   buffer_size_ = buffer_size;
   channels_ = channels;
   sample_rate_ = sample_rate;
   latency_format_ = latency_format;
   callback_ = callback;
 
   // Cleanup from any previous initialization.
   for (size_t i = 0; i < audio_data_.size(); ++i)
     delete [] audio_data_[i];
 
   audio_data_.reserve(channels);
   for (int i = 0; i < channels; ++i) {
     float* channel_data = new float[buffer_size];
     audio_data_.push_back(channel_data);
   }
 
   is_initialized_ = true;
 }
 
-bool AudioDevice::IsInitialized() {
-  return is_initialized_;
-}
-
 AudioDevice::~AudioDevice() {
   // The current design requires that the user calls Stop() before deleting
   // this class.
   CHECK_EQ(0, stream_id_);
   for (int i = 0; i < channels_; ++i)
     delete [] audio_data_[i];
 }
 
 void AudioDevice::Start() {
   AudioParameters params;
@@ skipping 239 matching lines @@
   if (audio_thread_.get()) {
     // Close the socket handler to terminate the main thread function in the
     // audio thread.
     {
       base::SyncSocket socket(socket_handle_);
     }
     audio_thread_->Join();
     audio_thread_.reset(NULL);
   }
 }