Adding audio support to the new webmediaplyer

media/base/rtc_audio_renderer.cc

Index: media/base/rtc_audio_renderer.cc
diff --git a/media/base/audio_renderer.cc b/media/base/rtc_audio_renderer.cc
similarity index 63%
copy from media/base/audio_renderer.cc
copy to media/base/rtc_audio_renderer.cc
index e7b737ece5e1cbbd1f44cd3ebc1b15209ee9169a..a2b64962b901fa68ca18e355ae495e6bec4a7c38 100644
--- a/media/base/audio_renderer.cc
+++ b/media/base/rtc_audio_renderer.cc
@@ -2,11 +2,11 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 

[wjia(left Chromium), 2012/10/24 22:05:15]

Is this a newly added file, or a renamed version of AudioRenderer? Somehow the
history of this file is different from its header file.

[no longer working on chromium, 2012/10/25 10:19:41]

Yes, it is weird. It is a new file. I copied the audio_renderer.h/cc and
modified them to rtc_audio_renderer.h/cc, for some reason git cl keeps the
history of audio_renderer.cc. I am not sure how to remove the wrong history.

Another thing here is the naming, it is unsure if rtc_audio_renderer is a good
name, since I use webrtc_audio_renderer for its implementation in
content/renderer/media. How do you think if I change the interface file to
webrtc_audio_renderer, and use webrtc_audio_renderer_impl as the implementation
file name?
Or any other suggestion?

[wjia(left Chromium), 2012/10/25 14:15:05]

From WebMediaPlayer point of view, it's audio_player, not just audio_renderer.
Some people think "pause" will pause the source of media element as well. I am
trying to sort out what's really desired. 

For now, it's ok to keep current name. We can refactor later.

-#include "media/base/audio_renderer.h"
+#include "media/base/rtc_audio_renderer.h"
 
 namespace media {
 
-AudioRenderer::AudioRenderer() {}
-AudioRenderer::~AudioRenderer() {}
+RtcAudioRenderer::RtcAudioRenderer() {}
+RtcAudioRenderer::~RtcAudioRenderer() {}
 
 }  // namespace media