| Index: media/blink/webmediaplayer_impl_unittest.cc
|
| diff --git a/media/blink/webmediaplayer_impl_unittest.cc b/media/blink/webmediaplayer_impl_unittest.cc
|
| index b8d9b807bebec3e6e29a2d464528711f9a0192e3..ee63fedbcd176d194646f790293cfbb44fbebcb7 100644
|
| --- a/media/blink/webmediaplayer_impl_unittest.cc
|
| +++ b/media/blink/webmediaplayer_impl_unittest.cc
|
| @@ -866,15 +866,9 @@
|
|
|
| // Video is always paused when suspension is on and only if matches the
|
| // optimization criteria if the optimization is on.
|
| -#if defined(OS_ANDROID)
|
| bool should_pause = IsMediaSuspendOn() ||
|
| (IsBackgroundOptimizationOn() && matches_requirements);
|
| EXPECT_EQ(should_pause, ShouldPauseVideoWhenHidden());
|
| -#else
|
| - // TODO(avayvod): Revert after merging into 58 so we keep getting data on the
|
| - // background video pause behavior on desktop. See https://crbug.com/699106.
|
| - EXPECT_FALSE(ShouldPauseVideoWhenHidden());
|
| -#endif
|
| }
|
|
|
| TEST_P(WebMediaPlayerImplBackgroundBehaviorTest, AudioVideo) {
|
| @@ -889,16 +883,10 @@
|
| EXPECT_EQ(IsBackgroundOptimizationOn() && matches_requirements,
|
| ShouldDisableVideoWhenHidden());
|
|
|
| -// Only pause audible videos if both media suspend and resume
|
| -// background videos is on.
|
| -#if defined(OS_ANDROID)
|
| + // Only pause audible videos on Android if both media suspend and resume
|
| + // background videos is on. On Desktop
|
| EXPECT_EQ(IsMediaSuspendOn() && IsResumeBackgroundVideoEnabled(),
|
| ShouldPauseVideoWhenHidden());
|
| -#else
|
| - // TODO(avayvod): Revert after merging into 58 so we keep getting data on the
|
| - // background video pause behavior on desktop. See https://crbug.com/699106.
|
| - EXPECT_FALSE(ShouldPauseVideoWhenHidden());
|
| -#endif
|
| }
|
|
|
| INSTANTIATE_TEST_CASE_P(BackgroundBehaviorTestInstances,
|
|
|