Formatting

Author: bkonyi

dwds/lib/src/debugging/chrome_inspector.dart

@@ -235,8 +235,10 @@ class ChromeAppInspector extends AppInspector {
     String selector, [
     List<dynamic> arguments = const [],
   ]) async {
-    final remoteArguments =
-        arguments.cast<String>().map(remoteObjectFor).toList();
+    final remoteArguments = arguments
+        .cast<String>()
+        .map(remoteObjectFor)
+        .toList();
     // We special case the Dart library, where invokeMethod won't work because
     // it's not really a Dart object.
     if (isLibraryId(targetId)) {
@@ -263,15 +265,15 @@ class ChromeAppInspector extends AppInspector {
     }
     return globalToolConfiguration.loadStrategy is DdcLibraryBundleStrategy
         ? _evaluateLibraryMethodWithDdcLibraryBundle(
-          libraryUri,
-          selector,
-          arguments,
-        )
+            libraryUri,
+            selector,
+            arguments,
+          )
         : _evaluateInLibrary(
-          libraryUri,
-          'function () { return this.$selector.apply(this, arguments); }',
-          arguments,
-        );
+            libraryUri,
+            'function () { return this.$selector.apply(this, arguments); }',
+            arguments,
+          );
   }
 
   /// Evaluate [expression] by calling Chrome's Runtime.evaluate.
@@ -575,14 +577,15 @@ class ChromeAppInspector extends AppInspector {
       offset: offset,
       length: length,
     );
-    final args =
-        [offset, rangeCount].map(dartIdFor).map(remoteObjectFor).toList();
+    final args = [
+      offset,
+      rangeCount,
+    ].map(dartIdFor).map(remoteObjectFor).toList();
     // If this is a List, just call sublist. If it's a Map, get the entries, but
     // avoid doing a toList on a large map using skip/take to get the section we
     // want. To make those alternatives easier in JS, pass both count and end.
-    final expression =
-        globalToolConfiguration.loadStrategy.dartRuntimeDebugger
-            .getSubRangeJsExpression();
+    final expression = globalToolConfiguration.loadStrategy.dartRuntimeDebugger
+        .getSubRangeJsExpression();
 
     return await jsCallFunctionOn(receiver, expression, args);
   }
