Fix camera preview orientation when device is held in portrait.

The problem is the rotation animation may still in progress during
onCreate. When we start the preview in onCreate, the display rotation
is still the value in portrait. So we check the display rotation
again in surfaceChanged. If it is different, restart the preview.

bug:3362860

Change-Id: Ib930c18d089114a3b5b9ca0f599e1ebdace73eb7

1 files changed, 12 insertions, 2 deletions

diff --git a/src/com/android/camera/Camera.java b/src/com/android/camera/Camera.java
index 84456e3..0a8cc99 100644
--- a/src/com/android/camera/Camera.java
+++ b/src/com/android/camera/Camera.java
@@ -190,6 +190,9 @@ public class Camera extends NoSearchActivity implements View.OnClickListener,
             "android.intent.extra.quickCapture";
 
     private boolean mPreviewing;
+    // The display rotation in degrees. This is only valid when mPreviewing is
+    // true.
+    private int mDisplayRotation;
     private boolean mPausing;
     private boolean mFirstTimeInitialized;
     private boolean mIsImageCaptureIntent;
@@ -1646,7 +1649,13 @@ public class Camera extends NoSearchActivity implements View.OnClickListener,
         // Ignore it.
         if (mPausing || isFinishing()) return;
 
-        if (mPreviewing && holder.isCreating()) {
+        // Set preview display if the surface is being created. Preview was
+        // already started. Also restart the preview if display rotation has
+        // changed. Sometimes this happens when the device is held in portrait
+        // and camera app is opened. Rotation animation takes some time and
+        // display rotation in onCreate may not be what we want.
+        if (mPreviewing && (Util.getDisplayRotation(this) == mDisplayRotation)
+                && holder.isCreating()) {
             // Set preview display if the surface is being created and preview
             // was already started. That means preview display was set to null
             // and we need to set it now.
@@ -1730,7 +1739,8 @@ public class Camera extends NoSearchActivity implements View.OnClickListener,
         if (mPreviewing) stopPreview();
 
         setPreviewDisplay(mSurfaceHolder);
-        Util.setCameraDisplayOrientation(this, mCameraId, mCameraDevice);
+        mDisplayRotation = Util.getDisplayRotation(this);
+        Util.setCameraDisplayOrientation(mDisplayRotation, mCameraId, mCameraDevice);
         setCameraParameters(UPDATE_PARAM_ALL);