Adding Rappor metrics for disabling cross-origin autoplay with sound experiment

third_party/WebKit/Source/core/html/AutoplayUmaHelper.h

 // 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.
 
 #ifndef AutoplayUmaHelper_h
 #define AutoplayUmaHelper_h
 
 #include "core/events/EventListener.h"
 #include "platform/heap/Handle.h"
 
+#include <set>
+
 namespace blink {
 
 // These values are used for histograms. Do not reorder.
 enum class AutoplaySource {
   // Autoplay comes from HTMLMediaElement `autoplay` attribute.
   Attribute = 0,
   // Autoplay comes from `play()` method.
   Method = 1,
   // This enum value must be last.
   NumberOfSources = 2,
 };
 
 // These values are used for histograms. Do not reorder.
 enum class AutoplayUnmuteActionStatus {
   Failure = 0,
   Success = 1,
   NumberOfStatus = 2,
 };
 
 // These values are used for histograms. Do not reorder.
 enum AutoplayBlockedReason {
   AutoplayBlockedReasonDataSaver = 0,
   AutoplayBlockedReasonSetting = 1,
   AutoplayBlockedReasonDataSaverAndSetting = 2,
   // Keey at the end.
   AutoplayBlockedReasonMax = 3
 };
 
+enum class AutoplayCrossOriginExperimentMetric {

[whywhat, 2016/11/29 21:42:04]

nit: I think we could avoid having experiment and metric in the metric enum
names
First, we may want to upgrade the code to stable if the experiment is successful
(and keep the metrics) - you probably don't want to rename everything then.
Second, Metric is not descriptive for what is being measured.

Maybe something like CrossOriginAutoplayResult?
Similar comment applies to other names in this and the dependent patch.

[Zhiqiang Zhang (Slow), 2016/11/30 17:30:52]

Done.

+  AutoplayAllowed,
+  AutoplayBlocked,
+  PlayedWithGesture,
+};
+
 class Document;
 class ElementVisibilityObserver;
 class HTMLMediaElement;
 
 class AutoplayUmaHelper final : public EventListener {
  public:
   static AutoplayUmaHelper* create(HTMLMediaElement*);
 
   ~AutoplayUmaHelper();
 
   bool operator==(const EventListener&) const override;
 
   void onAutoplayInitiated(AutoplaySource);
 
+  void recordCrossOriginExperimentMetric(AutoplayCrossOriginExperimentMetric);
   void recordAutoplayUnmuteStatus(AutoplayUnmuteActionStatus);
 
   void didMoveToNewDocument(Document& oldDocument);
 
   bool isVisible() const { return m_isVisible; }
 
   bool hasSource() const { return m_source != AutoplaySource::NumberOfSources; }
 
   DECLARE_VIRTUAL_TRACE();
 
@@ skipping 38 matching lines @@
 
   // The starting time of autoplaying muted video.
   int64_t m_mutedVideoAutoplayOffscreenStartTimeMS;
 
   // The duration an autoplaying muted video has been in offscreen.
   int64_t m_mutedVideoAutoplayOffscreenDurationMS;
 
   // Whether an autoplaying muted video is visible.
   bool m_isVisible;
 
+  std::set<AutoplayCrossOriginExperimentMetric>
+      m_recordedCrossOriginExperimentMetrics;
+
   // The observer is used to observer an autoplaying muted video changing it's
   // visibility, which is used for offscreen duration UMA.  The UMA is pending
   // for recording as long as this observer is non-null.
   Member<ElementVisibilityObserver>
       m_mutedVideoOffscreenDurationVisibilityObserver;
 };
 
 }  // namespace blink
 
 #endif  // AutoplayUmaHelper_h