refactor: Better custom image resizer

lib/utils/custom_image_resizer.dart

@@ -1,21 +1,43 @@
-import 'dart:typed_data';
 import 'dart:ui';
 
+import 'package:flutter/foundation.dart';
+import 'package:flutter/painting.dart';
+
 import 'package:matrix/matrix.dart';
 import 'package:native_imaging/native_imaging.dart' as native;
 
+(int, int) _scaleToBox(int width, int height, {required int boxSize}) {
+  final fit = applyBoxFit(
+    BoxFit.scaleDown,
+    Size(width.toDouble(), height.toDouble()),
+    Size(boxSize.toDouble(), boxSize.toDouble()),
+  ).destination;
+  return (fit.width.round(), fit.height.round());
+}
+
 Future<MatrixImageFileResizedResponse?> customImageResizer(
   MatrixImageFileResizeArguments arguments,
 ) async {
+  if (kIsWeb) {
+    throw UnsupportedError(
+      'customImageResizer only supports non-web platforms.',
+    );
+  }
+
   await native.init();
-  late native.Image nativeImg;
 
-  try {
+  var imageBytes = arguments.bytes;
-    nativeImg = await native.Image.loadEncoded(arguments.bytes); // load on web
+  String? blurhash;
-  } on UnsupportedError {
+
+  var originalWidth = 0;
+  var originalHeight = 0;
+  var width = 0;
+  var height = 0;
+
   try {
     // for the other platforms
     final dartCodec = await instantiateImageCodec(arguments.bytes);
+    final frameCount = dartCodec.frameCount;
     final dartFrame = await dartCodec.getNextFrame();
     final rgbaData = await dartFrame.image.toByteData();
     if (rgbaData == null) {
@@ -27,41 +49,55 @@ Future<MatrixImageFileResizedResponse?> customImageResizer(
       rgbaData.lengthInBytes,
     );
 
-      final width = dartFrame.image.width;
+    width = originalWidth = dartFrame.image.width;
-      final height = dartFrame.image.height;
+    height = originalHeight = dartFrame.image.height;
+
+    var nativeImg = native.Image.fromRGBA(width, height, rgba);
 
     dartFrame.image.dispose();
     dartCodec.dispose();
 
-      nativeImg = native.Image.fromRGBA(width, height, rgba);
+    if (arguments.calcBlurhash) {
-    } catch (e, s) {
+      // scale down image for blurhashing to speed it up
-      Logs().e("Could not generate preview", e, s);
+      final (blurW, blurH) = _scaleToBox(width, height, boxSize: 100);
-      rethrow;
+      final blurhashImg = nativeImg.resample(
-    }
+        blurW, blurH,
-  }
+        // nearest is unsupported...
+        native.Transform.bilinear,
+      );
+
+      blurhash = blurhashImg.toBlurhash(3, 3);
 
-  final width = nativeImg.width;
+      blurhashImg.free();
-  final height = nativeImg.height;
+    }
 
+    if (frameCount > 1) {
+      // Don't scale down animated images, since those would lose frames.
+      nativeImg.free();
+    } else {
       final max = arguments.maxDimension;
       if (width > max || height > max) {
-    var w = max, h = max;
+        (width, height) = _scaleToBox(width, height, boxSize: max);
-    if (width > height) {
-      h = max * height ~/ width;
-    } else {
-      w = max * width ~/ height;
-    }
 
-    final scaledImg = nativeImg.resample(w, h, native.Transform.lanczos);
+        final scaledImg =
+            nativeImg.resample(width, height, native.Transform.lanczos);
         nativeImg.free();
         nativeImg = scaledImg;
       }
-  final jpegBytes = await nativeImg.toJpeg(75);
+
+      imageBytes = await nativeImg.toJpeg(75);
+      nativeImg.free();
+    }
+  } catch (e, s) {
+    Logs().e("Could not generate preview", e, s);
+  }
 
   return MatrixImageFileResizedResponse(
-    bytes: jpegBytes,
+    bytes: imageBytes,
-    width: nativeImg.width,
+    width: width,
-    height: nativeImg.height,
+    height: height,
-    blurhash: arguments.calcBlurhash ? nativeImg.toBlurhash(3, 3) : null,
+    originalWidth: originalWidth,
+    originalHeight: originalHeight,
+    blurhash: blurhash,
   );
 }