Address review comments

Author: srujzs

dwds/lib/src/services/chrome_proxy_service.dart

@@ -301,9 +301,8 @@ class ChromeProxyService implements VmServiceInterface {
   /// with [destroyIsolate] and recreated with this method there is a hot
   /// restart or full page refresh.
   ///
-  /// If [newConnection] is true, does not recompute metadata information as
-  /// that is done only when we recreate the isolate the connection as source
-  /// data may have changed.
+  /// If [newConnection] is true, this method does not recompute metadata
+  /// information as the metadata couldn't have changed.
   Future<void> createIsolate(
     AppConnection appConnection, {
     bool newConnection = false,

dwds/test/common/hot_restart_common.dart

@@ -378,6 +378,43 @@ void runTests({
       expect(source.contains(newString), isTrue);
     });
 
+    test('can hot restart while paused', () async {
+      final client = context.debugConnection.vmService;
+      var vm = await client.getVM();
+      var isolateId = vm.isolates!.first.id!;
+      await client.streamListen('Debug');
+      final stream = client.onEvent('Debug');
+      final scriptList = await client.getScripts(isolateId);
+      final main = scriptList.scripts!.firstWhere(
+        (script) => script.uri!.contains('main.dart'),
+      );
+      final bpLine = await context.findBreakpointLine(
+        'printCount',
+        isolateId,
+        main,
+      );
+      await client.addBreakpoint(isolateId, main.id!, bpLine);
+      await stream.firstWhere(
+        (event) => event.kind == EventKind.kPauseBreakpoint,
+      );
+
+      await makeEditAndRecompile();
+      final hotRestart = context.getRegisteredServiceExtension('hotRestart');
+      await fakeClient.callServiceExtension(hotRestart!);
+      final source = await context.webDriver.pageSource;
+
+      // Main is re-invoked which shouldn't clear the state.
+      expect(source.contains(originalString), isTrue);
+      expect(source.contains(newString), isTrue);
+
+      vm = await client.getVM();
+      isolateId = vm.isolates!.first.id!;
+      final isolate = await client.getIsolate(isolateId);
+
+      // Previous breakpoint should be cleared.
+      expect(isolate.breakpoints!.isEmpty, isTrue);
+    });
+
     test('can evaluate expressions after hot restart', () async {
       final client = context.debugConnection.vmService;
 

dwds/test/hot_restart_breakpoints_test.dart

@@ -212,7 +212,7 @@ void main() {
       expect(consoleLogs.contains(newLog), true);
     });
 
-    test('add adding line, hot restart, removing line, and hot restart, '
+    test('after adding line, hot restart, removing line, and hot restart, '
         'breakpoint is correct across both hot restarts', () async {
       final genLog = 'main gen0';