Start reporting watch time if ABR adapts above 200p.

media/blink/watch_time_reporter.cc

 // Copyright 2016 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/watch_time_reporter.h"
 
 #include "base/power_monitor/power_monitor.h"
 
 namespace media {
 
@@ skipping 144 matching lines @@
   if (background_reporter_ && is_playing_)
     background_reporter_->OnPlaying();
 
   if (!has_video_)
     return;
 
   is_visible_ = false;
   MaybeFinalizeWatchTime(FinalizeTime::ON_NEXT_UPDATE);
 }
 
+bool WatchTimeReporter::IsSizeLargeEnoughToReportWatchTime() const {
+  return initial_video_size_.height() >= kMinimumVideoSize.height() &&
+         initial_video_size_.width() >= kMinimumVideoSize.width();
+}
+
 void WatchTimeReporter::OnPowerStateChange(bool on_battery_power) {
   if (!reporting_timer_.IsRunning())
     return;
 
   // Defer changing |is_on_battery_power_| until the next watch time report to
   // avoid momentary power changes from affecting the results.
   if (is_on_battery_power_ != on_battery_power) {
     end_timestamp_for_power_ = get_media_time_cb_.Run();
 
     // Restart the reporting timer so the full hysteresis is afforded.
     reporting_timer_.Start(FROM_HERE, reporting_interval_, this,
                            &WatchTimeReporter::UpdateWatchTime);
     return;
   }
 
   end_timestamp_for_power_ = kNoTimestamp;
 }
 
 bool WatchTimeReporter::ShouldReportWatchTime() {
   // Report listen time or watch time only for tracks that are audio-only or
   // have both an audio and video track of sufficient size.
   return (!has_video_ && has_audio_) ||
-         (has_video_ && has_audio_ &&
-          initial_video_size_.height() >= kMinimumVideoSize.height() &&
-          initial_video_size_.width() >= kMinimumVideoSize.width());
+         (has_video_ && has_audio_ && IsSizeLargeEnoughToReportWatchTime());
 }
 
 void WatchTimeReporter::MaybeStartReportingTimer(
     base::TimeDelta start_timestamp) {
   // Don't start the timer if any of our state indicates we shouldn't; this
   // check is important since the various event handlers do not have to care
   // about the state of other events.
   if (!ShouldReportWatchTime() || !is_playing_ || !volume_ || !is_visible_) {
     // If we reach this point the timer should already have been stopped or
     // there is a pending finalize in flight.
@@ skipping 132 matching lines @@
   }
 
   // Stop the timer if this is supposed to be our last tick.
   if (is_finalizing) {
     end_timestamp_ = kNoTimestamp;
     reporting_timer_.Stop();
   }
 }
 
 }  // namespace media