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

Unified Diff: remoting/host/policy_watcher.cc

Issue 2911893003: Deprecate NonThreadSafe in remoting in favor of SequenceChecker. (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/host/policy_watcher.h ('k') | remoting/host/setup/win/auth_code_getter.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/policy_watcher.cc
diff --git a/remoting/host/policy_watcher.cc b/remoting/host/policy_watcher.cc
index 2b9a60a429cddd20a81b0f9b734f6fecbfc95854..b80788638365b371dddac1edc3bf306d67a09c21 100644
--- a/remoting/host/policy_watcher.cc
+++ b/remoting/host/policy_watcher.cc
@@ -141,7 +141,7 @@ bool VerifyWellformedness(const base::DictionaryValue& changed_policies) {
void PolicyWatcher::StartWatching(
const PolicyUpdatedCallback& policy_updated_callback,
const PolicyErrorCallback& policy_error_callback) {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
DCHECK(!policy_updated_callback.is_null());
DCHECK(!policy_error_callback.is_null());
DCHECK(policy_updated_callback_.is_null());
@@ -207,6 +207,7 @@ PolicyWatcher::PolicyWatcher(
}
PolicyWatcher::~PolicyWatcher() {
+ DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
// Stop observing |policy_service_| if StartWatching() has been called.
if (!policy_updated_callback_.is_null()) {
policy_service_->RemoveObserver(policy::POLICY_DOMAIN_CHROME, this);
« no previous file with comments | « remoting/host/policy_watcher.h ('k') | remoting/host/setup/win/auth_code_getter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698