Replace deprecated base::NonThreadSafe in chrome/browser/profiles.

chrome/browser/profiles/profile_manager.h

Index: chrome/browser/profiles/profile_manager.h
diff --git a/chrome/browser/profiles/profile_manager.h b/chrome/browser/profiles/profile_manager.h
index 385f9c360fe2b718b6a9556d811bb8f0a4532b2e..8371ce9803cf261e7fadfd6b081973b9272746ac 100644
--- a/chrome/browser/profiles/profile_manager.h
+++ b/chrome/browser/profiles/profile_manager.h
@@ -19,7 +19,7 @@
 #include "base/macros.h"
 #include "base/memory/linked_ptr.h"
 #include "base/message_loop/message_loop.h"
-#include "base/threading/non_thread_safe.h"
+#include "base/sequence_checker.h"
 #include "build/build_config.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_metrics.h"
@@ -32,8 +32,7 @@
 class ProfileAttributesStorage;
 class ProfileInfoCache;
 
-class ProfileManager : public base::NonThreadSafe,
-                       public content::NotificationObserver,
+class ProfileManager : public content::NotificationObserver,
                        public Profile::Delegate {
  public:
   typedef base::Callback<void(Profile*, Profile::CreateStatus)> CreateCallback;
@@ -427,6 +426,8 @@ class ProfileManager : public base::NonThreadSafe,
   std::vector<Profile*> active_profiles_;
   bool closing_all_browsers_;
 
+  SEQUENCE_CHECKER(sequence_checker_);

[Bernhard Bauer, 2017/05/31 13:19:52]

I would get rid of this completely in favor of
DCHECK_CURRENTLY_ON(BrowserThread::UI) (which is already used in some places
anyway).

[gab, 2017/05/31 16:47:24]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(ProfileManager);
 };