Drop media session on playback error; avoids idle suspension.

media/blink/webmediaplayer_impl.cc

 // Copyright 2013 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 "media/blink/webmediaplayer_impl.h"
 
 #include <algorithm>
 #include <cmath>
 #include <limits>
 #include <string>
@@ skipping 920 matching lines @@
 }
 
 void WebMediaPlayerImpl::OnPipelineError(PipelineStatus error) {
   DVLOG(1) << __FUNCTION__;
   DCHECK(main_task_runner_->BelongsToCurrentThread());
   DCHECK_NE(error, PIPELINE_OK);
 
   if (suppress_destruction_errors_)
     return;
 
+  // Release the delegate for player errors; this drops the media session and
+  // avoids idle suspension from ticking.
+  if (delegate_)
+    delegate_->PlayerGone(delegate_id_);
+
 #if defined(OS_ANDROID)
   // For 10% of pipeline decode failures log the playback URL. The URL is set
   // as the crash-key 'subresource_url' during DoLoad().
   //
   // TODO(dalecurtis): This is temporary to track down higher than average
   // decode failure rates for video-only content. See http://crbug.com/595076.
   if (base::RandDouble() <= 0.1 && error == PIPELINE_ERROR_DECODE)
     base::debug::DumpWithoutCrashing();
 #endif
 
@@ skipping 561 matching lines @@
 bool WebMediaPlayerImpl::IsAutomaticResumeAllowed() {
 #if defined(OS_ANDROID)
   return !hasVideo() || (delegate_ && !delegate_->IsHidden());
 #else
   // On non-Android platforms Resume() is always allowed.
   return true;
 #endif
 }
 
 }  // namespace media