Keep playing audio even when opening another tab, going to another app etc.

content/renderer/render_view_impl.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.
 
 #include "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/bind.h"
@@ skipping 5662 matching lines @@
 }
 
 void RenderViewImpl::DidHandleTouchEvent(const WebTouchEvent& event) {
   FOR_EACH_OBSERVER(RenderViewObserver, observers_, DidHandleTouchEvent(event));
 }
 
 void RenderViewImpl::OnWasHidden() {
   RenderWidget::OnWasHidden();
 
 #if defined(OS_ANDROID)
+  DCHECK(media_player_manager_);

[qinmin, 2012/11/14 18:14:29]

this DCHECK is useless, if media_player_manager_ is null, it will crash in code
below.

[Miguel Garcia, 2012/11/14 18:50:32]

Done.

+
   // Inform WebMediaPlayerManagerAndroid to release all media player resources.
-  media_player_manager_->ReleaseMediaResources();
+  // unless some audio is playing.

[qinmin, 2012/11/14 18:14:29]

some media is playing

[Miguel Garcia, 2012/11/14 18:50:32]

Done.

+  // If something is in progress the resource will not be freed, it will
+  // only be freed once the tab is destroyed or if the user navigates away
+  // via WebMediaPlayerAndroid::Destroy
+  media_player_manager_->ReleaseMediaResourcesIfNotPlaying();
 #endif
 
   if (webview()) {
     webview()->settings()->setMinimumTimerInterval(
         webkit_glue::kBackgroundTabTimerInterval);
     webview()->setVisibilityState(visibilityState(), false);
   }
 
   // Inform PPAPI plugins that their page is no longer visible.
   pepper_delegate_.PageVisibilityChanged(false);
@@ skipping 666 matching lines @@
 }
 #endif
 
 void RenderViewImpl::OnReleaseDisambiguationPopupDIB(
     TransportDIB::Handle dib_handle) {
   TransportDIB* dib = TransportDIB::CreateWithHandle(dib_handle);
   RenderProcess::current()->ReleaseTransportDIB(dib);
 }
 
 }  // namespace content