Integrate HTMLMediaElement with Web Audio API's MediaElementAudioSourceNode

content/renderer/media/audio_device.cc

Index: content/renderer/media/audio_device.cc
===================================================================
--- content/renderer/media/audio_device.cc	(revision 114311)
+++ content/renderer/media/audio_device.cc	(working copy)
@@ -60,6 +60,10 @@
   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;
@@ -79,10 +83,6 @@
   is_initialized_ = true;
 }
 
-bool AudioDevice::IsInitialized() {
-  return is_initialized_;
-}
-
 AudioDevice::~AudioDevice() {
   // The current design requires that the user calls Stop() before deleting
   // this class.