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

media/blink/webmediaplayer_impl_unittest.cc

Index: media/blink/webmediaplayer_impl_unittest.cc
diff --git a/media/blink/webmediaplayer_impl_unittest.cc b/media/blink/webmediaplayer_impl_unittest.cc
index 2071eed5003cedb227b2b6e44cdff944a8b7af41..d06cd7e7407b6fc59fe8fa8a901d210d64bcf504 100644
--- a/media/blink/webmediaplayer_impl_unittest.cc
+++ b/media/blink/webmediaplayer_impl_unittest.cc
@@ -738,12 +738,25 @@ TEST_F(WebMediaPlayerImplTest, BackgroundOptimizationsFeatureDisabled) {
   // Video only (pausing is enabled on Android).
   SetMetadata(false, true);
   EXPECT_TRUE(IsBackgroundOptimizationCandidate());
+  EXPECT_FALSE(ShouldDisableVideoWhenHidden());
 #if defined(OS_ANDROID)
   EXPECT_TRUE(ShouldPauseVideoWhenHidden());
+
+  // On Android, the duration and keyframe distance don't matter for video-only.
+  SetDuration(base::TimeDelta::FromSeconds(5));
+  EXPECT_TRUE(IsBackgroundOptimizationCandidate());
+  EXPECT_TRUE(ShouldPauseVideoWhenHidden());
+
+  SetVideoKeyframeDistanceAverage(base::TimeDelta::FromSeconds(100));
+  SetDuration(base::TimeDelta::FromSeconds(300));
+  EXPECT_TRUE(IsBackgroundOptimizationCandidate());
+  EXPECT_TRUE(ShouldPauseVideoWhenHidden());
+
+  // Restore average keyframe distance.
+  SetVideoKeyframeDistanceAverage(base::TimeDelta::FromSeconds(5));
 #else
   EXPECT_FALSE(ShouldPauseVideoWhenHidden());
 #endif
-  EXPECT_FALSE(ShouldDisableVideoWhenHidden());
 
   // Audio only.
   SetMetadata(true, false);