Autoplay experiment metric fixes and additions.

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

 // Copyright 2015 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 AutoplayExperimentHelper_h
 #define AutoplayExperimentHelper_h
 
 #include "core/page/Page.h"
 #include "platform/Timer.h"
 #include "platform/geometry/IntRect.h"
@@ skipping 44 matching lines @@
 
     // play() failed to play due to gesture requirement.
     PlayMethodFailed = 10,
 
     // Some play, whether user initiated or not, started.
     AnyPlaybackStarted = 11,
     // Some play, whether user initiated or not, paused.
     AnyPlaybackPaused = 12,
     // Some playback, whether user initiated or not, bailed out early.
     AnyPlaybackBailout = 13,
+    // Some playback, whether user initiated or not, played to completion.
+    AnyPlaybackComplete = 14,
+
+    // Number of media elements detected.
+    AnyMediaElement = 15,
+    // Numer of video elements detected.
+    AnyVideoElement = 16,
+
+    // User gesture was bypassed,and playback started, and media played to

[philipj_slow, 2015/11/25 14:03:41]

space before comma

[liberato (no reviews please), 2015/11/25 18:58:54]

Done.

+    // completion without a user-supplied pause.

[philipj_slow, 2015/11/25 14:03:41]

user-initiated maybe?

[liberato (no reviews please), 2015/11/25 18:58:54]

Done.

+    AutoplayComplete = 17,
+
+    // The initial playback started with a user gesture being processed by the
+    // element at or before.

[philipj_slow, 2015/11/25 14:03:41]

at or before... what? :)

[liberato (no reviews please), 2015/11/25 18:58:54]

Done.

+    InitialPlayWithUserGesture = 18,
+
+    // The initial playback started while a user gesture was being processed.
+    InitialPlayDuringUserGesture = 19,
+
+    // The initial playback started while no user gesture was being processed,
+    // though one might have been processed earlier to remove any gesture
+    // requirement (e.g., during load).
+    InitialPlayNotDuringUserGesture = 20,
+
+    // The initial playback started without any user gesture being processed
+    // by the element now or at any point prior to this.
+    InitialPlayWithoutUserGesture = 21,
 
     // This enum value must be last.
     NumberOfAutoplayMetrics,
 };
 
 class AutoplayExperimentHelper final {
     DISALLOW_NEW_EXCEPT_PLACEMENT_NEW();
 public:
     explicit AutoplayExperimentHelper(HTMLMediaElement&);
     ~AutoplayExperimentHelper();
@@ skipping 55 matching lines @@
     // requires it, else do nothing.
     void muteIfNeeded();
 
     // Maybe override the requirement for a user gesture, and start playing
     // autoplay media.  Returns true if only if it starts playback.
     bool maybeStartPlaying();
 
     // Configure internal state to record that the autoplay experiment is
     // going to start playback.  This doesn't actually start playback, since
     // there are several different cases.
-    void prepareToPlay(AutoplayMetrics);
+    void prepareToAutoplay(AutoplayMetrics);
 
     // Process a timer for checking visibility.
     void viewportTimerFired(Timer<AutoplayExperimentHelper>*);
 
     // Return our media element's document.
     Document& document() const;
 
     HTMLMediaElement& element() const;
 
     inline bool enabled(Mode mode) const
@@ skipping 31 matching lines @@
     const AutoplayExperimentHelper::Mode& b)
 {
     a = static_cast<AutoplayExperimentHelper::Mode>(static_cast<int>(a) | static_cast<int>(b));
     return a;
 }
 
 
 } // namespace blink
 
 #endif // AutoplayExperimentHelper_h