[Camera] Fix rotation; disable rotation animation

Author: proninyaroslav

android/app/src/main/kotlin/org/proninyaroslav/blink_comparison/MainActivity.kt

@@ -21,19 +21,20 @@ package org.proninyaroslav.blink_comparison
 
 import android.os.Build
 import android.os.Bundle
-import androidx.core.view.WindowCompat
-import org.proninyaroslav.blink_comparison.channel.SaveRefImageServiceChannel
-import org.proninyaroslav.blink_comparison.channel.SaveRefImageServiceQueueChannel
-import org.proninyaroslav.blink_comparison.channel.SaveRefImageServiceResultChannel
 import io.flutter.embedding.android.FlutterActivity
 import io.flutter.embedding.engine.FlutterEngine
 import io.flutter.plugin.common.EventChannel
 import io.flutter.plugin.common.MethodChannel
+import org.proninyaroslav.blink_comparison.channel.SaveRefImageServiceChannel
+import org.proninyaroslav.blink_comparison.channel.SaveRefImageServiceQueueChannel
+import org.proninyaroslav.blink_comparison.channel.SaveRefImageServiceResultChannel
+import org.proninyaroslav.blink_comparison.channel.WindowManagerChannel
 
 class MainActivity : FlutterActivity() {
     private lateinit var saveRefImageServiceChannel: SaveRefImageServiceChannel
     private lateinit var saveRefImageServiceQueueChannel: SaveRefImageServiceQueueChannel
     private lateinit var saveRefImageServiceResultChannel: SaveRefImageServiceResultChannel
+    private lateinit var windowManagerChannel: WindowManagerChannel
 
     override fun onCreate(savedInstanceState: Bundle?) {
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
@@ -55,6 +56,7 @@ class MainActivity : FlutterActivity() {
             saveRefImageServiceQueueChannel,
             saveRefImageServiceResultChannel,
         )
+        windowManagerChannel = WindowManagerChannel { this }
         MethodChannel(
             flutterEngine.dartExecutor.binaryMessenger,
             SaveRefImageServiceChannel.channelName,
@@ -67,5 +69,9 @@ class MainActivity : FlutterActivity() {
             flutterEngine.dartExecutor.binaryMessenger,
             SaveRefImageServiceResultChannel.channelName,
         ).setStreamHandler(saveRefImageServiceResultChannel)
+        MethodChannel(
+            flutterEngine.dartExecutor.binaryMessenger,
+            WindowManagerChannel.channelName,
+        ).setMethodCallHandler(windowManagerChannel)
     }
 }

android/app/src/main/kotlin/org/proninyaroslav/blink_comparison/channel/WindowManagerChannel.kt

@@ -0,0 +1,65 @@
+/*
+ * Copyright (C) 2024 Yaroslav Pronin <[email protected]>
+ *
+ * This file is part of Blink Comparison.
+ *
+ * Blink Comparison is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Blink Comparison is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Blink Comparison.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+package org.proninyaroslav.blink_comparison.channel
+
+import android.app.Activity
+import android.view.WindowManager
+import io.flutter.plugin.common.MethodCall
+import io.flutter.plugin.common.MethodChannel
+
+open class WindowManagerChannel(private val getActivity: () -> Activity) :
+    MethodChannel.MethodCallHandler {
+
+    companion object {
+        const val channelName = "org.proninyaroslav.blink_comparison/window_manager"
+    }
+
+    object Methods {
+        const val setSecureFlag = "setSecureFlag"
+        const val setRotationAnimation = "setRotationAnimation"
+    }
+
+    override fun onMethodCall(call: MethodCall, result: MethodChannel.Result) {
+        val window = getActivity().window
+        when (call.method) {
+            Methods.setSecureFlag -> {
+                val enabled = call.arguments as Boolean
+                if (enabled) {
+                    window.addFlags(WindowManager.LayoutParams.FLAG_SECURE)
+                } else {
+                    window.clearFlags(WindowManager.LayoutParams.FLAG_SECURE)
+                }
+                return result.success(null)
+            }
+
+            Methods.setRotationAnimation -> {
+                val animationValue = call.arguments as Int
+                window.attributes = window.attributes.apply {
+                    rotationAnimation = animationValue
+                }
+                return result.success(null)
+            }
+
+            else -> {
+                return result.notImplemented()
+            }
+        }
+    }
+}

lib/core/window_manager.dart

@@ -15,26 +15,55 @@
 // You should have received a copy of the GNU General Public License
 // along with Blink Comparison.  If not, see <http://www.gnu.org/licenses/>.
 
-import 'package:flutter_windowmanager_plus/flutter_windowmanager_plus.dart';
+import 'package:blink_comparison/platform/window_manager_native.dart';
 import 'package:injectable/injectable.dart';
 
 abstract class WindowManager {
   /// Allow/disallow screenshots
-  Future<bool> setSecureFlag(bool enable);
+  Future<void> setSecureFlag(bool enable);
+
+  /// Define the exit and entry animations used on this window when the device is rotated.
+  /// This only has an affect if the incoming and outgoing topmost opaque windows
+  /// have the #FLAG_FULLSCREEN bit set and are not covered by other windows.
+  /// All other situations default to the [RotationAnimtation.rotate] behavior.
+  Future<void> setRotationAnimation(RotationAnimtation animation);
+}
+
+enum RotationAnimtation {
+  /// Window will visually rotate in or out following a rotation.
+  rotate(0),
+
+  /// Window will fade in or out following a rotation.
+  crossfade(1),
+
+  /// Window will immediately disappear or appear following a rotation.
+  jumpcut(2),
+
+  /// This works like [jumpcut] but will fall back to [crossfade] if
+  /// rotation can't be applied without pausing the screen. For example,
+  /// this is ideal for Camera apps which don't want the viewfinder contents to
+  /// ever rotate or fade (and rather to be seamless) but also don't want
+  /// [jumpcut] during app transition scenarios where seamless rotation can't be applied.
+  seamless(3);
+
+  final int value;
+
+  const RotationAnimtation(this.value);
+
+  static RotationAnimtation defaultValue = rotate;
 }
 
 @Singleton(as: WindowManager)
 class WindowManagerImpl implements WindowManager {
+  final WindowManagerNative _windowManager;
+
+  WindowManagerImpl(this._windowManager);
+
+  @override
+  Future<void> setSecureFlag(bool enable) async =>
+      _windowManager.setSecureFlag(enable);
+
   @override
-  Future<bool> setSecureFlag(bool enable) async {
-    if (enable) {
-      return FlutterWindowManagerPlus.addFlags(
-        FlutterWindowManagerPlus.FLAG_SECURE,
-      );
-    } else {
-      return FlutterWindowManagerPlus.clearFlags(
-        FlutterWindowManagerPlus.FLAG_SECURE,
-      );
-    }
-  }
+  Future<void> setRotationAnimation(RotationAnimtation animation) async =>
+      _windowManager.setRotationAnimation(animation);
 }

lib/injector.config.dart

@@ -0,0 +1,37 @@
+// Copyright (C) 2024 Yaroslav Pronin <[email protected]>
+//
+// This file is part of Blink Comparison.
+//
+// Blink Comparison is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Blink Comparison is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Blink Comparison.  If not, see <http://www.gnu.org/licenses/>.
+
+import 'package:blink_comparison/core/window_manager.dart';
+import 'package:flutter/material.dart';
+import 'package:flutter/services.dart';
+import 'package:injectable/injectable.dart';
+
+@injectable
+class WindowManagerNative {
+  @visibleForTesting
+  final channel = MethodChannel(
+    'org.proninyaroslav.blink_comparison/window_manager',
+  );
+
+  Future<void> setSecureFlag(bool enable) async {
+    await channel.invokeMethod('setSecureFlag', enable);
+  }
+
+  Future<void> setRotationAnimation(RotationAnimtation animtaion) async {
+    await channel.invokeMethod('setRotationAnimation', animtaion.value);
+  }
+}

lib/platform/window_manager_native.dart

@@ -15,9 +15,10 @@
 // You should have received a copy of the GNU General Public License
 // along with Blink Comparison.  If not, see <http://www.gnu.org/licenses/>.
 
+import 'package:blink_comparison/ui/components/widget.dart';
 import 'package:flutter/material.dart';
 
-class CameraAppBar extends StatelessWidget implements PreferredSizeWidget {
+class CameraAppBar extends StatelessWidget {
   final List<Widget>? actions;
   const CameraAppBar({
     super.key,
@@ -27,13 +28,25 @@ class CameraAppBar extends StatelessWidget implements PreferredSizeWidget {
   @override
   Widget build(BuildContext context) {
     final color = Theme.of(context).colorScheme.surface.withOpacity(0.54);
-    return AppBar(
-      backgroundColor: color,
-      actionsIconTheme: const IconThemeData(color: Colors.white),
-      actions: actions,
+    return PortraitOnlyWidget(
+      direction: RotateDirection.clockwise,
+      child: SizedBox.fromSize(
+        size: const Size.fromHeight(56.0),
+        child: AppBar(
+          leading: PortraitOnlyWidget(
+            child: BackButton(),
+          ),
+          backgroundColor: color,
+          actionsIconTheme: const IconThemeData(color: Colors.white),
+          actions: actions
+              ?.map(
+                (child) => PortraitOnlyWidget(
+                  child: child,
+                ),
+              )
+              .toList(),
+        ),
+      ),
     );
   }
-
-  @override
-  Size get preferredSize => const Size.fromHeight(56.0);
 }