Register MediaPlayerRenderer service

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 20 matching lines @@
                                      const GetMediaTimeCB& get_media_time_cb)
     : has_audio_(has_audio),
       has_video_(has_video),
       is_mse_(is_mse),
       is_encrypted_(is_encrypted),
       media_log_(std::move(media_log)),
       initial_video_size_(initial_video_size),
       get_media_time_cb_(get_media_time_cb) {
   DCHECK(!get_media_time_cb_.is_null());
   DCHECK(has_audio_ || has_video_);
-  if (has_video_)
-    DCHECK(!initial_video_size_.IsEmpty());
 
   if (base::PowerMonitor* pm = base::PowerMonitor::Get())
     pm->AddObserver(this);
 }
 
 WatchTimeReporter::~WatchTimeReporter() {
   // If the timer is still running, finalize immediately, this is our last
   // chance to capture metrics.
   if (reporting_timer_.IsRunning())
     MaybeFinalizeWatchTime(FinalizeTime::IMMEDIATELY);
@@ skipping 60 matching lines @@
                            &WatchTimeReporter::UpdateWatchTime);
     return;
   }
 
   end_timestamp_for_power_ = kNoTimestamp;
 }
 
 bool WatchTimeReporter::ShouldReportWatchTime() {
   // Only report watch time for media of sufficient size with both audio and
   // video tracks present.
+  // TODO(tguilbert): HLS playback will always have an |initial_video_size_| of
+  // (0,0) and never report watchtime. Fix this as part of HLS UMA cleanup.
+  // See  crbug.com/650891.
   return has_audio_ && has_video_ &&
          initial_video_size_.height() >= kMinimumVideoSize.height() &&
          initial_video_size_.width() >= kMinimumVideoSize.width();
 }
 
 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.
@@ skipping 120 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