@@ -625,9 +628,8 @@ class ChromeAppInspector extends AppInspector {
   /// Updates [Isolate.extensionRPCs] to this set.
   @override
   Future<Set<String>> getExtensionRpcs() async {
-    final expression =
-        globalToolConfiguration.loadStrategy.dartRuntimeDebugger
-            .getDartDeveloperExtensionNamesJsExpression();
+    final expression = globalToolConfiguration.loadStrategy.dartRuntimeDebugger
+        .getDartDeveloperExtensionNamesJsExpression();
     final extensionRpcs = <String>{};
     final params = {
       'expression': expression,

dwds/lib/src/debugging/classes.dart

@@ -142,10 +142,9 @@ class ChromeAppClassHelper {
 
     final superClassLibraryId = classDescriptor['superClassLibraryId'];
     final superClassName = classDescriptor['superClassName'];
-    final superClassRef =
-        superClassName == null
-            ? null
-            : classRefFor(superClassLibraryId, superClassName);
+    final superClassRef = superClassName == null
+        ? null
+        : classRefFor(superClassLibraryId, superClassName);
 
     // TODO: Implement the rest of these
     // https://github.com/dart-lang/webdev/issues/176.

dwds/lib/src/debugging/debugger.dart

@@ -86,10 +86,9 @@ class Debugger {
   // DevTools is showing an overlay. Both cannot be shown at the same time:
   // bool _pausedOverlayVisible = false;
 
-  String get pauseState =>
-      _pauseModePauseStates.entries
-          .firstWhere((entry) => entry.value == _pauseState)
-          .key;
+  String get pauseState => _pauseModePauseStates.entries
+      .firstWhere((entry) => entry.value == _pauseState)
+      .key;
 
   /// The JS frames at the current paused location.
   ///
@@ -540,17 +539,15 @@ class Debugger {
     final timestamp = DateTime.now().millisecondsSinceEpoch;
     final jsBreakpointIds = e.hitBreakpoints ?? [];
     if (jsBreakpointIds.isNotEmpty) {
-      final breakpointIds =
-          jsBreakpointIds
-              .map((id) => _breakpoints._dartIdByJsId[id])
-              // In case the breakpoint was set in Chrome DevTools outside of
-              // package:dwds.
-              .where((entry) => entry != null)
-              .toSet();
-      final pauseBreakpoints =
-          isolate.breakpoints
-              ?.where((bp) => breakpointIds.contains(bp.id))
-              .toList();
+      final breakpointIds = jsBreakpointIds
+          .map((id) => _breakpoints._dartIdByJsId[id])
+          // In case the breakpoint was set in Chrome DevTools outside of
+          // package:dwds.
+          .where((entry) => entry != null)
+          .toSet();
+      final pauseBreakpoints = isolate.breakpoints
+          ?.where((bp) => breakpointIds.contains(bp.id))
+          .toList();
       event = Event(
         kind: EventKind.kPauseBreakpoint,
         timestamp: timestamp,

dwds/lib/src/debugging/inspector.dart

@@ -145,10 +145,9 @@ abstract class AppInspector {
     ModifiedModuleReport? modifiedModuleReport,
   }) {
     return _scriptCacheMemoizer.runOnce(() async {
-      final scripts =
-          await globalToolConfiguration.loadStrategy
-              .metadataProviderFor(appConnection.request.entrypointPath)
-              .scripts;
+      final scripts = await globalToolConfiguration.loadStrategy
+          .metadataProviderFor(appConnection.request.entrypointPath)
+          .scripts;
       if (modifiedModuleReport != null) {
         // Invalidate any script caches that were computed for the now invalid
         // libraries. They will get repopulated below.

dwds/lib/src/debugging/instance.dart

@@ -267,8 +267,9 @@ class ChromeAppInstanceHelper {
       objectId,
       offset: offset,
       count: count,
-      length:
-          metaData.kind != InstanceKind.kPlainInstance ? metaData.length : null,
+      length: metaData.kind != InstanceKind.kPlainInstance
+          ? metaData.length
+          : null,
     );
 
     final dartProperties = await _dartFieldsFor(properties, remoteObject);
@@ -307,9 +308,8 @@ class ChromeAppInstanceHelper {
     // We do this in in awkward way because we want the keys and values, but we
     // can't return things by value or some Dart objects will come back as
     // values that we need to be RemoteObject, e.g. a List of int.
-    final expression =
-        globalToolConfiguration.loadStrategy.dartRuntimeDebugger
-            .getMapElementsJsExpression();
+    final expression = globalToolConfiguration.loadStrategy.dartRuntimeDebugger
+        .getMapElementsJsExpression();
 
     final keysAndValues = await inspector.jsCallFunctionOn(map, expression, []);
     final keys = await inspector.loadField(keysAndValues, 'keys');
@@ -507,10 +507,12 @@ class ChromeAppInstanceHelper {
 
     // Collect named fields in the requested range.
     // Account for already collected positional fields.
-    final namedRangeOffset =
-        offset == null ? null : _remainingCount(positionalCount, offset);
-    final namedRangeCount =
-        count == null ? null : _remainingCount(positionalRangeCount, count);
+    final namedRangeOffset = offset == null
+        ? null
+        : _remainingCount(positionalCount, offset);
+    final namedRangeCount = count == null
+        ? null
+        : _remainingCount(positionalRangeCount, count);
     final namedInstance = await instanceFor(
       namedObject,
       offset: namedRangeOffset,
@@ -537,9 +539,8 @@ class ChromeAppInstanceHelper {
     // We do this in in awkward way because we want the keys and values, but we
     // can't return things by value or some Dart objects will come back as
     // values that we need to be RemoteObject, e.g. a List of int.
-    final expression =
-        globalToolConfiguration.loadStrategy.dartRuntimeDebugger
-            .getRecordFieldsJsExpression();
+    final expression = globalToolConfiguration.loadStrategy.dartRuntimeDebugger
+        .getRecordFieldsJsExpression();
 
     final result = await inspector.jsCallFunctionOn(record, expression, []);
     final fieldNameElements = await _recordShapeFields(
@@ -676,9 +677,8 @@ class ChromeAppInstanceHelper {
     // We do this in in awkward way because we want the names and types, but we
     // can't return things by value or some Dart objects will come back as
     // values that we need to be RemoteObject, e.g. a List of int.
-    final expression =
-        globalToolConfiguration.loadStrategy.dartRuntimeDebugger
-            .getRecordTypeFieldsJsExpression();
+    final expression = globalToolConfiguration.loadStrategy.dartRuntimeDebugger
+        .getRecordTypeFieldsJsExpression();
 
     final result = await inspector.jsCallFunctionOn(record, expression, []);
     final fieldNameElements = await _recordShapeFields(
@@ -707,9 +707,8 @@ class ChromeAppInstanceHelper {
     final length = metaData.length;
     final objectId = remoteObject.objectId;
     if (objectId == null) return null;
-    final expression =
-        globalToolConfiguration.loadStrategy.dartRuntimeDebugger
-            .getSetElementsJsExpression();
+    final expression = globalToolConfiguration.loadStrategy.dartRuntimeDebugger
+        .getSetElementsJsExpression();
 
     final result = await inspector.jsCallFunctionOn(
       remoteObject,
@@ -812,9 +811,10 @@ class ChromeAppInstanceHelper {
     //
     // For maps and lists it's more complicated. Treat the actual SDK versions
     // of these as special.
-    final fieldNameExpression =
-        globalToolConfiguration.loadStrategy.dartRuntimeDebugger
-            .getObjectFieldNamesJsExpression();
+    final fieldNameExpression = globalToolConfiguration
+        .loadStrategy
+        .dartRuntimeDebugger
+        .getObjectFieldNamesJsExpression();
     final result = await inspector.jsCallFunctionOn(
       remoteObject,
       fieldNameExpression,
@@ -832,10 +832,9 @@ class ChromeAppInstanceHelper {
   /// be something returned by value from Chrome, e.g. number, boolean, or
   /// String.
   Future<InstanceRef?> instanceRefFor(Object value) {
-    final remote =
-        value is RemoteObject
-            ? value
-            : RemoteObject({'value': value, 'type': _chromeType(value)});
+    final remote = value is RemoteObject
+        ? value
+        : RemoteObject({'value': value, 'type': _chromeType(value)});
     return _instanceRefForRemote(remote);
   }
 

dwds/lib/src/debugging/libraries.dart

@@ -85,10 +85,9 @@ class LibraryHelper<T extends AppInspector> {
   /// using [initialize].
   Future<List<LibraryRef>> get libraryRefs async {
     if (_libraryRefsById.isNotEmpty) return _libraryRefsById.values.toList();
-    final libraries =
-        await globalToolConfiguration.loadStrategy
-            .metadataProviderFor(inspector.appConnection.request.entrypointPath)
-            .libraries;
+    final libraries = await globalToolConfiguration.loadStrategy
+        .metadataProviderFor(inspector.appConnection.request.entrypointPath)
+        .libraries;
     for (final library in libraries) {
       _libraryRefsById[library] = _createLibraryRef(library);
     }

dwds/lib/src/debugging/metadata/class.dart

@@ -149,9 +149,10 @@ class ChromeClassMetaDataHelper {
   /// Returns null if the [remoteObject] is not a Dart class.
   Future<ClassMetaData?> metaDataFor(RemoteObject remoteObject) async {
     try {
-      final evalExpression =
-          globalToolConfiguration.loadStrategy.dartRuntimeDebugger
-              .getObjectMetadataJsExpression();
+      final evalExpression = globalToolConfiguration
+          .loadStrategy
+          .dartRuntimeDebugger
+          .getObjectMetadataJsExpression();
 
       final result = await _inspector.jsCallFunctionOn(
         remoteObject,