Measure whether muted videos that started playing with play() become visible at some point

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

Index: third_party/WebKit/Source/core/html/HTMLMediaElement.h
diff --git a/third_party/WebKit/Source/core/html/HTMLMediaElement.h b/third_party/WebKit/Source/core/html/HTMLMediaElement.h
index 9863e39251d31afb640160653852f921101638df..08b7f2762daedc3465ba80817788810e1dd29e45 100644
--- a/third_party/WebKit/Source/core/html/HTMLMediaElement.h
+++ b/third_party/WebKit/Source/core/html/HTMLMediaElement.h
@@ -44,8 +44,11 @@
 
 namespace blink {
 
+enum class AutoplaySource;

[mlamouri (slow - plz ping), 2016/07/07 10:17:53]

I don't think you need this

[Zhiqiang Zhang (Slow), 2016/07/07 11:01:10]

Done.

+
 class AudioSourceProviderClient;
 class AudioTrackList;
+class AutoplayUmaHelper;
 class ContentType;
 class CueTimeline;
 class ElementVisibilityObserver;
@@ -298,13 +301,6 @@ protected:
     void recordAutoplayMetric(AutoplayMetrics);
 
 private:
-    // These values are used for histograms. Do not reorder.
-    enum AutoplayUnmuteActionStatus {
-        AutoplayUnmuteActionFailure = 0,
-        AutoplayUnmuteActionSuccess = 1,
-        AutoplayUnmuteActionMax
-    };
-
     void resetMediaPlayerAndMediaSource();
 
     bool alwaysCreateUserAgentShadowRoot() const final { return true; }
@@ -499,9 +495,6 @@ private:
 
     EnumerationHistogram& showControlsHistogram() const;
 
-    void recordAutoplaySourceMetric(int source);
-    void recordAutoplayUnmuteStatus(AutoplayUnmuteActionStatus);
-
     void onVisibilityChangedForAutoplay(bool isVisible);
 
     UnthrottledTimer<HTMLMediaElement> m_loadTimer;
@@ -678,6 +671,7 @@ private:
 
     Member<AutoplayExperimentHelper::Client> m_autoplayHelperClient;
     Member<AutoplayExperimentHelper> m_autoplayHelper;
+    Member<AutoplayUmaHelper> m_autoplayUmaHelper;
 
     WebRemotePlaybackClient* m_remotePlaybackClient;