Respect system auto-rotate setting.

  bug:10623244

Change-Id: I66757e50daf487680ac4f49d854c0f49d672b6ff

1 files changed, 10 insertions, 5 deletions

diff --git a/src/com/android/camera/app/OrientationManager.java b/src/com/android/camera/app/OrientationManager.java
index 7bf924218..ef03a0163 100644
--- a/src/com/android/camera/app/OrientationManager.java
+++ b/src/com/android/camera/app/OrientationManager.java
@@ -68,9 +68,12 @@ public class OrientationManager {
     //  the framework orientation, we always set the compensation value to 0.
     ////////////////////////////////////////////////////////////////////////////
 
-    // Lock the framework orientation to the current device orientation
+    /**
+     * Lock the framework orientation to the current device orientation
+     * rotates. No effect if the system setting of auto-rotation is off.
+     */
     public void lockOrientation() {
-        if (mOrientationLocked) return;
+        if (mOrientationLocked || mRotationLockedSetting) return;
         mOrientationLocked = true;
         if (ApiHelper.HAS_ORIENTATION_LOCK) {
             mActivity.setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_LOCKED);
@@ -79,10 +82,12 @@ public class OrientationManager {
         }
     }
 
-    // Unlock the framework orientation, so it can change when the device
-    // rotates.
+    /**
+     * Unlock the framework orientation, so it can change when the device
+     * rotates. No effect if the system setting of auto-rotation is off.
+     */
     public void unlockOrientation() {
-        if (!mOrientationLocked) return;
+        if (!mOrientationLocked || mRotationLockedSetting) return;
         mOrientationLocked = false;
         Log.d(TAG, "unlock orientation");
         mActivity.setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_FULL_SENSOR);