Commit cd37f908 authored by Miguel Casas's avatar Miguel Casas Committed by Commit Bot

GMBVideoFramePool UMA: only log non-texture VideoFrames

Follow up and correction to crrev.com/c/1132112: we should
only log those VideoFrames that are not supported _and_
that are not Textures/GMBs already. This CL adds the latter
part.

Bug: 787122
Change-Id: Ieb4e4fff0f2e14e9beb6949f1531507ac80733ce
Reviewed-on: https://chromium-review.googlesource.com/1140834Reviewed-by: default avatarDaniele Castagna <dcastagna@chromium.org>
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#575866}
parent 07f7e41c
...@@ -590,9 +590,11 @@ void GpuMemoryBufferVideoFramePool::PoolImpl::CreateHardwareFrame( ...@@ -590,9 +590,11 @@ void GpuMemoryBufferVideoFramePool::PoolImpl::CreateHardwareFrame(
case PIXEL_FORMAT_YUV444P12: case PIXEL_FORMAT_YUV444P12:
case PIXEL_FORMAT_Y16: case PIXEL_FORMAT_Y16:
case PIXEL_FORMAT_UNKNOWN: case PIXEL_FORMAT_UNKNOWN:
if (!video_frame->HasTextures()) {
UMA_HISTOGRAM_ENUMERATION( UMA_HISTOGRAM_ENUMERATION(
"Media.GpuMemoryBufferVideoFramePool.UnsupportedFormat", "Media.GpuMemoryBufferVideoFramePool.UnsupportedFormat",
video_frame->format(), PIXEL_FORMAT_MAX + 1); video_frame->format(), PIXEL_FORMAT_MAX + 1);
}
passthrough = true; passthrough = true;
} }
// TODO(dcastagna): Handle odd positioned video frame input, see // TODO(dcastagna): Handle odd positioned video frame input, see
......
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