Commit b173f5d7 authored by Chih-Yu Huang's avatar Chih-Yu Huang Committed by Commit Bot

media/gpu/chromeos: Remove redundent code in VideoDecodePipeline.

In VideoDecodePipeline, we set frame metadata POWER_EFFICIENT twice.
This CL remove the one at OnFrameDecoded().

Bug: none
Test: Play video at crosvideo.appspot.com on kevin
Change-Id: I3ef4b45e72d9d1ec52621b993cec12e7a0e65057
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1914598Reviewed-by: default avatarHirokazu Honda <hiroh@chromium.org>
Reviewed-by: default avatarDavid Staessens <dstaessens@chromium.org>
Commit-Queue: Chih-Yu Huang <akahuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#715188}
parent 59cec088
......@@ -330,9 +330,6 @@ void VideoDecoderPipeline::OnFrameDecoded(scoped_refptr<VideoFrame> frame) {
DCHECK(frame_converter_);
DVLOGF(4);
// Flag that the video frame was decoded in a power efficient way.
frame->metadata()->SetBoolean(VideoFrameMetadata::POWER_EFFICIENT, true);
frame_converter_->ConvertFrame(std::move(frame));
}
......@@ -347,8 +344,9 @@ void VideoDecoderPipeline::OnFrameConverted(scoped_refptr<VideoFrame> frame) {
return;
}
// Flag that the video frame is capable of being put in an overlay.
frame->metadata()->SetBoolean(VideoFrameMetadata::ALLOW_OVERLAY, true);
// TODO(crbug.com/998547): Remove this specific statement when not needed.
// Flag that the video frame was decoded in a power efficient way.
frame->metadata()->SetBoolean(VideoFrameMetadata::POWER_EFFICIENT, true);
// MojoVideoDecoderService expects the |output_cb_| to be called on the client
......
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