Record whether and why the media default controls are being shown.

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 6d9157f8db8f8c50a7e3d6a60212167a8dedaf9e..41472e1ab66ad61f5494c5f1c21fcb959ca8b4f5 100644
--- a/third_party/WebKit/Source/core/html/HTMLMediaElement.h
+++ b/third_party/WebKit/Source/core/html/HTMLMediaElement.h
@@ -445,6 +445,8 @@ private:
     // TODO(liberato): remove once autoplay gesture override experiment concludes.
     void triggerAutoplayViewportCheckForTesting();
 
+    bool shouldShowControlsInternal(bool record) const;

[sof, 2016/02/18 16:12:17]

We prefer using enums rather bools in Blink.

[philipj_slow, 2016/02/19 04:05:43]

Yes, see `enum class RecordMetricsBehavior { DoNotRecord, DoRecord }`. If you do
that with a default argument, it should suffice to just keep
shouldShowControls(), with one place that passes DoRecord.

[mlamouri (slow - plz ping), 2016/02/23 18:42:13]

Re-used this enum. My understanding was that Chromium avoids default arguments
but I can see Blink doing this in a few places.

+
     UnthrottledTimer<HTMLMediaElement> m_loadTimer;
     UnthrottledTimer<HTMLMediaElement> m_progressEventTimer;
     UnthrottledTimer<HTMLMediaElement> m_playbackProgressTimer;