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

Issue 787176 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 699255



Sign in to add a comment

Rename GpuJpegDecodeAcceleratorFactoryProvider to GpuJpegDecodeAcceleratorFactory

Project Member Reported by mcasas@chromium.org, Nov 21 2017

Issue description

So that it [1] can be like his peeps 
GpuVideoDecodeAcceleratorFactory and
GpuVideoEncodeAcceleratorFactory in the same folder

[1] https://cs.chromium.org/search/?q=GpuJpegDecodeAcceleratorFactoryProvider&sq=package:chromium&type=cs
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 21 2017

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

commit 366872053d5da2a353f4eeff12d5c4e87dc7d73b
Author: Miguel Casas <mcasas@chromium.org>
Date: Tue Nov 21 20:39:11 2017

move and rename GpuJpegDecodeAcceleratorFactoryProvider

This CL moves gpu_jpeg_decode_accelerator_factory_provider.* from
//media/gpu/ipc/service/ to //media/gpu/ and renames them (and the
class inside) to GpuJpegDecodeAcceleratorFactory, updating call
sites. No new code is intended.

TBR=rjkroege@chromium.org
 for components/viz/...gpu_service_impl.cc since it's pure renaming.

Bug:  787176 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ia85d58a9906752f49a5362c7cb85605986e44445
Reviewed-on: https://chromium-review.googlesource.com/781583
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: Chandan Padhi <c.padhi@samsung.com>
Cr-Commit-Position: refs/heads/master@{#518376}
[modify] https://crrev.com/366872053d5da2a353f4eeff12d5c4e87dc7d73b/components/viz/service/gl/gpu_service_impl.cc
[modify] https://crrev.com/366872053d5da2a353f4eeff12d5c4e87dc7d73b/media/gpu/BUILD.gn
[rename] https://crrev.com/366872053d5da2a353f4eeff12d5c4e87dc7d73b/media/gpu/gpu_jpeg_decode_accelerator_factory.cc
[rename] https://crrev.com/366872053d5da2a353f4eeff12d5c4e87dc7d73b/media/gpu/gpu_jpeg_decode_accelerator_factory.h
[modify] https://crrev.com/366872053d5da2a353f4eeff12d5c4e87dc7d73b/media/gpu/ipc/service/BUILD.gn
[modify] https://crrev.com/366872053d5da2a353f4eeff12d5c4e87dc7d73b/media/gpu/ipc/service/gpu_jpeg_decode_accelerator.cc
[modify] https://crrev.com/366872053d5da2a353f4eeff12d5c4e87dc7d73b/media/gpu/ipc/service/gpu_jpeg_decode_accelerator.h
[modify] https://crrev.com/366872053d5da2a353f4eeff12d5c4e87dc7d73b/media/gpu/jpeg_decode_accelerator_unittest.cc

Comment 2 by mcasas@chromium.org, Nov 22 2017

Blocking: 699255

Comment 3 by mcasas@chromium.org, Nov 22 2017

Cc: c.pa...@samsung.com
Status: Fixed (was: Assigned)

Sign in to add a comment