Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Unified Diff: tools/metrics/histograms/histograms.xml

Side-by-side diff isn't available for this file because of its large size.
Issue 297773004: Add UMA to study the effect of defaulting fullscreen video to landscape mode (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update comments Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
« no previous file with comments | « content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 8c420b2decee8dc16f7e733d5c01c7c725646ea1..4803d1feb74e3de33148829748771ddab824b6d6 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -10417,6 +10417,75 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="MobileFullscreenVideo.DurationAfterPotraitRotation"
+ units="milliseconds">
+ <owner>qinmin@chromium.org</owner>
+ <summary>
+ Android: Records the duration that a fullscreen video is played after device
+ rotates from portrait to landscape mode for the first time, and before it
+ exits fullscreen. If there is no device rotation or if the video starts with
+ landscape mode, it is not recorded. If there are mutiple rotations between
+ portrait and landscape mode, only one record is emitted and it is equal to
+ the time period from the first rotation to the moment when the video exits
+ fullscreen.
+ </summary>
+</histogram>
+
+<histogram name="MobileFullscreenVideo.LandscapeDuration" units="milliseconds">
+ <owner>qinmin@chromium.org</owner>
+ <summary>
+ Android: Records the duration that a fullscreen video is played in landscape
+ mode. If a video starts playing in landscape mode, and then it is switched
+ back and forth between landscape and portrait mode, only the time period
+ before the first switch is accounted. If a video starts playing in portrait
+ mode, it is not recorded.
Mark P 2014/05/23 22:01:40 Please make PortraitDuration needs analogously.
qinmin 2014/05/23 22:09:42 Done.
+ </summary>
+</histogram>
+
+<histogram name="MobileFullscreenVideo.LandscapeRotation" enum="BooleanEnabled">
+ <owner>qinmin@chromium.org</owner>
+ <summary>
+ Android: Records whether a fullscreen video is switched from landscape to
+ portrait mode at any point during playback.
+ </summary>
+</histogram>
+
+<histogram name="MobileFullscreenVideo.OrientationPortrait"
+ enum="BooleanEnabled">
+ <owner>qinmin@chromium.org</owner>
+ <summary>
+ Android: Records the device orientation when a video enters fullscreen. The
+ value is true if device orientation is portrait, or false otherwise. The
+ video doesn't necessarily needs to be in a playing state.
+ </summary>
+</histogram>
+
+<histogram name="MobileFullscreenVideo.PortraitDuration" units="milliseconds">
+ <owner>qinmin@chromium.org</owner>
+ <summary>
+ Android: Records the duration that a fullscreen video is played in portrait
+ mode. If a video is switched from portrait to landscape mode during the
+ playback, only the time period before the first switch is accounted. If a
+ video starts playing in landscape mode, it is not recorded.
+ </summary>
+</histogram>
+
+<histogram name="MobileFullscreenVideo.PortraitRotation" enum="BooleanEnabled">
+ <owner>qinmin@chromium.org</owner>
+ <summary>
+ Android: Records whether a fullscreen video is switched from portrait to
+ landscape mode at any point during playback.
+ </summary>
+</histogram>
+
+<histogram name="MobileFullscreenVideo.VideoPortrait" enum="BooleanEnabled">
+ <owner>qinmin@chromium.org</owner>
+ <summary>
+ Android: Records whether a video has a larger height than width when it
+ enters the fullscreen mode.
+ </summary>
+</histogram>
+
<histogram name="MobileStartup.MobileMultiWindowInstances">
<owner>dtrainor@chromium.org</owner>
<summary>
« no previous file with comments | « content/public/android/java/src/org/chromium/content/browser/ContentVideoView.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698