ThreadChecker instead of SequenceChecker in classes that use TLS.

chrome/service/cloud_print/cloud_print_token_store.cc

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 =

[gab, 2017/06/02 18:59:05]

Note: this probably shouldn't use TLS. Given access is sequenced (per the
SequenceChecker). From the API "this is only used with
CloudPrintProxyCoreThread". As such this could be a global variable (no TLS) to
which thread-safe access is enforced by a SequenceChecker.

I'll let OWNERS address this in a follow-up.

     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;
 }