[Video] Disable bg optimization if avg keyframe distance is >10s

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 2096 matching lines @@
     return false;
 #else   // defined(OS_ANDROID)
   if (!IsBackgroundVideoTrackOptimizationEnabled())
     return false;
 #endif  // defined(OS_ANDROID)
 
   return hasVideo() && !hasAudio();
 }
 
 bool WebMediaPlayerImpl::ShouldDisableVideoWhenHidden() const {
+  DCHECK(main_task_runner_->BelongsToCurrentThread());
   DCHECK(IsHidden());
+
+  PipelineStatistics stats = pipeline_.GetStatistics();
   return IsBackgroundVideoTrackOptimizationEnabled() && hasVideo() &&

[DaleCurtis, 2017/01/10 21:11:40]

Early return before grabbing statistics if the rest of this is false.

[whywhat, 2017/01/10 21:51:54]

Done.

-         hasAudio() && !IsStreaming();
+         hasAudio() && !IsStreaming() &&
+         stats.video_keyframe_distance_average_ms <
+             max_keyframe_distance_for_fast_decode_.InMilliseconds();
 }
 
 void WebMediaPlayerImpl::EnableVideoTrackIfNeeded() {
   DCHECK(!IsHidden());
 
   // Don't change video track while the pipeline is resuming or seeking.
   if (is_pipeline_resuming_ || seeking_)
     return;
 
   if (video_track_disabled_) {
@@ skipping 12 matching lines @@
   if (is_pipeline_resuming_ || seeking_)
     return;
 
   if (!video_track_disabled_ && ShouldDisableVideoWhenHidden()) {
     video_track_disabled_ = true;
     selectedVideoTrackChanged(nullptr);
   }
 }
 
 }  // namespace media