[Video, Android] Pause video only players ignoring duration and keyframes

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 2138 matching lines @@
     return false;
 
   // Disable video track only for players with audio that match the criteria for
   // being optimized.
   return hasAudio() && IsBackgroundOptimizationCandidate();
 }
 
 bool WebMediaPlayerImpl::IsBackgroundOptimizationCandidate() const {
   DCHECK(main_task_runner_->BelongsToCurrentThread());
 
-// Don't optimize players being Cast (Android only).
 #if defined(OS_ANDROID)  // WMPI_CAST
+  // Don't optimize players being Cast.
   if (isRemote())
     return false;
+
+  // Video-only players are always optimized (paused) on Android.
+  // Don't check the keyframe distance and duration.
+  if (!hasAudio() && hasVideo())
+    return true;
 #endif  // defined(OS_ANDROID)
 
   // Don't optimize audio-only or streaming players.
   if (!hasVideo() || IsStreaming())
     return false;
 
   // Videos shorter than the maximum allowed keyframe distance can be optimized.
   base::TimeDelta duration = GetPipelineMediaDuration();
   if (duration < max_keyframe_distance_to_disable_background_video_)
     return true;
@@ skipping 64 matching lines @@
     UMA_HISTOGRAM_TIMES(
         "Media.Video.TimeFromForegroundToFirstFrame.DisableTrack",
         time_to_first_frame);
   } else {
     UMA_HISTOGRAM_TIMES("Media.Video.TimeFromForegroundToFirstFrame.Paused",
                         time_to_first_frame);
   }
 }
 
 }  // namespace media