refactor: Added processing branching by kIsWasm

google · Dec 4, 2024 · 7856777 · 7856777
1 parent 4e8e9de
commit 7856777
Show file tree

Hide file tree

Showing 2 changed files with 41 additions and 14 deletions.
diff --git a/lib/src/crypto_subtle.dart b/lib/src/crypto_subtle.dart
@@ -19,7 +19,7 @@ library common;
 import 'dart:js_interop';
 import 'dart:typed_data';
 
-import 'package:meta/meta.dart';
+import 'package:flutter/foundation.dart';
 
 import 'jsonwebkey.dart' show JsonWebKey, RsaOtherPrimesInfo;
 
@@ -333,39 +333,49 @@ extension type JSRsaOtherPrimesInfo(JSObject _) implements JSObject {
 }
 
 TypedData getRandomValues(TypedData array) {
+  // Since dart2wasm does not reflect values in the array, use setAll to reflect them.
+  // see: https://github.com/dart-lang/sdk/issues/59651
   if (array is Uint8List) {
     final values = array.toJS;
     window.crypto.getRandomValues(values);
-    array.setAll(0, values.toDart);
-    return values.toDart;
+    if (kIsWasm) {
+      array.setAll(0, values.toDart);
+    }
   } else if (array is Uint16List) {
     final values = array.toJS;
     window.crypto.getRandomValues(values);
-    array.setAll(0, values.toDart);
-    return values.toDart;
+    if (kIsWasm) {
+      array.setAll(0, values.toDart);
+    }
   } else if (array is Uint32List) {
     final values = array.toJS;
     window.crypto.getRandomValues(values);
-    array.setAll(0, values.toDart);
-    return values.toDart;
+    if (kIsWasm) {
+      array.setAll(0, values.toDart);
+    }
   } else if (array is Int8List) {
     final values = array.toJS;
     window.crypto.getRandomValues(values);
-    array.setAll(0, values.toDart);
-    return values.toDart;
+    if (kIsWasm) {
+      array.setAll(0, values.toDart);
+    }
   } else if (array is Int16List) {
     final values = array.toJS;
     window.crypto.getRandomValues(values);
-    array.setAll(0, values.toDart);
-    return values.toDart;
+    if (kIsWasm) {
+      array.setAll(0, values.toDart);
+    }
   } else if (array is Int32List) {
     final values = array.toJS;
     window.crypto.getRandomValues(values);
-    array.setAll(0, values.toDart);
-    return values.toDart;
+    if (kIsWasm) {
+      array.setAll(0, values.toDart);
+    }
   } else {
     throw UnsupportedError('Unsupported TypedData type ${array.runtimeType}');
   }
+
+  return array;
 }
 
 Future<ByteBuffer> decrypt(

diff --git a/test/crypto_subtle_test.dart b/test/crypto_subtle_test.dart
@@ -62,6 +62,19 @@ void main() {
   });
 
   group('crypto', () {
+    test('getRandomValues: success', () {
+      final data = Uint8List(16 * 1024);
+      expect(
+        data.every((e) => e == 0),
+        isTrue,
+      );
+      subtle.window.crypto.getRandomValues(data.toJS);
+      expect(
+        data.any((e) => e != 0),
+        isTrue,
+      );
+    }, skip: 'dart2wasm');
+
     test('getRandomValues: success', () {
       final data = Uint8List(16 * 1024);
       expect(
@@ -70,11 +83,15 @@ void main() {
       );
       final values = data.toJS;
       subtle.window.crypto.getRandomValues(values);
+      expect(
+        data.every((e) => e == 0),
+        isTrue,
+      );
       expect(
         values.toDart.any((e) => e != 0),
         isTrue,
       );
-    });
+    }, skip: 'dart2js');
 
     test('getRandomValues: too long', () {
       try {