Fixing crash upon window manager creation

chrome/browser/ui/ash/multi_user/multi_user_window_manager_chromeos.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_ASH_MULTI_USER_MULTI_USER_WINDOW_MANAGER_CHROMEOS_H_
 #define CHROME_BROWSER_UI_ASH_MULTI_USER_MULTI_USER_WINDOW_MANAGER_CHROMEOS_H_
 
 #include <map>
 #include <string>
 
@@ skipping 54 matching lines @@
   enum AnimationSpeed {
     ANIMATION_SPEED_NORMAL,   // The normal animation speed.
     ANIMATION_SPEED_FAST,     // Unit test speed which test animations.
     ANIMATION_SPEED_DISABLED  // Unit tests which do not require animations.
   };
 
   // Create the manager and use |active_user_id| as the active user.
   explicit MultiUserWindowManagerChromeOS(const std::string& active_user_id);
   ~MultiUserWindowManagerChromeOS() override;
 
+  // Initializes the manager after its creation. Should only be called once.
+  void Init();
+
   // MultiUserWindowManager overrides:
   void SetWindowOwner(
       aura::Window* window, const std::string& user_id) override;
   const std::string& GetWindowOwner(
       aura::Window* window) const override;
   void ShowWindowForUser(
       aura::Window* window, const std::string& user_id) override;
   bool AreWindowsSharedAmongUsers() const override;
   void GetOwnersOfVisibleWindows(
       std::set<std::string>* user_ids) const override;
@@ skipping 147 matching lines @@
 
   // An observer list to be notified upon window owner changes.
   ObserverList<Observer> observers_;
 
   // A map which remembers for owned transient windows their own visibility.
   TransientWindowToVisibility transient_window_to_visibility_;
 
   // The currently selected active user. It is used to find the proper
   // visibility state in various cases. The state is stored here instead of
   // being read from the user manager to be in sync while a switch occurs.
   std::string current_user_id_;

[stevenjb, 2015/02/03 01:31:58]

This should be able to be const, which will help ensure that the test in Init()
is correct.

[Mr4D (OOO till 08-26), 2015/02/03 02:06:05]

The current user can be changed (ActiveUserChanged(..)).

 
   // The blocker which controls the desktop notification visibility based on the
   // current multi-user status.
   scoped_ptr<MultiUserNotificationBlockerChromeOS> notification_blocker_;
 
   // The notification registrar to track the creation of browser windows.
   content::NotificationRegistrar registrar_;
 
   // Suppress changes to the visibility flag while we are changing it ourselves.
   bool suppress_visibility_changes_;
 
   // Caching the current multi profile mode since the detection which mode is
   // used is quite expensive.
   static MultiProfileMode multi_user_mode_;
 
   // The speed which is used to perform any animations.
   AnimationSpeed animation_speed_;
 
   // The animation between users.
   scoped_ptr<UserSwichAnimatorChromeOS> animation_;
 
   DISALLOW_COPY_AND_ASSIGN(MultiUserWindowManagerChromeOS);
 };
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_UI_ASH_MULTI_USER_MULTI_USER_WINDOW_MANAGER_CHROMEOS_H_