chore: remove more unused code

packages/devtools_app/lib/src/framework/notifications_view.dart

@@ -265,7 +265,6 @@ class _NotificationState extends State<_Notification>
                           Flexible(
                             child: _NotificationMessage(
                               widget: widget,
-                              context: context,
                             ),
                           ),
                           _DismissAction(
@@ -277,7 +276,6 @@ class _NotificationState extends State<_Notification>
                       )
                     : _NotificationMessage(
                         widget: widget,
-                        context: context,
                       ),
                 const SizedBox(height: defaultSpacing),
                 _NotificationActions(widget: widget),
@@ -314,11 +312,9 @@ class _DismissAction extends StatelessWidget {
 class _NotificationMessage extends StatelessWidget {
   const _NotificationMessage({
     required this.widget,
-    required this.context,
   });
 
   final _Notification widget;
-  final BuildContext context;
 
   @override
   Widget build(BuildContext context) {

packages/devtools_app/lib/src/screens/debugger/codeview.dart

@@ -56,8 +56,6 @@ class CodeView extends StatefulWidget {
     this.lineRange,
     this.initialPosition,
     this.onSelected,
-    this.enableFileExplorer = true,
-    this.enableSearch = true,
     this.enableHistory = true,
   }) : super(key: key);
 
@@ -80,8 +78,6 @@ class CodeView extends StatefulWidget {
   // the script's source in its entirety, with lines outside of the range being
   // rendered as if they have been greyed out.
   final LineRange? lineRange;
-  final bool enableFileExplorer;
-  final bool enableSearch;
   final bool enableHistory;
 
   final void Function(ScriptRef scriptRef, int line)? onSelected;

packages/devtools_app/lib/src/screens/debugger/debugger_screen.dart

@@ -90,7 +90,6 @@ class DebuggerScreenBody extends StatefulWidget {
   const DebuggerScreenBody({super.key});
 
   static final codeViewKey = GlobalKey(debugLabel: 'codeViewKey');
-  static final scriptViewKey = GlobalKey(debugLabel: 'scriptViewKey');
   static const callStackCopyButtonKey =
       Key('debugger_call_stack_copy_to_clipboard_button');
 

packages/devtools_app/lib/src/screens/inspector/inspector_controller.dart

@@ -60,10 +60,7 @@ class InspectorController extends DisposableController
     _refreshRateLimiter = RateLimiter(refreshFramesPerSecond, refresh);
 
     inspectorTree.config = InspectorTreeConfig(
-      summaryTree: isSummaryTree,
-      treeType: treeType,
       onNodeAdded: _onNodeAdded,
-      onHover: highlightShowNode,
       onSelectionChange: selectionChanged,
       onExpand: _onExpand,
       onClientActiveChange: _onClientChange,
@@ -495,10 +492,6 @@ class InspectorController extends DisposableController
     details?.setSubtreeRoot(subtreeRoot, subtreeSelection);
   }
 
-  InspectorInstanceRef? getSubtreeRootValue() {
-    return subtreeRoot?.valueRef;
-  }
-
   void setSubtreeRoot(
     RemoteDiagnosticsNode? node,
     RemoteDiagnosticsNode? selection,

packages/devtools_app/lib/src/screens/inspector/inspector_tree_controller.dart

@@ -181,8 +181,6 @@ class InspectorTreeController extends DisposableController
     });
   }
 
-  RemoteDiagnosticsNode? subtreeRoot; // Optional.
-
   InspectorTreeNode? get selection => _selection;
   InspectorTreeNode? _selection;
 

packages/devtools_app/lib/src/screens/logging/logging_controller.dart

@@ -14,7 +14,6 @@ import 'package:path/path.dart' as path;
 import 'package:vm_service/vm_service.dart';
 
 import '../../service/vm_service_wrapper.dart';
-import '../../shared/console/eval/inspector_tree.dart';
 import '../../shared/diagnostics/diagnostics_node.dart';
 import '../../shared/diagnostics/inspector_service.dart';
 import '../../shared/globals.dart';
@@ -63,100 +62,6 @@ Future<String> _retrieveFullStringValue(
       Future.value(fallback);
 }
 
-class LoggingDetailsController {
-  LoggingDetailsController({
-    required this.onShowInspector,
-    required this.onShowDetails,
-    required this.createLoggingTree,
-  });
-
-  static const JsonEncoder jsonEncoder = JsonEncoder.withIndent('  ');
-
-  late LogData data;
-
-  /// Callback to execute to show the inspector.
-  final VoidCallback onShowInspector;
-
-  /// Callback to execute to show the data from the details tree in the view.
-  final OnShowDetails onShowDetails;
-
-  /// Callback to create an inspectorTree for the logging view of the correct
-  /// type.
-  final CreateLoggingTree createLoggingTree;
-
-  InspectorTreeController? tree;
-
-  void setData(LogData data) {
-    this.data = data;
-
-    tree = null;
-
-    if (data.node != null) {
-      tree = createLoggingTree(
-        onSelectionChange: () {
-          final InspectorTreeNode node = tree!.selection!;
-          unawaited(tree!.maybePopulateChildren(node));
-
-          // TODO(jacobr): node.diagnostic.isDiagnosticableValue isn't quite
-          // right.
-          final diagnosticLocal = node.diagnostic!;
-          if (diagnosticLocal.isDiagnosticableValue) {
-            // TODO(jacobr): warn if the selection can't be set as the node is
-            // stale which is likely if this is an old log entry.
-            onShowInspector();
-            unawaited(diagnosticLocal.setSelectionInspector(false));
-          }
-        },
-      );
-
-      final InspectorTreeNode root = tree!.setupInspectorTreeNode(
-        tree!.createNode(),
-        data.node!,
-        expandChildren: true,
-        expandProperties: true,
-      );
-      // No sense in collapsing the root node.
-      root.allowExpandCollapse = false;
-      tree!.root = root;
-      onShowDetails(tree: tree);
-
-      return;
-    }
-
-    // See if we need to asynchronously compute the log entry details.
-    if (data.needsComputing) {
-      onShowDetails(text: '');
-
-      unawaited(
-        data.compute().then((_) {
-          // If we're still displaying the same log entry, then update the UI with
-          // the calculated value.
-          if (this.data == data) {
-            _updateUIFromData();
-          }
-        }),
-      );
-    } else {
-      _updateUIFromData();
-    }
-  }
-
-  void _updateUIFromData() {
-    if (data.details?.startsWith('{') == true &&
-        data.details?.endsWith('}') == true) {
-      try {
-        // If the string decodes properly, then format the json.
-        final result = jsonDecode(data.details!);
-        onShowDetails(text: jsonEncoder.convert(result));
-      } catch (e) {
-        onShowDetails(text: data.details);
-      }
-    } else {
-      onShowDetails(text: data.details);
-    }
-  }
-}
-
 class LoggingController extends DisposableController
     with
         SearchControllerMixin<LogData>,
@@ -809,22 +714,6 @@ class LogData with SearchableDataMixin {
     }
   }
 
-  bool matchesFilter(String filter) {
-    if (kind.toLowerCase().contains(filter)) {
-      return true;
-    }
-
-    if (summary != null && summary!.toLowerCase().contains(filter)) {
-      return true;
-    }
-
-    if (_details != null && _details!.toLowerCase().contains(filter)) {
-      return true;
-    }
-
-    return false;
-  }
-
   @override
   bool matchesSearchToken(RegExp regExpSearch) {
     return (summary?.caseInsensitiveContains(regExpSearch) == true) ||
@@ -836,23 +725,19 @@ class LogData with SearchableDataMixin {
 }
 
 class FrameInfo {
-  FrameInfo(this.number, this.elapsedMs, this.startTimeMs);
+  FrameInfo(this.number, this.elapsedMs);
 
   static const String eventName = 'Flutter.Frame';
 
-  static const double kTargetMaxFrameTimeMs = 1000.0 / 60;
-
   static FrameInfo from(Map<String, dynamic> data) {
     return FrameInfo(
       data['number'],
       data['elapsed'] / 1000,
-      data['startTime'] / 1000,
     );
   }
 
   final int? number;
   final num? elapsedMs;
-  final num? startTimeMs;
 
   @override
   String toString() => 'frame $number ${elapsedMs!.toStringAsFixed(1)}ms';

packages/devtools_app/lib/src/screens/memory/framework/connected/memory_controller.dart

@@ -108,19 +108,11 @@ class MemoryController extends DisposableController
 
   final _refreshCharts = ValueNotifier<int>(0);
 
-  void refreshAllCharts() {
-    _refreshCharts.value++;
-    _updateAndroidChartVisibility();
-  }
-
   /// Default is to display default tick width based on width of chart of the collected
   /// data in the chart.
   final _displayIntervalNotifier =
       ValueNotifier<ChartInterval>(ChartInterval.theDefault);
 
-  ValueListenable<ChartInterval> get displayIntervalNotifier =>
-      _displayIntervalNotifier;
-
   set displayInterval(ChartInterval interval) {
     _displayIntervalNotifier.value = interval;
   }

packages/devtools_app/lib/src/screens/memory/panes/diff/controller/diff_pane_controller.dart

@@ -42,10 +42,6 @@ class DiffPaneController extends DisposableController {
   /// informational item.
   bool get hasSnapshots => core.snapshots.value.length > 1;
 
-  // This value should never be reset. It is incremented for every snapshot that
-  // is taken, and is used to assign a unique id to each [SnapshotListItem].
-  int _snapshotId = 0;
-
   Future<void> takeSnapshot() async {
     _isTakingSnapshot.value = true;
     ga.select(
@@ -54,7 +50,6 @@ class DiffPaneController extends DisposableController {
     );
 
     final item = SnapshotInstanceItem(
-      id: _snapshotId++,
       displayNumber: _nextDisplayNumber(),
       isolateName: selectedIsolateName ?? '<isolate-not-detected>',
     );

packages/devtools_app/lib/src/screens/memory/panes/diff/controller/item_controller.dart

@@ -33,13 +33,10 @@ class SnapshotInstanceItem extends SnapshotItem {
   SnapshotInstanceItem({
     required this.displayNumber,
     required this.isolateName,
-    required this.id,
   }) {
     _isProcessing.value = true;
   }
 
-  final int id;
-
   final String isolateName;
 
   AdaptedHeap? heap;

packages/devtools_app/lib/src/screens/memory/panes/diff/widgets/instances.dart

@@ -42,7 +42,6 @@ class HeapInstanceTableCell extends StatelessWidget {
   Widget build(BuildContext context) {
     return InstanceViewWithContextMenu(
       count: _count,
-      gaContext: gaContext,
       menuBuilder: () => _buildHeapInstancesMenu(
         sampler: _sampleObtainer,
         liveItemsEnabled: liveItemsEnabled,

packages/devtools_app/lib/src/screens/memory/panes/leaks/controller.dart

@@ -80,7 +80,7 @@ class LeaksPaneController {
     List<LeakReport> reports,
   ) async {
     final graph = (await snapshotMemoryInSelectedIsolate())!;
-    return NotGCedAnalyzerTask.fromSnapshot(graph, reports, selectedIsolateId!);
+    return NotGCedAnalyzerTask.fromSnapshot(graph, reports);
   }
 
   Future<void> requestLeaksAndSaveToYaml() async {

packages/devtools_app/lib/src/screens/memory/panes/leaks/diagnostics/model.dart

@@ -34,10 +34,9 @@ class NotGCedAnalyzerTask {
   static Future<NotGCedAnalyzerTask> fromSnapshot(
     HeapSnapshotGraph graph,
     List<LeakReport> reports,
-    String isolateId,
   ) async {
     return NotGCedAnalyzerTask(
-      heap: await AdaptedHeapData.fromHeapSnapshot(graph, isolateId: isolateId),
+      heap: await AdaptedHeapData.fromHeapSnapshot(graph),
       reports: reports,
     );
   }

packages/devtools_app/lib/src/screens/memory/panes/profile/instances.dart

@@ -33,7 +33,6 @@ class ProfileInstanceTableCell extends StatelessWidget {
   Widget build(BuildContext context) {
     return InstanceViewWithContextMenu(
       count: count,
-      gaContext: gaContext,
       menuBuilder: _shouldShowMenu ? _buildHeapInstancesMenu : null,
     );
   }

packages/devtools_app/lib/src/screens/memory/shared/heap/model.dart

@@ -109,22 +109,18 @@ class SnapshotTaker {
   Future<AdaptedHeapData?> take() async {
     final snapshot = await snapshotMemoryInSelectedIsolate();
     if (snapshot == null) return null;
-    final result =
-        await _adaptSnapshotGaWrapper(snapshot, isolateId: selectedIsolateId!);
+    final result = await _adaptSnapshotGaWrapper(snapshot);
     return result;
   }
 }
 
-Future<AdaptedHeapData> _adaptSnapshotGaWrapper(
-  HeapSnapshotGraph graph, {
-  required String isolateId,
-}) async {
+Future<AdaptedHeapData> _adaptSnapshotGaWrapper(HeapSnapshotGraph graph) async {
   late final AdaptedHeapData result;
   await ga.timeAsync(
     gac.memory,
     gac.MemoryTime.adaptSnapshot,
-    asyncOperation: () async => result =
-        await AdaptedHeapData.fromHeapSnapshot(graph, isolateId: isolateId),
+    asyncOperation: () async =>
+        result = await AdaptedHeapData.fromHeapSnapshot(graph),
     screenMetricsProvider: () => MemoryScreenMetrics(
       heapObjectsTotal: graph.objects.length,
     ),

packages/devtools_app/lib/src/screens/memory/shared/primitives/instance_context_menu.dart

@@ -4,7 +4,6 @@
 
 import 'package:flutter/material.dart';
 
-import '../../../../shared/analytics/constants.dart';
 import '../../../../shared/common_widgets.dart';
 import '../../../../shared/primitives/utils.dart';
 
@@ -15,14 +14,11 @@ class InstanceViewWithContextMenu extends StatelessWidget {
   const InstanceViewWithContextMenu({
     super.key,
     required this.count,
-    required this.gaContext,
     required this.menuBuilder,
   }) : assert(count >= 0);
 
   final int count;
   final MenuBuilder? menuBuilder;
-  final MemoryAreas gaContext;
-
   @override
   Widget build(BuildContext context) {
     final menu = menuBuilder?.call() ?? [];