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

Unified Diff: remoting/host/security_key/security_key_socket.cc

Issue 2888733002: Adding ThreadChecker validation to d'tors for host 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
Index: remoting/host/security_key/security_key_socket.cc
diff --git a/remoting/host/security_key/security_key_socket.cc b/remoting/host/security_key/security_key_socket.cc
index 569e53cf12e4d9430e7fdfd8134d936ef5a6613e..bac15d6aa2a7366a124dcbd8e22cbcece4e4651e 100644
--- a/remoting/host/security_key/security_key_socket.cc
+++ b/remoting/host/security_key/security_key_socket.cc
@@ -35,7 +35,9 @@ SecurityKeySocket::SecurityKeySocket(std::unique_ptr<net::StreamSocket> socket,
timer_->Start(FROM_HERE, timeout, timeout_callback);
}
-SecurityKeySocket::~SecurityKeySocket() {}
+SecurityKeySocket::~SecurityKeySocket() {
+ DCHECK(thread_checker_.CalledOnValidThread());
+}
bool SecurityKeySocket::GetAndClearRequestData(std::string* data_out) {
DCHECK(thread_checker_.CalledOnValidThread());
« no previous file with comments | « remoting/host/security_key/security_key_message_handler.cc ('k') | remoting/host/win/elevated_native_messaging_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698