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

Issue 851488 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 28
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 733482
issue 857216
issue 857217
issue 857218

Blocking:
issue 709332



Sign in to add a comment

content::GpuClient[Impl] should move into viz

Project Member Reported by sadrul@chromium.org, Jun 11 2018

Issue description

The ui::mojom::Gpu implementation in content: content::GpuClientImpl should be part of viz, and so it should move into components/viz/host/
 

Comment 1 by xing...@intel.com, Jun 14 2018

Cc: xing...@intel.com
Hi, I am willing to help on this issue. Does this still available?

Comment 2 by moh...@chromium.org, Jun 27 2018

Blockedon: 857216

Comment 3 by moh...@chromium.org, Jun 27 2018

Blockedon: 857217

Comment 4 by moh...@chromium.org, Jun 27 2018

Blockedon: 857218
Blockedon: 733482
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 20

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

commit c25273c32118532fcbe1d8857f292f510c58922b
Author: Mohsen Izadi <mohsen@chromium.org>
Date: Mon Aug 20 19:35:04 2018

Move GpuClientImpl to //components/viz/host/

GpuClientImpl is moved from //content/browser/gpu/ into
//components/viz/host/ and renamed to GpuClient.

BUG= 851488 

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: I27c1586bc269d7e6b7e55f1de16451e1c3528285
Reviewed-on: https://chromium-review.googlesource.com/1171665
Commit-Queue: Mohsen Izadi <mohsen@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584533}
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/ash/content/content_gpu_interface_provider.cc
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/components/viz/host/BUILD.gn
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/components/viz/host/DEPS
[rename] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/components/viz/host/gpu_client.cc
[rename] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/components/viz/host/gpu_client.h
[rename] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/components/viz/host/gpu_client_delegate.h
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/browser/BUILD.gn
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/browser/gpu/browser_gpu_client_delegate.cc
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/browser/gpu/browser_gpu_client_delegate.h
[add] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/browser/gpu/gpu_client.cc
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/browser/renderer_host/media/service_video_capture_provider.cc
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/browser/renderer_host/render_process_host_impl.h
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/browser/service_manager/common_browser_interfaces.cc
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/public/browser/BUILD.gn
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/public/browser/DEPS
[modify] https://crrev.com/c25273c32118532fcbe1d8857f292f510c58922b/content/public/browser/gpu_client.h

Status: Fixed (was: Assigned)

Sign in to add a comment