[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 2099 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());
-  return IsBackgroundVideoTrackOptimizationEnabled() && hasVideo() &&
-         hasAudio() && !IsStreaming();
+
+  if (!IsBackgroundVideoTrackOptimizationEnabled() || !hasVideo() ||
+      !hasAudio() || IsStreaming())
+    return false;

[DaleCurtis, 2017/01/11 01:14:51]

Multiline if needs {}

[whywhat, 2017/01/11 17:17:33]

Done.

+
+  PipelineStatistics stats = pipeline_.GetStatistics();
+  return stats.video_keyframe_distance_average <
+         max_keyframe_distance_for_fast_decode_;
 }
 
 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