Magnifier: Prevent useless operation in enabling/disabling magnifier.

chrome/browser/chromeos/accessibility/magnification_manager.h

 // Copyright (c) 2012 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_CHROMEOS_ACCESSIBILITY_MAGNIFICATION_MANAGER_H_
 #define CHROME_BROWSER_CHROMEOS_ACCESSIBILITY_MAGNIFICATION_MANAGER_H_
 
 #include "chrome/browser/chromeos/accessibility/accessibility_util.h"
 
 namespace chromeos {
 
+class MagnificationObserver {
+ public:
+  virtual void OnMagnifierTypeChanged(ash::MagnifierType new_type) = 0;
+};
+
 // MagnificationManager controls the full screen magnifier from chrome-browser
 // side (not ash side).
 //
 // MagnificationManager does:
 //   - Watch logged-in. Changes the behavior between the login screen and user
 //     desktop.
 //   - Watch change of the pref. When the pref changes, the setting of the
 //     magnifier will interlock with it.
 
 class MagnificationManager {
+ protected:
+  MagnificationManager() {}
+  virtual ~MagnificationManager() {}

[stevenjb, 2012/12/04 18:06:26]

protected section needs to follow public. Only the destructor really needs to be
protected, and since the constructor does nothing it can be omitted.

+
  public:
   // Creates an instance of MagnificationManager. This should be called once,
+  // Returns the existing instance. If there is no instance, creates one.
   // because only one instance should exist at the same time.
-  static MagnificationManager* CreateInstance();
+  static void Initialize();
+  // Deletes the existing instance of MagnificationManager.
+  static void Shutdown();
   // Returns the existing instance. If there is no instance, returns NULL.
-  static MagnificationManager* GetInstance();
+  static MagnificationManager* Get();
 
-  virtual ~MagnificationManager() {}
+  virtual void AddObserver(MagnificationObserver* observer) = 0;
+  virtual void RemoveObserver(MagnificationObserver* observer) = 0;
 
   // Returns the current type of the screen magnifier.
   virtual ash::MagnifierType GetMagnifierType() = 0;
   // Changes the type of the screen magnifier.
   virtual void SetMagnifier(ash::MagnifierType type) = 0;
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_ACCESSIBILITY_MAGNIFICATION_MANAGER_H_