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

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

Issue 2886453004: Adding ThreadChecker validation to d'tors for client classes (Closed)
Patch Set: Created 3 years, 7 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 | « remoting/client/display/fake_canvas.cc ('k') | remoting/client/display/gl_cursor_feedback.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 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_cursor.h" 5 #include "remoting/client/display/gl_cursor.h"
6 6
7 #include "remoting/base/util.h" 7 #include "remoting/base/util.h"
8 #include "remoting/client/display/gl_canvas.h" 8 #include "remoting/client/display/gl_canvas.h"
9 #include "remoting/client/display/gl_math.h" 9 #include "remoting/client/display/gl_math.h"
10 #include "remoting/client/display/gl_render_layer.h" 10 #include "remoting/client/display/gl_render_layer.h"
11 #include "remoting/client/display/gl_texture_ids.h" 11 #include "remoting/client/display/gl_texture_ids.h"
12 #include "remoting/proto/control.pb.h" 12 #include "remoting/proto/control.pb.h"
13 #include "third_party/libyuv/include/libyuv/convert_argb.h" 13 #include "third_party/libyuv/include/libyuv/convert_argb.h"
14 14
15 namespace remoting { 15 namespace remoting {
16 16
17 namespace { 17 namespace {
18 const int kDefaultCursorDataSize = 32 * 32 * GlRenderLayer::kBytesPerPixel; 18 const int kDefaultCursorDataSize = 32 * 32 * GlRenderLayer::kBytesPerPixel;
19 } // namespace 19 } // namespace
20 20
21 GlCursor::GlCursor() : weak_factory_(this) {} 21 GlCursor::GlCursor() : weak_factory_(this) {}
22 22
23 GlCursor::~GlCursor() {} 23 GlCursor::~GlCursor() {
24 DCHECK(thread_checker_.CalledOnValidThread());
25 }
24 26
25 void GlCursor::SetCursorShape(const protocol::CursorShapeInfo& cursor_shape) { 27 void GlCursor::SetCursorShape(const protocol::CursorShapeInfo& cursor_shape) {
26 int data_size = cursor_shape.width() * cursor_shape.height() * 28 int data_size = cursor_shape.width() * cursor_shape.height() *
27 GlRenderLayer::kBytesPerPixel; 29 GlRenderLayer::kBytesPerPixel;
28 if (current_cursor_data_size_ < data_size) { 30 if (current_cursor_data_size_ < data_size) {
29 current_cursor_data_size_ = 31 current_cursor_data_size_ =
30 kDefaultCursorDataSize > data_size ? kDefaultCursorDataSize : data_size; 32 kDefaultCursorDataSize > data_size ? kDefaultCursorDataSize : data_size;
31 current_cursor_data_.reset(new uint8_t[current_cursor_data_size_]); 33 current_cursor_data_.reset(new uint8_t[current_cursor_data_size_]);
32 } 34 }
33 int stride = cursor_shape.width() * GlRenderLayer::kBytesPerPixel; 35 int stride = cursor_shape.width() * GlRenderLayer::kBytesPerPixel;
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 } 105 }
104 } 106 }
105 } 107 }
106 108
107 base::WeakPtr<Drawable> GlCursor::GetWeakPtr() { 109 base::WeakPtr<Drawable> GlCursor::GetWeakPtr() {
108 DCHECK(thread_checker_.CalledOnValidThread()); 110 DCHECK(thread_checker_.CalledOnValidThread());
109 return weak_factory_.GetWeakPtr(); 111 return weak_factory_.GetWeakPtr();
110 } 112 }
111 113
112 } // namespace remoting 114 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/client/display/fake_canvas.cc ('k') | remoting/client/display/gl_cursor_feedback.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698