New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 870050 link

Starred by 1 user
Status: Fixed
Owner:
█ hubbe@chromium.org
Last visit > 30 days ago
Closed: Aug 3
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

AOM codec should favor color space information from the stream, not the container

Project Member Reported by hubbe@chromium.org, Aug 1 
Chrome Version: ToT

Currently, the AOM codec defaults to using color space information from the container (if present) and only uses color space information from the stream if the container doesn't specify a color space. Since the stream has greater per-frame information and the container generally only has one color space for the whole stream, we should prefer to use the information from the stream over the information from the container.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 1 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/da6c4aa687511090c3f2dd543558ff6287bce646

commit da6c4aa687511090c3f2dd543558ff6287bce646
Author: Fredrik Hubinette <hubbe@google.com>
Date: Wed Aug 01 23:34:38 2018

Prefer the color spaces from the frames in AOM codec

Frames have more granular information than the container,
so let's use the color space from the frames rather than
from the container.

BUG= 870050 

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I4dd972561dd85f8ddce3ca55a01e34dc3a8cf386
Reviewed-on: https://chromium-review.googlesource.com/1155785
Commit-Queue: Fredrik Hubinette <hubbe@chromium.org>
Reviewed-by: Tom Finegan <tomfinegan@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579991}
[modify] https://crrev.com/da6c4aa687511090c3f2dd543558ff6287bce646/media/filters/aom_video_decoder.cc

Comment 2 by hubbe@chromium.org, Aug 3

Status: Fixed (was: Untriaged)
►

Sign in to add a comment