Disable idle player suspend on OSX.

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 1087 matching lines @@
 
 void WebMediaPlayerImpl::OnSuspendRequested(bool must_suspend) {
   DCHECK(main_task_runner_->BelongsToCurrentThread());
 
 #if defined(OS_ANDROID)  // WMPI_CAST
   // If we're remote, the pipeline should already be suspended.
   if (isRemote())
     return;
 #endif
 
+#if defined(OS_MACOSX)
+  // TODO(sandersd): Idle suspend is disabled on OSX since hardware decoded
+  // frames are owned by the video decoder in the GPU process. A mechanism for
+  // detaching ownership from the decoder is needed. http://crbug.com/595716.
+  return;
+#else
   // Suspend should never be requested unless required or we're already in an
   // idle state (paused or ended).
   DCHECK(must_suspend || paused_ || ended_);
 
   // Always suspend, but only notify the delegate if we must; this allows any
   // exposed UI for player controls to continue to function even though the
   // player has now been suspended.
   pipeline_controller_.Suspend();
   if (must_suspend && delegate_)
     delegate_->PlayerGone(delegate_id_);
+#endif
 }
 
 void WebMediaPlayerImpl::OnPlay() {
   play();
   client_->playbackStateChanged();
 }
 
 void WebMediaPlayerImpl::OnPause() {
   pause();
   client_->playbackStateChanged();
@@ skipping 377 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