New issue
Advanced search Search tips

Issue 832913 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug

Blocking:
issue 827649



Sign in to add a comment

d3d11 zero copy: unit tests

Project Member Reported by liber...@chromium.org, Apr 13 2018

Issue description

mid-feb, rkuroiwa@ added d3d11 mocks.  we should use these for the d3d11 video decoder, which currently has no tests(!).
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 22 2018

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

commit 9ae98e6deba6d6e2602c4075a57bb739a741c2d8
Author: liberato@chromium.org <liberato@chromium.org>
Date: Sun Apr 22 06:41:04 2018

Started D3D11VideoDecoder unit tests.

These tests don't do much -- they just make sure that it fails to
init VPx but not H264.

Bug: 832913
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: Id1780306d9f4796a55a42e296c7e2c280483acbd
Reviewed-on: https://chromium-review.googlesource.com/1020227
Commit-Queue: Frank Liberato <liberato@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552601}
[modify] https://crrev.com/9ae98e6deba6d6e2602c4075a57bb739a741c2d8/media/base/test_helpers.cc
[modify] https://crrev.com/9ae98e6deba6d6e2602c4075a57bb739a741c2d8/media/base/test_helpers.h
[modify] https://crrev.com/9ae98e6deba6d6e2602c4075a57bb739a741c2d8/media/gpu/BUILD.gn
[modify] https://crrev.com/9ae98e6deba6d6e2602c4075a57bb739a741c2d8/media/gpu/windows/d3d11_video_decoder.cc
[modify] https://crrev.com/9ae98e6deba6d6e2602c4075a57bb739a741c2d8/media/gpu/windows/d3d11_video_decoder.h
[add] https://crrev.com/9ae98e6deba6d6e2602c4075a57bb739a741c2d8/media/gpu/windows/d3d11_video_decoder_unittest.cc
[modify] https://crrev.com/9ae98e6deba6d6e2602c4075a57bb739a741c2d8/media/mojo/services/gpu_mojo_media_client.cc

Owner: liber...@chromium.org
Status: Assigned (was: Available)
Assigning all P1 bugs.
You're the owner of a Pri-1 M-69 chrome media issue. M-69 is now in beta and will ship to stable in coming weeks. See go/chromeschedule. Please work on resolving your issue ASAP if it needs fixing for the M-69 branch.

Pri-1 means the work is required for the branch. Alternatively, update the milestone to M-70 or remove the milestone and drop the priority to P-3.
Labels: -Pri-1 Pri-2
Labels: -M-69 M-70
Labels: Pri-3

Sign in to add a comment