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

Unified Diff: net/dns/dns_config_service.h

Issue 2910473005: Deprecate NonThreadSafe in net/ in favor of SequenceChecker/ThreadChecker. (Closed)
Patch Set: another compile nit 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: net/dns/dns_config_service.h
diff --git a/net/dns/dns_config_service.h b/net/dns/dns_config_service.h
index e0fd54a6f3bd490c0b3ae85e133dc50bec5fe8a5..2efc1f201b40e6911ff841b83606c0bd804c4e19 100644
--- a/net/dns/dns_config_service.h
+++ b/net/dns/dns_config_service.h
@@ -12,7 +12,7 @@
#include "base/macros.h"
#include "base/memory/scoped_vector.h"
-#include "base/threading/non_thread_safe.h"
+#include "base/sequence_checker.h"
#include "base/time/time.h"
#include "base/timer/timer.h"
// Needed on shared build with MSVS2010 to avoid multiple definitions of
@@ -92,8 +92,7 @@ struct NET_EXPORT_PRIVATE DnsConfig {
// Service for reading system DNS settings, on demand or when signalled by
// internal watchers and NetworkChangeNotifier.
-class NET_EXPORT_PRIVATE DnsConfigService
- : NON_EXPORTED_BASE(public base::NonThreadSafe) {
+class NET_EXPORT_PRIVATE DnsConfigService {
public:
// Callback interface for the client, called on the same thread as
// ReadConfig() and WatchConfig().
@@ -174,6 +173,8 @@ class NET_EXPORT_PRIVATE DnsConfigService
// Started in Invalidate*, cleared in On*Read.
base::OneShotTimer timer_;
+ SEQUENCE_CHECKER(sequence_checker_);
+
DISALLOW_COPY_AND_ASSIGN(DnsConfigService);
};

Powered by Google App Engine
This is Rietveld 408576698