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

Side by Side Diff: remoting/client/display/gl_renderer.cc

Issue 2591363002: Adding drawable to CRD andorid and iOS gl rendering pipeline. (Closed)
Patch Set: Cleanup and adding interfaces based on review feedback.: Created 3 years, 11 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "remoting/client/display/gl_renderer.h" 5 #include "remoting/client/display/gl_renderer.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/threading/thread_task_runner_handle.h" 9 #include "base/threading/thread_task_runner_handle.h"
10 #include "remoting/client/display/drawable.h"
10 #include "remoting/client/display/gl_canvas.h" 11 #include "remoting/client/display/gl_canvas.h"
11 #include "remoting/client/display/gl_math.h" 12 #include "remoting/client/display/gl_math.h"
12 #include "remoting/client/display/gl_renderer_delegate.h" 13 #include "remoting/client/display/gl_renderer_delegate.h"
13 #include "remoting/client/display/sys_opengl.h" 14 #include "remoting/client/display/sys_opengl.h"
14 #include "third_party/webrtc/modules/desktop_capture/desktop_frame.h" 15 #include "third_party/webrtc/modules/desktop_capture/desktop_frame.h"
15 16
16 namespace remoting { 17 namespace remoting {
17 18
18 GlRenderer::GlRenderer() : 19 GlRenderer::GlRenderer() :
19 weak_factory_(this) { 20 weak_factory_(this) {
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 pending_done_callbacks_.push(done); 84 pending_done_callbacks_.push(done);
84 RequestRender(); 85 RequestRender();
85 } 86 }
86 87
87 void GlRenderer::OnCursorShapeChanged(const protocol::CursorShapeInfo& shape) { 88 void GlRenderer::OnCursorShapeChanged(const protocol::CursorShapeInfo& shape) {
88 DCHECK(thread_checker_.CalledOnValidThread()); 89 DCHECK(thread_checker_.CalledOnValidThread());
89 cursor_.SetCursorShape(shape); 90 cursor_.SetCursorShape(shape);
90 RequestRender(); 91 RequestRender();
91 } 92 }
92 93
93 void GlRenderer::OnSurfaceCreated(int gl_version) { 94 void GlRenderer::OnSurfaceCreated(std::unique_ptr<Canvas> canvas) {
94 DCHECK(thread_checker_.CalledOnValidThread()); 95 DCHECK(thread_checker_.CalledOnValidThread());
95 #ifndef NDEBUG 96 canvas_ = std::move(canvas);
96 // Set the background clear color to bright green for debugging purposes. 97 for (auto& drawable : drawables_) {
97 glClearColor(0.0f, 1.0f, 0.0f, 1.0f); 98 drawable->SetCanvas(canvas_.get());
98 #else 99 }
99 // Set the background clear color to black.
100 glClearColor(0.0f, 0.0f, 0.0f, 1.0f);
101 #endif
102 canvas_.reset(new GlCanvas(gl_version));
103 desktop_.SetCanvas(canvas_.get());
104 cursor_.SetCanvas(canvas_.get());
105 cursor_feedback_.SetCanvas(canvas_.get());
106 } 100 }
107 101
108 void GlRenderer::OnSurfaceChanged(int view_width, int view_height) { 102 void GlRenderer::OnSurfaceChanged(int view_width, int view_height) {
109 DCHECK(thread_checker_.CalledOnValidThread()); 103 DCHECK(thread_checker_.CalledOnValidThread());
110 if (!canvas_) { 104 if (!canvas_) {
111 LOG(WARNING) << "Trying to set the view size when the canvas is not ready."; 105 LOG(WARNING) << "Trying to set the view size when the canvas is not ready.";
112 return; 106 return;
113 } 107 }
114 canvas_->SetViewSize(view_width, view_height); 108 canvas_->SetViewSize(view_width, view_height);
115 RequestRender(); 109 RequestRender();
(...skipping 14 matching lines...) Expand all
130 void GlRenderer::RequestRender() { 124 void GlRenderer::RequestRender() {
131 DCHECK(thread_checker_.CalledOnValidThread()); 125 DCHECK(thread_checker_.CalledOnValidThread());
132 if (render_scheduled_) { 126 if (render_scheduled_) {
133 return; 127 return;
134 } 128 }
135 base::ThreadTaskRunnerHandle::Get()->PostTask( 129 base::ThreadTaskRunnerHandle::Get()->PostTask(
136 FROM_HERE, base::Bind(&GlRenderer::OnRender, weak_ptr_)); 130 FROM_HERE, base::Bind(&GlRenderer::OnRender, weak_ptr_));
137 render_scheduled_ = true; 131 render_scheduled_ = true;
138 } 132 }
139 133
134 void GlRenderer::AddDrawable(base::WeakPtr<Drawable> drawable) {
135 drawable->SetCanvas(canvas_.get());
136 drawables_.push_back(drawable);
137 std::sort(drawables_.begin(), drawables_.end(), drawable::ZOrderComparator);
138 }
139
140 void GlRenderer::OnRender() { 140 void GlRenderer::OnRender() {
141 DCHECK(thread_checker_.CalledOnValidThread()); 141 DCHECK(thread_checker_.CalledOnValidThread());
142 render_scheduled_ = false; 142 render_scheduled_ = false;
143 if (!delegate_ || !delegate_->CanRenderFrame()) { 143 if (!delegate_ || !delegate_->CanRenderFrame()) {
144 return; 144 return;
145 } 145 }
146 146
147 if (canvas_) { 147 if (canvas_) {
148 glClear(GL_COLOR_BUFFER_BIT); 148 canvas_->Clear();
149 149 // Draw each drawable in order.
150 // Layers will be drawn from bottom to top. 150 for (auto& drawable : drawables_) {
151 desktop_.Draw(); 151 if (drawable->Draw()) {
152 152 RequestRender();
153 // |cursor_feedback_| should be drawn before |cursor_| so that the cursor 153 }
154 // won't be covered by the feedback animation.
155 if (cursor_feedback_.Draw()) {
156 RequestRender();
157 } 154 }
158
159 cursor_.Draw();
160 } 155 }
161 156
162 delegate_->OnFrameRendered(); 157 delegate_->OnFrameRendered();
163 158
164 while (!pending_done_callbacks_.empty()) { 159 while (!pending_done_callbacks_.empty()) {
165 pending_done_callbacks_.front().Run(); 160 pending_done_callbacks_.front().Run();
166 pending_done_callbacks_.pop(); 161 pending_done_callbacks_.pop();
167 } 162 }
168 } 163 }
169 164
165 std::unique_ptr<GlRenderer> GlRenderer::CreateGlRendererWithDesktop() {
166 std::unique_ptr<GlRenderer> renderer(new GlRenderer());
167 renderer->AddDrawable(renderer->desktop_.GetWeakPtr());
168 renderer->AddDrawable(renderer->cursor_.GetWeakPtr());
169 renderer->AddDrawable(renderer->cursor_feedback_.GetWeakPtr());
170 return renderer;
171 }
172
170 } // namespace remoting 173 } // namespace remoting
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698