Autoplay time metric

third_party/WebKit/Source/core/html/media/AutoplayUmaHelper.cpp

 // 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 "core/html/media/AutoplayUmaHelper.h"
 
 #include "core/dom/Document.h"
 #include "core/dom/ElementVisibilityObserver.h"
 #include "core/events/Event.h"
 #include "core/frame/Settings.h"
 #include "core/frame/UseCounter.h"
 #include "core/html/HTMLMediaElement.h"
 #include "core/html/media/AutoplayPolicy.h"
 #include "platform/Histogram.h"
 #include "platform/wtf/CurrentTime.h"
 #include "public/platform/Platform.h"
 
 namespace blink {
 
 namespace {
 
 const int32_t kMaxOffscreenDurationUmaMS = 60 * 60 * 1000;
 const int32_t kOffscreenDurationUmaBucketCount = 50;
+const int32_t kMaxWaitTimeUmaMS = 30 * 1000;
+const int32_t kWaitTimeBucketCount = 50;
 
 }  // namespace
 
 AutoplayUmaHelper* AutoplayUmaHelper::Create(HTMLMediaElement* element) {
   return new AutoplayUmaHelper(element);
 }
 
 AutoplayUmaHelper::AutoplayUmaHelper(HTMLMediaElement* element)
     : EventListener(kCPPEventListenerType),
       ContextLifecycleObserver(nullptr),
       element_(element),
       muted_video_play_method_visibility_observer_(nullptr),
       muted_video_autoplay_offscreen_start_time_ms_(0),
       muted_video_autoplay_offscreen_duration_ms_(0),
       is_visible_(false),
-      muted_video_offscreen_duration_visibility_observer_(nullptr) {}
+      muted_video_offscreen_duration_visibility_observer_(nullptr),
+      load_start_time_ms_(0.0) {}
 
 AutoplayUmaHelper::~AutoplayUmaHelper() = default;
 
 bool AutoplayUmaHelper::operator==(const EventListener& other) const {
   return this == &other;
 }
 
+void AutoplayUmaHelper::OnLoadStarted(WebMediaPlayer::LoadType load_type) {
+  if (load_type == WebMediaPlayer::kLoadTypeURL)
+    load_start_time_ms_ = MonotonicallyIncreasingTimeMS();
+}

[mlamouri (slow - plz ping), 2017/05/16 12:59:40]

Instead of adding a "OnLoadStarted", I think it would be more efficient to do:
- in the ctor, add `element->addEventListener(EventTypeNames::loadstart, this,
false);
- in handleEvent, forward the call to a private "OnLoadStarted"
- in OnLoadStarted, check `element->GetLoadType()` instead of using `load_type`.

This will reduce the hard depedency between HTMLMediaElement and
AutoplayUmaHelper.

[hubbe, 2017/05/16 20:57:13]

Much better, done.
(I just had to make GetLoadType() public)

+
 void AutoplayUmaHelper::OnAutoplayInitiated(AutoplaySource source) {
+  int32_t autoplay_wait_time_ms = -1;
+  if (load_start_time_ms_ != 0.0) {
+    autoplay_wait_time_ms = static_cast<int32_t>(
+        std::min<int64_t>(MonotonicallyIncreasingTimeMS() - load_start_time_ms_,
+                          std::numeric_limits<int32_t>::max()));
+  }
   DEFINE_STATIC_LOCAL(EnumerationHistogram, video_histogram,
                       ("Media.Video.Autoplay",
                        static_cast<int>(AutoplaySource::kNumberOfUmaSources)));
   DEFINE_STATIC_LOCAL(EnumerationHistogram, muted_video_histogram,
                       ("Media.Video.Autoplay.Muted",
                        static_cast<int>(AutoplaySource::kNumberOfUmaSources)));
   DEFINE_STATIC_LOCAL(EnumerationHistogram, audio_histogram,
                       ("Media.Audio.Autoplay",
                        static_cast<int>(AutoplaySource::kNumberOfUmaSources)));
   DEFINE_STATIC_LOCAL(
       EnumerationHistogram, blocked_muted_video_histogram,
       ("Media.Video.Autoplay.Muted.Blocked", kAutoplayBlockedReasonMax));
 
+  DEFINE_STATIC_LOCAL(CustomCountHistogram, wait_time_video_attrib_histogram,
+                      ("Media.Video.Autoplay.Attribute.WaitTime", 1,
+                       kMaxWaitTimeUmaMS, kWaitTimeBucketCount));
+  DEFINE_STATIC_LOCAL(CustomCountHistogram, wait_time_audio_attrib_histogram,
+                      ("Media.Audio.Autoplay.Attribute.WaitTime", 1,
+                       kMaxWaitTimeUmaMS, kWaitTimeBucketCount));
+  DEFINE_STATIC_LOCAL(CustomCountHistogram, wait_time_video_play_histogram,
+                      ("Media.Video.Autoplay.PlayMethod.WaitTime", 1,
+                       kMaxWaitTimeUmaMS, kWaitTimeBucketCount));
+  DEFINE_STATIC_LOCAL(CustomCountHistogram, wait_time_audio_play_histogram,
+                      ("Media.Audio.Autoplay.PlayMethod.WaitTime", 1,
+                       kMaxWaitTimeUmaMS, kWaitTimeBucketCount));
+
   // Autoplay already initiated
   if (sources_.count(source))
     return;
 
   sources_.insert(source);
 
   // Record the source.
   if (element_->IsHTMLVideoElement()) {
     video_histogram.Count(static_cast<int>(source));
     if (element_->muted())
       muted_video_histogram.Count(static_cast<int>(source));
+    if (autoplay_wait_time_ms >= 0) {
+      if (source == AutoplaySource::kAttribute) {
+        wait_time_video_attrib_histogram.Count(autoplay_wait_time_ms);
+      } else if (source == AutoplaySource::kMethod) {
+        wait_time_video_play_histogram.Count(autoplay_wait_time_ms);
+      }
+    }
   } else {
     audio_histogram.Count(static_cast<int>(source));
+    if (autoplay_wait_time_ms >= 0) {
+      if (source == AutoplaySource::kAttribute) {
+        wait_time_audio_attrib_histogram.Count(autoplay_wait_time_ms);
+      } else if (source == AutoplaySource::kMethod) {
+        wait_time_audio_play_histogram.Count(autoplay_wait_time_ms);
+      }
+    }
   }
 
   // Record dual source.
   if (sources_.size() ==
       static_cast<size_t>(AutoplaySource::kNumberOfSources)) {
     if (element_->IsHTMLVideoElement()) {
       video_histogram.Count(static_cast<int>(AutoplaySource::kDualSource));
       if (element_->muted())
         muted_video_histogram.Count(
             static_cast<int>(AutoplaySource::kDualSource));
@@ skipping 315 matching lines @@
 
 DEFINE_TRACE(AutoplayUmaHelper) {
   EventListener::Trace(visitor);
   ContextLifecycleObserver::Trace(visitor);
   visitor->Trace(element_);
   visitor->Trace(muted_video_play_method_visibility_observer_);
   visitor->Trace(muted_video_offscreen_duration_visibility_observer_);
 }
 
 }  // namespace blink