Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2679)

Unified Diff: services/ui/ws/gpu_service_proxy.h

Issue 2481263002: Introduce Display Compositor mojo interface. Use InProcessContextProvider. (Closed)
Patch Set: Speculative fix for android build issue Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: services/ui/ws/gpu_service_proxy.h
diff --git a/services/ui/ws/gpu_service_proxy.h b/services/ui/ws/gpu_service_proxy.h
index f5d8446173665a22047830fe3655377dfcd9b1ce..beec7a93a03cb57dd1aec48c8bcf3b8cdaf764b0 100644
--- a/services/ui/ws/gpu_service_proxy.h
+++ b/services/ui/ws/gpu_service_proxy.h
@@ -17,36 +17,33 @@
#include "services/ui/public/interfaces/gpu_memory_buffer.mojom.h"
#include "services/ui/public/interfaces/gpu_service.mojom.h"
-namespace gpu {
-class GpuChannelHost;
-}
-
namespace ui {
class GpuServiceInternal;
+class MusGpuMemoryBufferManager;
namespace ws {
class GpuServiceProxyDelegate;
-class MusGpuMemoryBufferManager;
// The proxy implementation that relays requests from clients to the real
// service implementation in the GPU process over mojom.GpuServiceInternal.
-class GpuServiceProxy : public mojom::GpuService,
- public gpu::GpuChannelHostFactory {
+class GpuServiceProxy : public mojom::GpuService {
public:
explicit GpuServiceProxy(GpuServiceProxyDelegate* delegate);
~GpuServiceProxy() override;
void Add(mojom::GpuServiceRequest request);
+ // Requests a cc::mojom::DisplayCompositor interface from mus-gpu.
+ void CreateDisplayCompositor(cc::mojom::DisplayCompositorRequest request,
+ cc::mojom::DisplayCompositorClientPtr client);
+
private:
void OnInitialized(const gpu::GPUInfo& gpu_info);
void OnGpuChannelEstablished(const EstablishGpuChannelCallback& callback,
int32_t client_id,
mojo::ScopedMessagePipeHandle channel_handle);
- void OnInternalGpuChannelEstablished(
- mojo::ScopedMessagePipeHandle channel_handle);
// mojom::GpuService overrides:
void EstablishGpuChannel(
@@ -61,22 +58,12 @@ class GpuServiceProxy : public mojom::GpuService,
void DestroyGpuMemoryBuffer(gfx::GpuMemoryBufferId id,
const gpu::SyncToken& sync_token) override;
- // gpu::GpuChannelHostFactory overrides:
- bool IsMainThread() override;
- scoped_refptr<base::SingleThreadTaskRunner> GetIOThreadTaskRunner() override;
- std::unique_ptr<base::SharedMemory> AllocateSharedMemory(
- size_t size) override;
-
- GpuServiceProxyDelegate* delegate_;
+ GpuServiceProxyDelegate* const delegate_;
int32_t next_client_id_;
scoped_refptr<base::SingleThreadTaskRunner> main_thread_task_runner_;
mojom::GpuServiceInternalPtr gpu_service_;
mojo::BindingSet<mojom::GpuService> bindings_;
gpu::GPUInfo gpu_info_;
- scoped_refptr<gpu::GpuChannelHost> gpu_channel_;
- base::WaitableEvent shutdown_event_;
- std::unique_ptr<base::Thread> io_thread_;
- std::unique_ptr<MusGpuMemoryBufferManager> gpu_memory_buffer_manager_;
GpuMain gpu_main_;
DISALLOW_COPY_AND_ASSIGN(GpuServiceProxy);

Powered by Google App Engine
This is Rietveld 408576698