Revert "Migrate benchmarks to package:web" (#127207)

Reverts flutter/flutter#126848

Triggered some kind of measuring discrepancy / performance regression.

Author: joshualitt

dev/benchmarks/macrobenchmarks/lib/src/web/bench_image_decoding.dart

@@ -2,12 +2,10 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-import 'dart:js_interop';
+import 'dart:html' as html;
 import 'dart:typed_data';
 import 'dart:ui' as ui;
 
-import 'package:web/web.dart' as web;
-
 import 'recorder.dart';
 
 // Measures the performance of image decoding.
@@ -45,11 +43,8 @@ class BenchImageDecoding extends RawRecorder {
       return;
     }
     for (final String imageUrl in _imageUrls) {
-      final Future<JSAny?> fetchFuture = web.window.fetch(imageUrl.toJS).toDart;
-      final web.Body image = (await fetchFuture)! as web.Body;
-      final Future<JSAny?> imageFuture = image.arrayBuffer().toDart;
-      final JSArrayBuffer imageBuffer = (await imageFuture)! as JSArrayBuffer;
-      _imageData.add(imageBuffer.toDart.asUint8List());
+      final html.Body image = await html.window.fetch(imageUrl) as html.Body;
+      _imageData.add((await image.arrayBuffer() as ByteBuffer).asUint8List());
     }
   }
 

dev/benchmarks/macrobenchmarks/lib/src/web/bench_platform_view_infinite_scroll.dart

@@ -3,32 +3,30 @@
 // found in the LICENSE file.
 
 import 'dart:async';
-import 'dart:js_interop';
+import 'dart:html' as html;
 
 import 'package:flutter/material.dart';
-import 'package:web/web.dart' as web;
 
 // TODO(mdebbar): flutter/flutter#55000 Remove this conditional import once
 // web-only dart:ui_web APIs are exposed from a dedicated place.
 import 'platform_views/non_web.dart'
-    if (dart.library.js_interop) 'platform_views/web.dart';
+    if (dart.library.html) 'platform_views/web.dart';
 import 'recorder.dart';
 
 const String benchmarkViewType = 'benchmark_element';
 
 void _registerFactory() {
   platformViewRegistry.registerViewFactory(benchmarkViewType, (int viewId) {
-    final web.HTMLElement htmlElement = web.document.createElement('div'.toJS)
-        as web.HTMLDivElement;
-    htmlElement.id = '${benchmarkViewType}_$viewId'.toJS;
-    htmlElement.innerText = 'Google'.toJS;
+    final html.Element htmlElement = html.DivElement();
+    htmlElement.id = '${benchmarkViewType}_$viewId';
+    htmlElement.innerText = 'Google';
     htmlElement.style
-      ..setProperty('width'.toJS, '100%'.toJS)
-      ..setProperty('height'.toJS, '100%'.toJS)
-      ..setProperty('color'.toJS, 'black'.toJS)
-      ..setProperty('backgroundColor'.toJS, 'rgba(0, 255, 0, .5)'.toJS)
-      ..setProperty('textAlign'.toJS, 'center'.toJS)
-      ..setProperty('border'.toJS, '1px solid black'.toJS);
+      ..width = '100%'
+      ..height = '100%'
+      ..color = 'black'
+      ..backgroundColor = 'rgba(0, 255, 0, .5)'
+      ..textAlign = 'center'
+      ..border = '1px solid black';
     return htmlElement;
   });
 }

dev/benchmarks/macrobenchmarks/lib/src/web/recorder.dart

@@ -3,11 +3,8 @@
 // found in the LICENSE file.
 
 import 'dart:async';
-import 'dart:js_interop';
-// The analyzer currently thinks `js_interop_unsafe` is unused, but it is used
-// for `JSObject.[]=`.
-// ignore: unused_import
-import 'dart:js_interop_unsafe';
+import 'dart:html' as html;
+import 'dart:js_util' as js_util;
 import 'dart:math' as math;
 import 'dart:ui';
 
@@ -18,7 +15,6 @@ import 'package:flutter/scheduler.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter/widgets.dart';
 import 'package:meta/meta.dart';
-import 'package:web/web.dart' as web;
 
 /// The default number of samples from warm-up iterations.
 ///
@@ -1257,7 +1253,7 @@ void startMeasureFrame(Profile profile) {
 
   if (!profile.isWarmingUp) {
     // Tell the browser to mark the beginning of the frame.
-    web.window.performance.mark('measured_frame_start#$_currentFrameNumber'.toJS);
+    html.window.performance.mark('measured_frame_start#$_currentFrameNumber');
 
     _isMeasuringFrame = true;
   }
@@ -1280,11 +1276,11 @@ void endMeasureFrame() {
 
   if (_isMeasuringFrame) {
     // Tell the browser to mark the end of the frame, and measure the duration.
-    web.window.performance.mark('measured_frame_end#$_currentFrameNumber'.toJS);
-    web.window.performance.measure(
-      'measured_frame'.toJS,
-      'measured_frame_start#$_currentFrameNumber'.toJS,
-      'measured_frame_end#$_currentFrameNumber'.toJS,
+    html.window.performance.mark('measured_frame_end#$_currentFrameNumber');
+    html.window.performance.measure(
+      'measured_frame',
+      'measured_frame_start#$_currentFrameNumber',
+      'measured_frame_end#$_currentFrameNumber',
     );
 
     // Increment the current frame number.
@@ -1314,10 +1310,7 @@ void registerEngineBenchmarkValueListener(String name, EngineBenchmarkValueListe
 
   if (_engineBenchmarkListeners.isEmpty) {
     // The first listener is being registered. Register the global listener.
-    web.window['_flutter_internal_on_benchmark'.toJS] =
-        // Upcast to [Object] to export.
-        // ignore: unnecessary_cast
-        (_dispatchEngineBenchmarkValue as Object).toJS;
+    js_util.setProperty(html.window, '_flutter_internal_on_benchmark', _dispatchEngineBenchmarkValue);
   }
 
   _engineBenchmarkListeners[name] = listener;
@@ -1328,7 +1321,7 @@ void stopListeningToEngineBenchmarkValues(String name) {
   _engineBenchmarkListeners.remove(name);
   if (_engineBenchmarkListeners.isEmpty) {
     // The last listener unregistered. Remove the global listener.
-    web.window['_flutter_internal_on_benchmark'.toJS] = null;
+    js_util.setProperty(html.window, '_flutter_internal_on_benchmark', null);
   }
 }