[Blink>Media] Adding rappor metrics to record autoplay muted by attribute usage

third_party/WebKit/Source/core/html/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/AutoplayUmaHelper.h"
 
 #include "core/dom/Document.h"
 #include "core/dom/ElementVisibilityObserver.h"
 #include "core/events/Event.h"
 #include "core/frame/Settings.h"
@@ skipping 56 matching lines @@
 
   // Record the source.
   if (m_element->isHTMLVideoElement()) {
     videoHistogram.count(static_cast<int>(m_source));
     if (m_element->muted())
       mutedVideoHistogram.count(static_cast<int>(m_source));
   } else {
     audioHistogram.count(static_cast<int>(m_source));
   }
 
+  // Record the child frame and top-level frame URLs for autoplay muted videos
+  // by attribute.
+  if (m_elemnet->isHTMLVideoElement() && m_element->muted() &&
+      source == AutoplaySource::Attribute) {
+    Platform::current()->recordRapporURL(
+        "Media.Video.Autoplay.Muted.Attribute.ChildFrame",
+        m_element->document().url());
+    Platform::current()->recordRapporURL(
+        "Media.Video.Autoplay.Muted.Attribute.TopLevelFrame",
+        m_element->document().topDocument().url());

[mlamouri (slow - plz ping), 2017/02/20 11:23:43]

FWIW, I'm not sure if that's the right thing to do: it actually doesn't record
the URL of the top level frame but the top level local frame. Maybe not
recording when the top level is a remote frame would be better than recording a
wrong value?

[Zhiqiang Zhang (Slow), 2017/02/20 17:49:43]

Maybe we just don't record the top-level frames at all? After OOPIFs is enabled,
the metric would barely make sense unless we can find some other way to get the
top-level frame URL.

+  }
+
   // Record if it will be blocked by Data Saver or Autoplay setting.
   if (m_element->isHTMLVideoElement() && m_element->muted() &&
       RuntimeEnabledFeatures::autoplayMutedVideosEnabled()) {
     bool dataSaverEnabled =
         m_element->document().settings() &&
         m_element->document().settings()->getDataSaverEnabled();
     bool blockedBySetting = !m_element->isAutoplayAllowedPerSettings();
 
     if (dataSaverEnabled && blockedBySetting) {
       blockedMutedVideoHistogram.count(
@@ skipping 274 matching lines @@
 
 DEFINE_TRACE(AutoplayUmaHelper) {
   EventListener::trace(visitor);
   ContextLifecycleObserver::trace(visitor);
   visitor->trace(m_element);
   visitor->trace(m_mutedVideoPlayMethodVisibilityObserver);
   visitor->trace(m_mutedVideoOffscreenDurationVisibilityObserver);
 }
 
 }  // namespace blink