Move Screen Rotation from MaximizeModeController to ScreenOrientationController

ash/content/display/screen_orientation_delegate_chromeos.h

 // Copyright 2014 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 ASH_CONTENT_DISPLAY_SCREEN_ORIENTATION_DELEGATE_CHROMEOS_H_
 #define ASH_CONTENT_DISPLAY_SCREEN_ORIENTATION_DELEGATE_CHROMEOS_H_
 
+#include "ash/ash_export.h"
 #include "base/macros.h"
+#include "base/observer_list.h"
 #include "content/public/browser/screen_orientation_delegate.h"
 #include "third_party/WebKit/public/platform/WebScreenOrientationLockType.h"
+#include "ui/gfx/display.h"
 
 namespace aura {
 class Window;
 }
 
 namespace content {
 class WebContents;
 }
 
 namespace ash {
 
 // Implements ChromeOS specific functionality for ScreenOrientationProvider.
-class ScreenOrientationDelegate : public content::ScreenOrientationDelegate {
+class ASH_EXPORT ScreenOrientationDelegate
+    : public content::ScreenOrientationDelegate {
  public:
+  // Observer that reports changes to the state of ScreenOrientationProvider's
+  // rotation lock.
+  class Observer {
+   public:
+    // Invoked whenever |rotation_locked_| is changed.

[flackr, 2014/11/27 15:39:15]

Since |rotation_locked_| isn't a member of the observer, describe this by the
meaning.

Invoked when rotation is locked or unlocked.

[jonross, 2014/12/10 17:57:30]

Done.

+    virtual void OnRotationLockChanged(bool rotation_locked) {}
+
+   protected:
+    virtual ~Observer() {}
+  };
+
   ScreenOrientationDelegate();
   virtual ~ScreenOrientationDelegate();
 
+  // Add/Remove observers.
+  void AddObserver(Observer* observer);
+  void RemoveObserver(Observer* observer);
+
+  bool ignore_display_configuration_updates() const {
+    return ignore_display_configuration_updates_;
+  }
+
+  // True if |rotation_lock_| has been set and accelerometer updates should not
+  // rotate the display.
+  bool rotation_locked() { return rotation_locked_; }
+
+  // If |rotation_locked| future accelerometer updates should not change the
+  // display rotation.
+  void SetRotationLocked(bool rotation_locked);
+
+  // Sets the display rotation and suppresses display notifications.
+  void SetDisplayRotation(gfx::Display::Rotation rotation);
+
   // content::ScreenOrientationDelegate:
   bool FullScreenRequired(content::WebContents* web_contents) override;
   void Lock(content::WebContents* web_contents,
             blink::WebScreenOrientationLockType lock_orientation) override;
   bool ScreenOrientationProviderSupported() override;
   void Unlock(content::WebContents* web_contents) override;
 
  private:
+  // Sets the display rotation to |rotation|. Future calls to
+  // OnAccelerometerUpdated should not be used to change the rotation.

[flackr, 2014/11/27 15:39:15]

Can you update this comment? OnAccelerometerUpdated doesn't refer to something
local.

[jonross, 2014/12/10 17:57:30]

Done.

+  // SetRotationLocked(false) removes the rotation lock.
+  void LockRotation(gfx::Display::Rotation rotation);
+
   // Locks rotation to the angle matching the primary orientation for
   // |lock_orientation|.
   void LockRotationToPrimaryOrientation(
       blink::WebScreenOrientationLockType lock_orientation);
 
   // Locks rotation to the angle matching the secondary orientation for
   // |lock_orientation|.
   void LockRotationToSecondaryOrientation(
       blink::WebScreenOrientationLockType lock_orientation);
 
   // For orientations that do not specify primary or secondary, locks to the
   // current rotation if it matches |lock_orientation|. Otherwise locks to a
   // matching rotation.
   void LockToRotationMatchingOrientation(
       blink::WebScreenOrientationLockType lock_orientation);
 
   // The window that has applied the current lock. No other window can apply a
   // lock until the current window unlocks rotation.
   aura::Window* locking_window_;
 
   // The orientation of the display when at a rotation of 0.
   blink::WebScreenOrientationLockType natural_orientation_;
 
+  // True when changes being applied cause OnDisplayConfigurationChanged() to be
+  // called, and for which these changes should be ignored.
+  bool ignore_display_configuration_updates_;
+
+  // When true then accelerometer updates should not rotate the display.
+  bool rotation_locked_;
+
+  // Rotation Lock observers.
+  ObserverList<Observer> observers_;
+
   DISALLOW_COPY_AND_ASSIGN(ScreenOrientationDelegate);
 };
 
 }  // namespace ash
 
 #endif  // ASH_CONTENT_DISPLAY_SCREEN_ORIENTATION_DELEGATE_CHROMEOS_H_