cros: Fix flaky owner detection

components/user_manager/user_manager_base.h

Index: components/user_manager/user_manager_base.h
diff --git a/components/user_manager/user_manager_base.h b/components/user_manager/user_manager_base.h
index dbf8ebe8a75487dedc94a586df7fdeaa76297562..70776a72077441725ab4f296962a28b52f8453fa 100644
--- a/components/user_manager/user_manager_base.h
+++ b/components/user_manager/user_manager_base.h
@@ -251,8 +251,6 @@ class USER_MANAGER_EXPORT UserManagerBase : public UserManager {
 
   // Getters/setters for private members.
 
-  virtual void SetCurrentUserIsOwner(bool is_current_user_owner);
-
   virtual bool GetEphemeralUsersEnabled() const;
   virtual void SetEphemeralUsersEnabled(bool enabled);
 
@@ -345,11 +343,6 @@ class USER_MANAGER_EXPORT UserManagerBase : public UserManager {
   // Indicates stage of loading user from prefs.
   UserLoadStage user_loading_stage_ = STAGE_NOT_LOADED;
 
-  // Cached flag of whether currently logged-in user is owner or not.
-  // May be accessed on different threads, requires locking.
-  bool is_current_user_owner_ = false;
-  mutable base::Lock is_current_user_owner_lock_;

[xiyuan, 2017/04/03 19:02:56]

The comment and the lock is obsolete.

Both IsCurrentUserOwner and SetCurrentUserIsOwner has 
  DCHECK(task_runner_->RunsTasksOnCurrentThread());

Think that means all calls must happen on the same thread now.

-
   // Cached flag of whether the currently logged-in user existed before this
   // login.
   bool is_current_user_new_ = false;