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

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

Issue 2468633002: Replaced cc::Display::SetSurfaceId() with SetLocalFrameId() (Closed)
Patch Set: rebase and minor bug fixes 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
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/server_window_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"
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 surface_factory_.Create(local_frame_id_); 76 surface_factory_.Create(local_frame_id_);
77 if (display_) 77 if (display_)
78 display_->Resize(frame_size); 78 display_->Resize(frame_size);
79 } 79 }
80 ++ack_pending_count_; 80 ++ack_pending_count_;
81 surface_factory_.SubmitCompositorFrame( 81 surface_factory_.SubmitCompositorFrame(
82 local_frame_id_, std::move(frame), 82 local_frame_id_, std::move(frame),
83 base::Bind(&ServerWindowCompositorFrameSink::DidReceiveCompositorFrameAck, 83 base::Bind(&ServerWindowCompositorFrameSink::DidReceiveCompositorFrameAck,
84 base::Unretained(this))); 84 base::Unretained(this)));
85 if (display_) { 85 if (display_) {
86 display_->SetSurfaceId(cc::SurfaceId(frame_sink_id_, local_frame_id_), 86 display_->SetLocalFrameId(local_frame_id_,
87 frame.metadata.device_scale_factor); 87 frame.metadata.device_scale_factor);
88 } 88 }
89 last_submitted_frame_size_ = frame_size; 89 last_submitted_frame_size_ = frame_size;
90 } 90 }
91 91
92 void ServerWindowCompositorFrameSink::DidReceiveCompositorFrameAck() { 92 void ServerWindowCompositorFrameSink::DidReceiveCompositorFrameAck() {
93 if (!client_) 93 if (!client_)
94 return; 94 return;
95 client_->DidReceiveCompositorFrameAck(); 95 client_->DidReceiveCompositorFrameAck();
96 DCHECK_GT(ack_pending_count_, 0); 96 DCHECK_GT(ack_pending_count_, 0);
97 if (!surface_returned_resources_.empty()) { 97 if (!surface_returned_resources_.empty()) {
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 int max_frames_pending = 143 int max_frames_pending =
144 display_output_surface->capabilities().max_frames_pending; 144 display_output_surface->capabilities().max_frames_pending;
145 DCHECK_GT(max_frames_pending, 0); 145 DCHECK_GT(max_frames_pending, 0);
146 146
147 std::unique_ptr<cc::DisplayScheduler> scheduler( 147 std::unique_ptr<cc::DisplayScheduler> scheduler(
148 new cc::DisplayScheduler(synthetic_begin_frame_source.get(), 148 new cc::DisplayScheduler(synthetic_begin_frame_source.get(),
149 task_runner_.get(), max_frames_pending)); 149 task_runner_.get(), max_frames_pending));
150 150
151 display_.reset(new cc::Display( 151 display_.reset(new cc::Display(
152 nullptr /* bitmap_manager */, gpu_memory_buffer_manager, 152 nullptr /* bitmap_manager */, gpu_memory_buffer_manager,
153 cc::RendererSettings(), std::move(synthetic_begin_frame_source), 153 cc::RendererSettings(), frame_sink_id_,
154 std::move(synthetic_begin_frame_source),
154 std::move(display_output_surface), std::move(scheduler), 155 std::move(display_output_surface), std::move(scheduler),
155 base::MakeUnique<cc::TextureMailboxDeleter>(task_runner_.get()))); 156 base::MakeUnique<cc::TextureMailboxDeleter>(task_runner_.get())));
156 display_->Initialize(this, display_compositor_->manager(), frame_sink_id_); 157 display_->Initialize(this, display_compositor_->manager());
157 display_->SetVisible(true); 158 display_->SetVisible(true);
158 } 159 }
159 160
160 void ServerWindowCompositorFrameSink::DisplayOutputSurfaceLost() {} 161 void ServerWindowCompositorFrameSink::DisplayOutputSurfaceLost() {}
161 162
162 void ServerWindowCompositorFrameSink::DisplayWillDrawAndSwap( 163 void ServerWindowCompositorFrameSink::DisplayWillDrawAndSwap(
163 bool will_draw_and_swap, 164 bool will_draw_and_swap,
164 const cc::RenderPassList& render_passes) {} 165 const cc::RenderPassList& render_passes) {}
165 166
166 void ServerWindowCompositorFrameSink::DisplayDidDrawAndSwap() {} 167 void ServerWindowCompositorFrameSink::DisplayDidDrawAndSwap() {}
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
215 216
216 added_frame_observer_ = needs_begin_frame_; 217 added_frame_observer_ = needs_begin_frame_;
217 if (needs_begin_frame_) 218 if (needs_begin_frame_)
218 begin_frame_source_->AddObserver(this); 219 begin_frame_source_->AddObserver(this);
219 else 220 else
220 begin_frame_source_->RemoveObserver(this); 221 begin_frame_source_->RemoveObserver(this);
221 } 222 }
222 223
223 } // namespace ws 224 } // namespace ws
224 } // namespace ui 225 } // namespace ui
OLDNEW
« no previous file with comments | « content/renderer/android/synchronous_compositor_frame_sink.cc ('k') | ui/compositor/test/in_process_context_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698