5334-Revert_force_resolution_set_on_camera_mode (ace26487) · Commits · e / os / Camera

app/src/main/java/net/sourceforge/opencamera/MainActivity.java

+7 −4

Original line number	Diff line number	Diff line
		@@ -154,6 +154,7 @@ public class MainActivity extends AppCompatActivity implements PreferenceFragmen
		//private boolean ui_placement_right = true;

		private boolean want_no_limits; // whether we want to run with FLAG_LAYOUT_NO_LIMITS
		private boolean can_draw_nav_bar = true;
		private boolean set_window_insets_listener; // whether we've enabled a setOnApplyWindowInsetsListener()
		private int navigation_gap;
		public static volatile boolean test_preview_want_no_limits; // test flag, if set to true then instead use test_preview_want_no_limits_value; needs to be static, as it needs to be set before activity is created to take effect
		@@ -772,8 +773,10 @@ public class MainActivity extends AppCompatActivity implements PreferenceFragmen
		}
		}

		public void handleDecorFitsSystemWindows() {
		setDecorFitsSystemWindows(!isInFullScreenMode());
		public void handleDecorFitsSystemWindows(double previewRatio, double screenRatio) {
		boolean decorFitsSystemWindows = screenRatio >= previewRatio;
		setDecorFitsSystemWindows(decorFitsSystemWindows);
		can_draw_nav_bar = !decorFitsSystemWindows;
		}

		/**
		@@ -3624,10 +3627,10 @@ public class MainActivity extends AppCompatActivity implements PreferenceFragmen
		}

		/**
		* if sdk>=R & not in full screen, means the DecorFitsSystemWindows=true, then we can ignore navigationGap.
		* if sdk>=R & can't draw navigation bar, means the DecorFitsSystemWindows=true, then we can ignore navigationGap.
		*/
		public int getNavigationGap() {
		if(Build.VERSION.SDK_INT >= Build.VERSION_CODES.R && !isInFullScreenMode()) {
		if(Build.VERSION.SDK_INT >= Build.VERSION_CODES.R && !can_draw_nav_bar) {
		return 0;
		}
		return want_no_limits ? navigation_gap : 0;

app/src/main/java/net/sourceforge/opencamera/PreferenceSubPhoto.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -71,7 +71,7 @@ public class PreferenceSubPhoto extends PreferenceSubScreen {
		if( MyDebug.LOG )
		Log.d(TAG, "supports_photo_video_recording: " + supports_photo_video_recording);

		if( false && widths != null && heights != null && supports_burst != null ) {
		if( widths != null && heights != null && supports_burst != null ) {
		CharSequence [] entries = new CharSequence[widths.length];
		CharSequence [] values = new CharSequence[widths.length];
		for(int i=0;i<widths.length;i++) {

app/src/main/java/net/sourceforge/opencamera/preview/Preview.java

+8 −24

Original line number	Diff line number	Diff line
		@@ -3873,21 +3873,6 @@ public class Preview implements SurfaceHolder.Callback, TextureView.SurfaceTextu
		return optimalSize;
		}

		private static CameraController.Size getClosestSizeForFullScreen(List<CameraController.Size> sizes, double targetRatio) {
		if (MyDebug.LOG)
		Log.d(TAG, "getClosestSize()");
		CameraController.Size optimalSize = null;
		double minDiff = Double.MIN_VALUE;
		for (CameraController.Size size : sizes) {
		double ratio = (double) size.width / size.height;
		if ((ratio - targetRatio) > minDiff) {
		optimalSize = size;
		minDiff = ratio - targetRatio;
		}
		}
		return (optimalSize != null) ? optimalSize : getClosestSize(sizes, targetRatio, null);
		}

		public CameraController.Size getOptimalPreviewSize(List<CameraController.Size> sizes) {
		if( MyDebug.LOG )
		Log.d(TAG, "getOptimalPreviewSize()");
		@@ -3923,7 +3908,6 @@ public class Preview implements SurfaceHolder.Callback, TextureView.SurfaceTextu
		Log.d(TAG, "swapped display_size to: " + display_size.x + " x " + display_size.y);
		}
		}
		((MainActivity)getContext()).handleDecorFitsSystemWindows();
		double targetRatio = calculateTargetRatioForPreview(display_size);
		int targetHeight = Math.min(display_size.y, display_size.x);
		if( targetHeight <= 0 ) {
		@@ -3953,16 +3937,10 @@ public class Preview implements SurfaceHolder.Callback, TextureView.SurfaceTextu
		// can't find match for aspect ratio, so find closest one
		if( MyDebug.LOG )
		Log.d(TAG, "no preview size matches the aspect ratio");
		if (((MainActivity)getContext()).isInFullScreenMode()) {
		optimalSize = getClosestSizeForFullScreen(sizes, targetRatio);
		} else {
		optimalSize = getClosestSize(sizes, targetRatio, null);
		}
		}

		if (((MainActivity)getContext()).isInFullScreenMode()) {
		camera_controller.setPictureSize(optimalSize.width, optimalSize.height);
		}
		handleDecorFitsSystemWindows(optimalSize, display_size);

		if( MyDebug.LOG ) {
		Log.d(TAG, "chose optimalSize: " + optimalSize.width + " x " + optimalSize.height);
		@@ -3971,6 +3949,12 @@ public class Preview implements SurfaceHolder.Callback, TextureView.SurfaceTextu
		return optimalSize;
		}

		private void handleDecorFitsSystemWindows(CameraController.Size optimalSize, Point display_size) {
		double previewRatio = (double) optimalSize.width / optimalSize.height;
		double screenRatio = (double) display_size.x / display_size.y;
		((MainActivity)getContext()).handleDecorFitsSystemWindows(previewRatio, screenRatio);
		}

		public CameraController.Size getOptimalVideoPictureSize(List<CameraController.Size> sizes, double targetRatio) {
		if( MyDebug.LOG )
		Log.d(TAG, "getOptimalVideoPictureSize()");

app/src/main/java/net/sourceforge/opencamera/ui/PopupView.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -375,7 +375,7 @@ public class PopupView extends LinearLayout {
		if( MyDebug.LOG )
		Log.d(TAG, "PopupView time 8: " + (System.nanoTime() - debug_time));

		if( !preview.isVideo() && photo_mode != MyApplicationInterface.PhotoMode.Panorama && !main_activity.isInFullScreenMode()) {
		if( !preview.isVideo() && photo_mode != MyApplicationInterface.PhotoMode.Panorama ) {
		// Only show photo resolutions in photo mode - even if photo snapshots whilst recording video is supported, the
		// resolutions for that won't match what the user has requested for photo mode resolutions.
		// And Panorama mode chooses its own resolution.