[web] Updates package:web dependency to ^0.5.0. (flutter#5791)

This PR updates the packages that are using `package:web` to version `^0.5.0`.

Ancilliary changes:

* Bump `environment` to `flutter: ">=3.19.0"` and `sdk: ^3.3.0`.
* Bump version to next `Y`
* Clean-up code that was kept for compatibility with versions of `web: <0.5.0`.

The main exception to this is `package:google_sign_in_web`, which depends on a version of `google_identity_services_web` that has a dependency on package:web that is `<0.5.0`, so that package needs to have a range until `google_identity_services_web` gets published with the new ^0.5.0 dependency.

Co-Authored-By: David Iglesias<[email protected]>

Author: kevmoo

packages/cross_file/CHANGELOG.md

@@ -1,6 +1,7 @@
-## NEXT
+## 0.3.4
 
-* Updates minimum supported SDK version to Flutter 3.13/Dart 3.1.
+* Updates to web code to package `web: ^0.5.0`.
+* Updates SDK version to Dart `^3.3.0`.
 
 ## 0.3.3+8
 

packages/cross_file/lib/src/types/html.dart

@@ -66,9 +66,7 @@ class XFile extends XFileBase {
         super(path) {
     if (path == null) {
       _browserBlob = _createBlobFromBytes(bytes, mimeType);
-      // TODO(kevmoo): drop ignore when pkg:web constraint excludes v0.3
-      // ignore: unnecessary_cast
-      _path = URL.createObjectURL(_browserBlob! as JSObject);
+      _path = URL.createObjectURL(_browserBlob!);
     } else {
       _path = path;
     }
@@ -131,9 +129,7 @@ class XFile extends XFileBase {
 
     // Attempt to re-hydrate the blob from the `path` via a (local) HttpRequest.
     // Note that safari hangs if the Blob is >=4GB, so bail out in that case.
-    // TODO(kevmoo): Remove ignore and fix when the MIN Dart SDK is 3.3
-    // ignore: unnecessary_non_null_assertion
-    if (isSafari() && _length != null && _length! >= _fourGigabytes) {
+    if (isSafari() && _length != null && _length >= _fourGigabytes) {
       throw Exception('Safari cannot handle XFiles larger than 4GB.');
     }
 

packages/cross_file/pubspec.yaml

@@ -2,14 +2,14 @@ name: cross_file
 description: An abstraction to allow working with files across multiple platforms.
 repository: https://github.com/flutter/packages/tree/main/packages/cross_file
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+cross_file%22
-version: 0.3.3+8
+version: 0.3.4
 
 environment:
-  sdk: ^3.2.0
+  sdk: ^3.3.0
 
 dependencies:
   meta: ^1.3.0
-  web: '>=0.3.0 <0.5.0'
+  web: ^0.5.0
 
 dev_dependencies:
   path: ^1.8.1

packages/cross_file/test/x_file_html_test.dart

@@ -69,10 +69,10 @@ void main() {
     test('Stores data as a Blob', () async {
       // Read the blob from its path 'natively'
       final html.Response response =
-          (await html.window.fetch(file.path.toJS).toDart)! as html.Response;
+          await html.window.fetch(file.path.toJS).toDart;
 
-      final JSAny? arrayBuffer = await response.arrayBuffer().toDart;
-      final ByteBuffer data = (arrayBuffer! as JSArrayBuffer).toDart;
+      final JSAny arrayBuffer = await response.arrayBuffer().toDart;
+      final ByteBuffer data = (arrayBuffer as JSArrayBuffer).toDart;
       expect(data.asUint8List(), equals(bytes));
     });
 

packages/file_selector/file_selector_web/CHANGELOG.md

@@ -1,6 +1,7 @@
-## NEXT
+## 0.9.4
 
-* Updates minimum supported SDK version to Flutter 3.13/Dart 3.1.
+* Updates web code to package `web: ^0.5.0`.
+* Updates SDK version to Dart `^3.3.0`. Flutter `^3.16.0`.
 
 ## 0.9.3
 

packages/file_selector/file_selector_web/example/integration_test/dom_helper_test.dart

@@ -18,11 +18,10 @@ void main() {
 
     FileList? createFileList(List<File> files) {
       final DataTransfer dataTransfer = DataTransfer();
+      // Tear-offs of external extension type interop member 'add' are disallowed.
+      // ignore: prefer_foreach
       for (final File e in files) {
-        // TODO(srujzs): This is necessary in order to support package:web 0.4.0.
-        // This was not needed with 0.3.0, hence the lint.
-        // ignore: unnecessary_cast
-        dataTransfer.items.add(e as JSAny);
+        dataTransfer.items.add(e);
       }
       return dataTransfer.files;
     }
@@ -46,13 +45,8 @@ void main() {
     });
 
     group('getFiles', () {
-      final File mockFile1 =
-          // TODO(srujzs): Remove once typed JSArrays (JSArray<T>) get to `stable`.
-          // ignore: always_specify_types
-          File(<Object>['123456'].jsify as JSArray, 'file1.txt');
-      // TODO(srujzs): Remove once typed JSArrays (JSArray<T>) get to `stable`.
-      // ignore: always_specify_types
-      final File mockFile2 = File(<Object>[].jsify as JSArray, 'file2.txt');
+      final File mockFile1 = File(<JSAny>['123456'.toJS].toJS, 'file1.txt');
+      final File mockFile2 = File(<JSAny>[].toJS, 'file2.txt');
 
       testWidgets('works', (_) async {
         final Future<List<XFile>> futureFiles = domHelper.getFiles(
@@ -114,32 +108,27 @@ void main() {
       testWidgets('sets the <input /> attributes and clicks it', (_) async {
         const String accept = '.jpg,.png';
         const bool multiple = true;
-        bool wasClicked = false;
-
-        //ignore: unawaited_futures
-        input.onClick.first.then((_) => wasClicked = true);
+        final Future<bool> wasClicked = input.onClick.first.then((_) => true);
 
         final Future<List<XFile>> futureFile = domHelper.getFiles(
           accept: accept,
           multiple: multiple,
           input: input,
         );
 
-        expect(input.matches('body'), true);
+        expect(input.isConnected, true,
+            reason: 'input must be injected into the DOM');
         expect(input.accept, accept);
         expect(input.multiple, multiple);
-        expect(
-          wasClicked,
-          true,
-          reason:
-              'The <input /> should be clicked otherwise no dialog will be shown',
-        );
+        expect(await wasClicked, true,
+            reason:
+                'The <input /> should be clicked otherwise no dialog will be shown');
 
         setFilesAndTriggerChange(<File>[]);
         await futureFile;
 
         // It should be already removed from the DOM after the file is resolved.
-        expect(input.parentElement, isNull);
+        expect(input.isConnected, isFalse);
       });
     });
   });

packages/file_selector/file_selector_web/example/pubspec.yaml

@@ -2,16 +2,16 @@ name: file_selector_web_integration_tests
 publish_to: none
 
 environment:
-  sdk: ^3.1.0
-  flutter: ">=3.13.0"
+  sdk: ^3.3.0
+  flutter: ">=3.19.0"
 
 dependencies:
   file_selector_platform_interface: ^2.6.0
   file_selector_web:
     path: ../
   flutter:
     sdk: flutter
-  web: '>=0.3.0 <0.5.0'
+  web: ^0.5.0
 
 dev_dependencies:
   flutter_test:

packages/file_selector/file_selector_web/lib/src/dom_helper.dart

@@ -14,11 +14,11 @@ import 'package:web/helpers.dart';
 class DomHelper {
   /// Default constructor, initializes the container DOM element.
   DomHelper() {
-    final Element body = querySelector('body')!;
+    final Element body = document.querySelector('body')!;
     body.appendChild(_container);
   }
 
-  final Element _container = createElementTag('file-selector');
+  final Element _container = document.createElement('file-selector');
 
   /// Sets the <input /> attributes and waits for a file to be selected.
   Future<List<XFile>> getFiles({
@@ -28,7 +28,7 @@ class DomHelper {
   }) {
     final Completer<List<XFile>> completer = Completer<List<XFile>>();
     final HTMLInputElement inputElement =
-        input ?? (createElementTag('input') as HTMLInputElement)
+        input ?? (document.createElement('input') as HTMLInputElement)
           ..type = 'file';
 
     _container.appendChild(
@@ -72,10 +72,7 @@ class DomHelper {
   }
 
   XFile _convertFileToXFile(File file) => XFile(
-        // TODO(srujzs): This is necessary in order to support package:web 0.4.0.
-        // This was not needed with 0.3.0, hence the lint.
-        // ignore: unnecessary_cast
-        URL.createObjectURL(file as JSObject),
+        URL.createObjectURL(file),
         name: file.name,
         length: file.size,
         lastModified: DateTime.fromMillisecondsSinceEpoch(file.lastModified),

packages/file_selector/file_selector_web/pubspec.yaml

@@ -2,11 +2,11 @@ name: file_selector_web
 description: Web platform implementation of file_selector
 repository: https://github.com/flutter/packages/tree/main/packages/file_selector/file_selector_web
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+file_selector%22
-version: 0.9.3
+version: 0.9.4
 
 environment:
-  sdk: ^3.2.0
-  flutter: ">=3.16.0"
+  sdk: ^3.3.0
+  flutter: ">=3.19.0"
 
 flutter:
   plugin:
@@ -22,7 +22,7 @@ dependencies:
     sdk: flutter
   flutter_web_plugins:
     sdk: flutter
-  web: '>=0.3.0 <0.5.0'
+  web: ^0.5.0
 
 dev_dependencies:
   flutter_test:

packages/google_identity_services_web/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 0.3.1
+
+* Updates web code to package `web: ^0.5.0`.
+* Updates SDK version to Dart `^3.3.0`. Flutter `^3.19.0`.
+
 ## 0.3.0+2
 
 * Adds `fedcm_auto` to `CredentialSelectBy` enum.