Magnifier: Prevent useless operation in enabling/disabling magnifier.

chrome/browser/chromeos/accessibility/magnification_manager.h

Index: chrome/browser/chromeos/accessibility/magnification_manager.h
diff --git a/chrome/browser/chromeos/accessibility/magnification_manager.h b/chrome/browser/chromeos/accessibility/magnification_manager.h
index 29529b5b647fbbbb7b78f21e0df909204022bf8c..b375bfd10426d879a98eb1c8d7b727aaa667621f 100644
--- a/chrome/browser/chromeos/accessibility/magnification_manager.h
+++ b/chrome/browser/chromeos/accessibility/magnification_manager.h
@@ -9,6 +9,11 @@
 
 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).
 //
@@ -19,14 +24,22 @@ namespace chromeos {
 //     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;