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

Unified Diff: remoting/ios/display/gl_demo_screen.mm

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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/client/software_video_renderer.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/ios/display/gl_demo_screen.mm
diff --git a/remoting/ios/display/gl_demo_screen.mm b/remoting/ios/display/gl_demo_screen.mm
index c293fe5cacabf1eae067c34957275cfaf1d15dec..738d1a67c048d731938d6d8a0f910c2ba9b500f7 100644
--- a/remoting/ios/display/gl_demo_screen.mm
+++ b/remoting/ios/display/gl_demo_screen.mm
@@ -35,7 +35,9 @@ const GLchar* a_position = "a_position";
// integration. This will draw an expanding checkerboard pattern to the screen.
GlDemoScreen::GlDemoScreen() : weak_factory_(this) {}
-GlDemoScreen::~GlDemoScreen() {}
+GlDemoScreen::~GlDemoScreen() {
+ DCHECK(thread_checker_.CalledOnValidThread());
+}
void GlDemoScreen::SetCanvas(base::WeakPtr<Canvas> canvas) {
canvas_ = canvas;
« no previous file with comments | « remoting/client/software_video_renderer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698