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

Side by Side Diff: components/viz/client/client_compositor_frame_sink.h

Issue 2903853002: Make RendererCompositorFrameSink derive from ClientCompositorFrameSink (Closed)
Patch Set: Add comments Created 3 years, 6 months 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 unified diff | Download patch
« no previous file with comments | « no previous file | components/viz/client/client_compositor_frame_sink.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_VIZ_CLIENT_CLIENT_COMPOSITOR_FRAME_SINK_H_ 5 #ifndef COMPONENTS_VIZ_CLIENT_CLIENT_COMPOSITOR_FRAME_SINK_H_
6 #define COMPONENTS_VIZ_CLIENT_CLIENT_COMPOSITOR_FRAME_SINK_H_ 6 #define COMPONENTS_VIZ_CLIENT_CLIENT_COMPOSITOR_FRAME_SINK_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "cc/ipc/mojo_compositor_frame_sink.mojom.h" 9 #include "cc/ipc/mojo_compositor_frame_sink.mojom.h"
10 #include "cc/output/compositor_frame_sink.h" 10 #include "cc/output/compositor_frame_sink.h"
11 #include "cc/output/context_provider.h" 11 #include "cc/output/context_provider.h"
12 #include "cc/scheduler/begin_frame_source.h" 12 #include "cc/scheduler/begin_frame_source.h"
13 #include "cc/surfaces/local_surface_id_allocator.h" 13 #include "cc/surfaces/local_surface_id_allocator.h"
14 #include "cc/surfaces/surface_id.h" 14 #include "cc/surfaces/surface_id.h"
15 #include "mojo/public/cpp/bindings/binding.h" 15 #include "mojo/public/cpp/bindings/binding.h"
16 16
17 namespace viz { 17 namespace viz {
18 18
19 class ClientCompositorFrameSink 19 class ClientCompositorFrameSink
20 : public cc::CompositorFrameSink, 20 : public cc::CompositorFrameSink,
21 public cc::mojom::MojoCompositorFrameSinkClient, 21 public cc::mojom::MojoCompositorFrameSinkClient,
22 public cc::ExternalBeginFrameSourceClient { 22 public cc::ExternalBeginFrameSourceClient {
23 public: 23 public:
24 ClientCompositorFrameSink( 24 ClientCompositorFrameSink(
25 scoped_refptr<cc::ContextProvider> context_provider, 25 scoped_refptr<cc::ContextProvider> context_provider,
26 scoped_refptr<cc::ContextProvider> worker_context_provider,
26 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager, 27 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
28 cc::SharedBitmapManager* shared_bitmap_manager,
29 std::unique_ptr<cc::SyntheticBeginFrameSource>
30 synthetic_begin_frame_source,
27 cc::mojom::MojoCompositorFrameSinkPtrInfo compositor_frame_sink_info, 31 cc::mojom::MojoCompositorFrameSinkPtrInfo compositor_frame_sink_info,
28 cc::mojom::MojoCompositorFrameSinkClientRequest client_request, 32 cc::mojom::MojoCompositorFrameSinkClientRequest client_request,
29 bool enable_surface_synchronization); 33 bool enable_surface_synchronization);
34
35 ClientCompositorFrameSink(
36 scoped_refptr<cc::VulkanContextProvider> vulkan_context_provider,
37 std::unique_ptr<cc::SyntheticBeginFrameSource>
38 synthetic_begin_frame_source,
39 cc::mojom::MojoCompositorFrameSinkPtrInfo compositor_frame_sink_info,
40 cc::mojom::MojoCompositorFrameSinkClientRequest client_request,
41 bool enable_surface_synchronization);
30 42
31 ~ClientCompositorFrameSink() override; 43 ~ClientCompositorFrameSink() override;
32 44
33 // cc::CompositorFrameSink implementation. 45 // cc::CompositorFrameSink implementation.
34 bool BindToClient(cc::CompositorFrameSinkClient* client) override; 46 bool BindToClient(cc::CompositorFrameSinkClient* client) override;
35 void DetachFromClient() override; 47 void DetachFromClient() override;
36 void SetLocalSurfaceId(const cc::LocalSurfaceId& local_surface_id) override; 48 void SetLocalSurfaceId(const cc::LocalSurfaceId& local_surface_id) override;
37 void SubmitCompositorFrame(cc::CompositorFrame frame) override; 49 void SubmitCompositorFrame(cc::CompositorFrame frame) override;
38 void DidNotProduceFrame(const cc::BeginFrameAck& ack) override; 50 void DidNotProduceFrame(const cc::BeginFrameAck& ack) override;
39 51
40 private: 52 private:
53 virtual bool ShouldAllocateNewLocalSurfaceId(
54 const cc::CompositorFrame& frame);
55
41 // cc::mojom::MojoCompositorFrameSinkClient implementation: 56 // cc::mojom::MojoCompositorFrameSinkClient implementation:
42 void DidReceiveCompositorFrameAck( 57 void DidReceiveCompositorFrameAck(
43 const cc::ReturnedResourceArray& resources) override; 58 const cc::ReturnedResourceArray& resources) override;
44 void OnBeginFrame(const cc::BeginFrameArgs& begin_frame_args) override; 59 void OnBeginFrame(const cc::BeginFrameArgs& begin_frame_args) override;
45 void ReclaimResources(const cc::ReturnedResourceArray& resources) override; 60 void ReclaimResources(const cc::ReturnedResourceArray& resources) override;
46 61
47 // cc::ExternalBeginFrameSourceClient implementation. 62 // cc::ExternalBeginFrameSourceClient implementation.
48 void OnNeedsBeginFrames(bool needs_begin_frames) override; 63 void OnNeedsBeginFrames(bool needs_begin_frames) override;
49 64
50 gfx::Size last_submitted_frame_size_; 65 gfx::Size surface_size_;
66 float device_scale_factor_ = 0.f;
67
51 cc::LocalSurfaceId local_surface_id_; 68 cc::LocalSurfaceId local_surface_id_;
52 cc::LocalSurfaceIdAllocator id_allocator_; 69 cc::LocalSurfaceIdAllocator id_allocator_;
53 std::unique_ptr<cc::ExternalBeginFrameSource> begin_frame_source_; 70 std::unique_ptr<cc::ExternalBeginFrameSource> begin_frame_source_;
71 std::unique_ptr<cc::SyntheticBeginFrameSource> synthetic_begin_frame_source_;
54 cc::mojom::MojoCompositorFrameSinkPtrInfo compositor_frame_sink_info_; 72 cc::mojom::MojoCompositorFrameSinkPtrInfo compositor_frame_sink_info_;
55 cc::mojom::MojoCompositorFrameSinkClientRequest client_request_; 73 cc::mojom::MojoCompositorFrameSinkClientRequest client_request_;
56 cc::mojom::MojoCompositorFrameSinkPtr compositor_frame_sink_; 74 cc::mojom::MojoCompositorFrameSinkPtr compositor_frame_sink_;
57 std::unique_ptr<mojo::Binding<cc::mojom::MojoCompositorFrameSinkClient>> 75 mojo::Binding<cc::mojom::MojoCompositorFrameSinkClient> client_binding_;
58 client_binding_;
59 THREAD_CHECKER(thread_checker_); 76 THREAD_CHECKER(thread_checker_);
60 const bool enable_surface_synchronization_; 77 const bool enable_surface_synchronization_;
61 78
62 DISALLOW_COPY_AND_ASSIGN(ClientCompositorFrameSink); 79 DISALLOW_COPY_AND_ASSIGN(ClientCompositorFrameSink);
63 }; 80 };
64 81
65 } // namespace viz 82 } // namespace viz
66 83
67 #endif // COMPONENTS_VIZ_CLIENT_CLIENT_COMPOSITOR_FRAME_SINK_H_ 84 #endif // COMPONENTS_VIZ_CLIENT_CLIENT_COMPOSITOR_FRAME_SINK_H_
OLDNEW
« no previous file with comments | « no previous file | components/viz/client/client_compositor_frame_sink.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698