Commit eddfef94 authored by suderman@chromium.org's avatar suderman@chromium.org

Added UMA logging for Video Rotation metadata.

BUG=47554

Review URL: https://codereview.chromium.org/394373002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283962 0039d316-1c4b-4281-b951-d872f2087c98
parent 20d74a74
...@@ -963,6 +963,9 @@ void WebMediaPlayerImpl::OnPipelineMetadata( ...@@ -963,6 +963,9 @@ void WebMediaPlayerImpl::OnPipelineMetadata(
pipeline_metadata_ = metadata; pipeline_metadata_ = metadata;
UMA_HISTOGRAM_ENUMERATION("Media.VideoRotation",
metadata.video_rotation,
media::VIDEO_ROTATION_MAX + 1);
SetReadyState(WebMediaPlayer::ReadyStateHaveMetadata); SetReadyState(WebMediaPlayer::ReadyStateHaveMetadata);
if (hasVideo()) { if (hasVideo()) {
......
...@@ -13,7 +13,8 @@ enum VideoRotation { ...@@ -13,7 +13,8 @@ enum VideoRotation {
VIDEO_ROTATION_0 = 0, VIDEO_ROTATION_0 = 0,
VIDEO_ROTATION_90, VIDEO_ROTATION_90,
VIDEO_ROTATION_180, VIDEO_ROTATION_180,
VIDEO_ROTATION_270 VIDEO_ROTATION_270,
VIDEO_ROTATION_MAX = VIDEO_ROTATION_270
}; };
} // namespace media } // namespace media
......
...@@ -10639,6 +10639,11 @@ Therefore, the affected-histogram name has to have at least one dot in it. ...@@ -10639,6 +10639,11 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<summary>Pixel format used in HTML5 video. Emitted on video load.</summary> <summary>Pixel format used in HTML5 video. Emitted on video load.</summary>
</histogram> </histogram>
<histogram name="Media.VideoRotation" enum="VideoRotation">
<owner>suderman@chromium.org</owner>
<summary>Metadata rotation in mp4 videos. Emitted during demuxing.</summary>
</histogram>
<histogram name="Media.VideoVisibleAspectRatio"> <histogram name="Media.VideoVisibleAspectRatio">
<owner>scherkus@chromium.org</owner> <owner>scherkus@chromium.org</owner>
<summary>Visible aspect ratio of HTML5 video.</summary> <summary>Visible aspect ratio of HTML5 video.</summary>
...@@ -47944,6 +47949,13 @@ Therefore, the affected-histogram name has to have at least one dot in it. ...@@ -47944,6 +47949,13 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<int value="7" label="YV12J"/> <int value="7" label="YV12J"/>
</enum> </enum>
<enum name="VideoRotation" type="int">
<int value="0" label="VIDEO_ROTATION_0"/>
<int value="1" label="VIDEO_ROTATION_90"/>
<int value="2" label="VIDEO_ROTATION_180"/>
<int value="3" label="VIDEO_ROTATION_270"/>
</enum>
<enum name="ViewFileType" type="int"> <enum name="ViewFileType" type="int">
<int value="0" label="other"/> <int value="0" label="other"/>
<int value="1" label=".3ga"/> <int value="1" label=".3ga"/>
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment