Remove asserts in RenderView for audio mesages
The asserts is too aggressive which are causing problems
when closing the tab or reload.
Review URL: http://codereview.chromium.org/62033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@13216 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc
index 0b36bd0..eb5a7e3 100644
--- a/chrome/renderer/render_view.cc
+++ b/chrome/renderer/render_view.cc
@@ -2901,7 +2901,9 @@
void RenderView::OnRequestAudioPacket(int stream_id) {
AudioRendererImpl* audio_renderer = audio_renderers_.Lookup(stream_id);
if (!audio_renderer) {
- NOTREACHED();
+ // It is possible that AudioRendererImpl is un-registered but we still
+ // receives packet requests here, because of closing a stream is not a
+ // synchronous operation with the browser process.
return;
}
audio_renderer->OnRequestPacket();
@@ -2911,7 +2913,6 @@
int stream_id, base::SharedMemoryHandle handle, int length) {
AudioRendererImpl* audio_renderer = audio_renderers_.Lookup(stream_id);
if (!audio_renderer) {
- NOTREACHED();
return;
}
audio_renderer->OnCreated(handle, length);
@@ -2921,7 +2922,6 @@
int stream_id, AudioOutputStream::State state, int info) {
AudioRendererImpl* audio_renderer = audio_renderers_.Lookup(stream_id);
if (!audio_renderer) {
- NOTREACHED();
return;
}
audio_renderer->OnStateChanged(state, info);
@@ -2930,7 +2930,6 @@
void RenderView::OnAudioStreamVolume(int stream_id, double left, double right) {
AudioRendererImpl* audio_renderer = audio_renderers_.Lookup(stream_id);
if (!audio_renderer) {
- NOTREACHED();
return;
}
audio_renderer->OnVolume(left, right);
