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

Unified Diff: chrome/service/cloud_print/cloud_print_token_store.cc

Issue 2916303002: ThreadChecker instead of SequenceChecker in classes that use TLS. (Closed)
Patch Set: fix compile 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 | « chrome/service/cloud_print/cloud_print_token_store.h ('k') | content/child/fileapi/webfilesystem_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/service/cloud_print/cloud_print_token_store.cc
diff --git a/chrome/service/cloud_print/cloud_print_token_store.cc b/chrome/service/cloud_print/cloud_print_token_store.cc
index 84006eed246fdb07552a747da07953d2d671ea89..bebf5958784b2b25ef605efa8d66e1df0e719a9e 100644
--- a/chrome/service/cloud_print/cloud_print_token_store.cc
+++ b/chrome/service/cloud_print/cloud_print_token_store.cc
@@ -15,6 +15,7 @@ static base::LazyInstance<
base::ThreadLocalPointer<CloudPrintTokenStore>>::DestructorAtExit lazy_tls =
LAZY_INSTANCE_INITIALIZER;
+// static
CloudPrintTokenStore* CloudPrintTokenStore::current() {
return lazy_tls.Pointer()->Get();
}
@@ -24,12 +25,12 @@ CloudPrintTokenStore::CloudPrintTokenStore() {
}
CloudPrintTokenStore::~CloudPrintTokenStore() {
- DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
lazy_tls.Pointer()->Set(NULL);
}
void CloudPrintTokenStore::SetToken(const std::string& token) {
- DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
token_ = token;
}
« no previous file with comments | « chrome/service/cloud_print/cloud_print_token_store.h ('k') | content/child/fileapi/webfilesystem_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698