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

Side by Side Diff: services/ui/ws/gpu_compositor_frame_sink.cc

Issue 2494663004: Mus+Ash: ServerWindowCompositorFrameSink => GpuCompositorFrameSink (Closed)
Patch Set: ServerWindowCompositorFrameSinkManager should not reference SurfaceFactory or SurfaceManager 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 unified diff | Download patch
« no previous file with comments | « services/ui/ws/gpu_compositor_frame_sink.h ('k') | services/ui/ws/server_window.h » ('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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #include "services/ui/ws/server_window_compositor_frame_sink.h" 5 #include "services/ui/ws/gpu_compositor_frame_sink.h"
6 6
7 #include "base/callback.h" 7 #include "base/callback.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "base/single_thread_task_runner.h" 9 #include "base/single_thread_task_runner.h"
10 #include "base/threading/thread_task_runner_handle.h" 10 #include "base/threading/thread_task_runner_handle.h"
11 #include "cc/output/compositor_frame.h" 11 #include "cc/output/compositor_frame.h"
12 #include "cc/output/output_surface.h" 12 #include "cc/output/output_surface.h"
13 #include "cc/output/texture_mailbox_deleter.h" 13 #include "cc/output/texture_mailbox_deleter.h"
14 #include "cc/quads/shared_quad_state.h" 14 #include "cc/quads/shared_quad_state.h"
15 #include "cc/quads/surface_draw_quad.h" 15 #include "cc/quads/surface_draw_quad.h"
16 #include "cc/scheduler/begin_frame_source.h" 16 #include "cc/scheduler/begin_frame_source.h"
17 #include "cc/surfaces/display_scheduler.h" 17 #include "cc/surfaces/display_scheduler.h"
18 #include "services/ui/surfaces/direct_output_surface.h" 18 #include "services/ui/surfaces/direct_output_surface.h"
19 #include "services/ui/surfaces/display_compositor.h" 19 #include "services/ui/surfaces/display_compositor.h"
20 20
21 #if defined(USE_OZONE) 21 #if defined(USE_OZONE)
22 #include "gpu/command_buffer/client/gles2_interface.h" 22 #include "gpu/command_buffer/client/gles2_interface.h"
23 #include "services/ui/surfaces/direct_output_surface_ozone.h" 23 #include "services/ui/surfaces/direct_output_surface_ozone.h"
24 #endif 24 #endif
25 25
26 namespace ui { 26 namespace ui {
27 namespace ws { 27 namespace ws {
28 28
29 ServerWindowCompositorFrameSink::ServerWindowCompositorFrameSink( 29 GpuCompositorFrameSink::GpuCompositorFrameSink(
30 scoped_refptr<DisplayCompositor> display_compositor, 30 scoped_refptr<DisplayCompositor> display_compositor,
31 const cc::FrameSinkId& frame_sink_id, 31 const cc::FrameSinkId& frame_sink_id,
32 gfx::AcceleratedWidget widget, 32 gfx::AcceleratedWidget widget,
33 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager, 33 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
34 scoped_refptr<SurfacesContextProvider> context_provider, 34 scoped_refptr<SurfacesContextProvider> context_provider,
35 cc::mojom::MojoCompositorFrameSinkRequest request, 35 cc::mojom::MojoCompositorFrameSinkRequest request,
36 cc::mojom::MojoCompositorFrameSinkClientPtr client) 36 cc::mojom::MojoCompositorFrameSinkClientPtr client)
37 : frame_sink_id_(frame_sink_id), 37 : frame_sink_id_(frame_sink_id),
38 task_runner_(base::ThreadTaskRunnerHandle::Get()), 38 task_runner_(base::ThreadTaskRunnerHandle::Get()),
39 display_compositor_(display_compositor), 39 display_compositor_(display_compositor),
40 surface_factory_(frame_sink_id_, display_compositor_->manager(), this), 40 surface_factory_(frame_sink_id_, display_compositor_->manager(), this),
41 client_(std::move(client)), 41 client_(std::move(client)),
42 binding_(this, std::move(request)) { 42 binding_(this, std::move(request)) {
43 display_compositor_->manager()->RegisterFrameSinkId(frame_sink_id_); 43 display_compositor_->manager()->RegisterFrameSinkId(frame_sink_id_);
44 display_compositor_->manager()->RegisterSurfaceFactoryClient(frame_sink_id_, 44 display_compositor_->manager()->RegisterSurfaceFactoryClient(frame_sink_id_,
45 this); 45 this);
46 46
47 if (widget != gfx::kNullAcceleratedWidget) 47 if (widget != gfx::kNullAcceleratedWidget)
48 InitDisplay(widget, gpu_memory_buffer_manager, std::move(context_provider)); 48 InitDisplay(widget, gpu_memory_buffer_manager, std::move(context_provider));
49 } 49 }
50 50
51 ServerWindowCompositorFrameSink::~ServerWindowCompositorFrameSink() { 51 GpuCompositorFrameSink::~GpuCompositorFrameSink() {
52 // SurfaceFactory's destructor will attempt to return resources which will 52 // SurfaceFactory's destructor will attempt to return resources which will
53 // call back into here and access |client_| so we should destroy 53 // call back into here and access |client_| so we should destroy
54 // |surface_factory_|'s resources early on. 54 // |surface_factory_|'s resources early on.
55 surface_factory_.DestroyAll(); 55 surface_factory_.DestroyAll();
56 display_compositor_->manager()->UnregisterSurfaceFactoryClient( 56 display_compositor_->manager()->UnregisterSurfaceFactoryClient(
57 frame_sink_id_); 57 frame_sink_id_);
58 display_compositor_->manager()->InvalidateFrameSinkId(frame_sink_id_); 58 display_compositor_->manager()->InvalidateFrameSinkId(frame_sink_id_);
59 } 59 }
60 60
61 void ServerWindowCompositorFrameSink::SetNeedsBeginFrame( 61 void GpuCompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) {
62 bool needs_begin_frame) {
63 needs_begin_frame_ = needs_begin_frame; 62 needs_begin_frame_ = needs_begin_frame;
64 UpdateNeedsBeginFramesInternal(); 63 UpdateNeedsBeginFramesInternal();
65 } 64 }
66 65
67 void ServerWindowCompositorFrameSink::SubmitCompositorFrame( 66 void GpuCompositorFrameSink::SubmitCompositorFrame(cc::CompositorFrame frame) {
68 cc::CompositorFrame frame) {
69 gfx::Size frame_size = frame.render_pass_list[0]->output_rect.size(); 67 gfx::Size frame_size = frame.render_pass_list[0]->output_rect.size();
70 // If the size of the CompostiorFrame has changed then destroy the existing 68 // If the size of the CompostiorFrame has changed then destroy the existing
71 // Surface and create a new one of the appropriate size. 69 // Surface and create a new one of the appropriate size.
72 if (!local_frame_id_.is_valid() || frame_size != last_submitted_frame_size_) { 70 if (!local_frame_id_.is_valid() || frame_size != last_submitted_frame_size_) {
73 if (local_frame_id_.is_valid()) 71 if (local_frame_id_.is_valid())
74 surface_factory_.Destroy(local_frame_id_); 72 surface_factory_.Destroy(local_frame_id_);
75 local_frame_id_ = surface_id_allocator_.GenerateId(); 73 local_frame_id_ = surface_id_allocator_.GenerateId();
76 surface_factory_.Create(local_frame_id_); 74 surface_factory_.Create(local_frame_id_);
77 if (display_) 75 if (display_)
78 display_->Resize(frame_size); 76 display_->Resize(frame_size);
79 } 77 }
80 ++ack_pending_count_; 78 ++ack_pending_count_;
81 surface_factory_.SubmitCompositorFrame( 79 surface_factory_.SubmitCompositorFrame(
82 local_frame_id_, std::move(frame), 80 local_frame_id_, std::move(frame),
83 base::Bind(&ServerWindowCompositorFrameSink::DidReceiveCompositorFrameAck, 81 base::Bind(&GpuCompositorFrameSink::DidReceiveCompositorFrameAck,
84 base::Unretained(this))); 82 base::Unretained(this)));
85 if (display_) { 83 if (display_) {
86 display_->SetLocalFrameId(local_frame_id_, 84 display_->SetLocalFrameId(local_frame_id_,
87 frame.metadata.device_scale_factor); 85 frame.metadata.device_scale_factor);
88 } 86 }
89 last_submitted_frame_size_ = frame_size; 87 last_submitted_frame_size_ = frame_size;
90 } 88 }
91 89
92 void ServerWindowCompositorFrameSink::DidReceiveCompositorFrameAck() { 90 void GpuCompositorFrameSink::DidReceiveCompositorFrameAck() {
93 if (!client_) 91 if (!client_)
94 return; 92 return;
95 client_->DidReceiveCompositorFrameAck(); 93 client_->DidReceiveCompositorFrameAck();
96 DCHECK_GT(ack_pending_count_, 0); 94 DCHECK_GT(ack_pending_count_, 0);
97 if (!surface_returned_resources_.empty()) { 95 if (!surface_returned_resources_.empty()) {
98 client_->ReclaimResources(surface_returned_resources_); 96 client_->ReclaimResources(surface_returned_resources_);
99 surface_returned_resources_.clear(); 97 surface_returned_resources_.clear();
100 } 98 }
101 ack_pending_count_--; 99 ack_pending_count_--;
102 } 100 }
103 101
104 void ServerWindowCompositorFrameSink::AddChildFrameSink( 102 void GpuCompositorFrameSink::AddChildFrameSink(
105 const cc::FrameSinkId& child_frame_sink_id) { 103 const cc::FrameSinkId& child_frame_sink_id) {
106 cc::SurfaceManager* surface_manager = display_compositor_->manager(); 104 cc::SurfaceManager* surface_manager = display_compositor_->manager();
107 surface_manager->RegisterFrameSinkHierarchy(frame_sink_id_, 105 surface_manager->RegisterFrameSinkHierarchy(frame_sink_id_,
108 child_frame_sink_id); 106 child_frame_sink_id);
109 } 107 }
110 108
111 void ServerWindowCompositorFrameSink::RemoveChildFrameSink( 109 void GpuCompositorFrameSink::RemoveChildFrameSink(
112 const cc::FrameSinkId& child_frame_sink_id) { 110 const cc::FrameSinkId& child_frame_sink_id) {
113 cc::SurfaceManager* surface_manager = display_compositor_->manager(); 111 cc::SurfaceManager* surface_manager = display_compositor_->manager();
114 surface_manager->UnregisterFrameSinkHierarchy(frame_sink_id_, 112 surface_manager->UnregisterFrameSinkHierarchy(frame_sink_id_,
115 child_frame_sink_id); 113 child_frame_sink_id);
116 } 114 }
117 115
118 void ServerWindowCompositorFrameSink::InitDisplay( 116 void GpuCompositorFrameSink::InitDisplay(
119 gfx::AcceleratedWidget widget, 117 gfx::AcceleratedWidget widget,
120 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager, 118 gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
121 scoped_refptr<SurfacesContextProvider> context_provider) { 119 scoped_refptr<SurfacesContextProvider> context_provider) {
122 // TODO(rjkroege): If there is something better to do than CHECK, add it. 120 // TODO(rjkroege): If there is something better to do than CHECK, add it.
123 CHECK(context_provider->BindToCurrentThread()); 121 CHECK(context_provider->BindToCurrentThread());
124 122
125 std::unique_ptr<cc::SyntheticBeginFrameSource> synthetic_begin_frame_source( 123 std::unique_ptr<cc::SyntheticBeginFrameSource> synthetic_begin_frame_source(
126 new cc::DelayBasedBeginFrameSource( 124 new cc::DelayBasedBeginFrameSource(
127 base::MakeUnique<cc::DelayBasedTimeSource>(task_runner_.get()))); 125 base::MakeUnique<cc::DelayBasedTimeSource>(task_runner_.get())));
128 126
(...skipping 22 matching lines...) Expand all
151 display_.reset(new cc::Display( 149 display_.reset(new cc::Display(
152 nullptr /* bitmap_manager */, gpu_memory_buffer_manager, 150 nullptr /* bitmap_manager */, gpu_memory_buffer_manager,
153 cc::RendererSettings(), frame_sink_id_, 151 cc::RendererSettings(), frame_sink_id_,
154 std::move(synthetic_begin_frame_source), 152 std::move(synthetic_begin_frame_source),
155 std::move(display_output_surface), std::move(scheduler), 153 std::move(display_output_surface), std::move(scheduler),
156 base::MakeUnique<cc::TextureMailboxDeleter>(task_runner_.get()))); 154 base::MakeUnique<cc::TextureMailboxDeleter>(task_runner_.get())));
157 display_->Initialize(this, display_compositor_->manager()); 155 display_->Initialize(this, display_compositor_->manager());
158 display_->SetVisible(true); 156 display_->SetVisible(true);
159 } 157 }
160 158
161 void ServerWindowCompositorFrameSink::DisplayOutputSurfaceLost() {} 159 void GpuCompositorFrameSink::DisplayOutputSurfaceLost() {}
162 160
163 void ServerWindowCompositorFrameSink::DisplayWillDrawAndSwap( 161 void GpuCompositorFrameSink::DisplayWillDrawAndSwap(
164 bool will_draw_and_swap, 162 bool will_draw_and_swap,
165 const cc::RenderPassList& render_passes) {} 163 const cc::RenderPassList& render_passes) {}
166 164
167 void ServerWindowCompositorFrameSink::DisplayDidDrawAndSwap() {} 165 void GpuCompositorFrameSink::DisplayDidDrawAndSwap() {}
168 166
169 void ServerWindowCompositorFrameSink::ReturnResources( 167 void GpuCompositorFrameSink::ReturnResources(
170 const cc::ReturnedResourceArray& resources) { 168 const cc::ReturnedResourceArray& resources) {
171 if (resources.empty()) 169 if (resources.empty())
172 return; 170 return;
173 171
174 if (!ack_pending_count_ && client_) { 172 if (!ack_pending_count_ && client_) {
175 client_->ReclaimResources(resources); 173 client_->ReclaimResources(resources);
176 return; 174 return;
177 } 175 }
178 176
179 std::copy(resources.begin(), resources.end(), 177 std::copy(resources.begin(), resources.end(),
180 std::back_inserter(surface_returned_resources_)); 178 std::back_inserter(surface_returned_resources_));
181 } 179 }
182 180
183 void ServerWindowCompositorFrameSink::SetBeginFrameSource( 181 void GpuCompositorFrameSink::SetBeginFrameSource(
184 cc::BeginFrameSource* begin_frame_source) { 182 cc::BeginFrameSource* begin_frame_source) {
185 // TODO(tansell): Implement this. 183 // TODO(tansell): Implement this.
186 if (begin_frame_source_ && added_frame_observer_) { 184 if (begin_frame_source_ && added_frame_observer_) {
187 begin_frame_source_->RemoveObserver(this); 185 begin_frame_source_->RemoveObserver(this);
188 added_frame_observer_ = false; 186 added_frame_observer_ = false;
189 } 187 }
190 begin_frame_source_ = begin_frame_source; 188 begin_frame_source_ = begin_frame_source;
191 UpdateNeedsBeginFramesInternal(); 189 UpdateNeedsBeginFramesInternal();
192 } 190 }
193 191
194 void ServerWindowCompositorFrameSink::OnBeginFrame( 192 void GpuCompositorFrameSink::OnBeginFrame(const cc::BeginFrameArgs& args) {
195 const cc::BeginFrameArgs& args) {
196 UpdateNeedsBeginFramesInternal(); 193 UpdateNeedsBeginFramesInternal();
197 last_begin_frame_args_ = args; 194 last_begin_frame_args_ = args;
198 if (client_) 195 if (client_)
199 client_->OnBeginFrame(args); 196 client_->OnBeginFrame(args);
200 } 197 }
201 198
202 const cc::BeginFrameArgs& 199 const cc::BeginFrameArgs& GpuCompositorFrameSink::LastUsedBeginFrameArgs()
203 ServerWindowCompositorFrameSink::LastUsedBeginFrameArgs() const { 200 const {
204 return last_begin_frame_args_; 201 return last_begin_frame_args_;
205 } 202 }
206 203
207 void ServerWindowCompositorFrameSink::OnBeginFrameSourcePausedChanged( 204 void GpuCompositorFrameSink::OnBeginFrameSourcePausedChanged(bool paused) {}
208 bool paused) {}
209 205
210 void ServerWindowCompositorFrameSink::UpdateNeedsBeginFramesInternal() { 206 void GpuCompositorFrameSink::UpdateNeedsBeginFramesInternal() {
211 if (!begin_frame_source_) 207 if (!begin_frame_source_)
212 return; 208 return;
213 209
214 if (needs_begin_frame_ == added_frame_observer_) 210 if (needs_begin_frame_ == added_frame_observer_)
215 return; 211 return;
216 212
217 added_frame_observer_ = needs_begin_frame_; 213 added_frame_observer_ = needs_begin_frame_;
218 if (needs_begin_frame_) 214 if (needs_begin_frame_)
219 begin_frame_source_->AddObserver(this); 215 begin_frame_source_->AddObserver(this);
220 else 216 else
221 begin_frame_source_->RemoveObserver(this); 217 begin_frame_source_->RemoveObserver(this);
222 } 218 }
223 219
224 } // namespace ws 220 } // namespace ws
225 } // namespace ui 221 } // namespace ui
OLDNEW
« no previous file with comments | « services/ui/ws/gpu_compositor_frame_sink.h ('k') | services/ui/ws/server_window.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698