Skip to content

Commit b0d1a1d

Browse files
bkonyiTytaniumDev
authored andcommitted
Launch DDS from Dart SDK and prepare to serve DevTools from DDS (flutter#146593)
This change is a major step towards moving away from shipping DDS via Pub. The first component of this PR is the move away from importing package:dds to launch DDS. Instead, DDS is launched out of process using the `dart development-service` command shipped with the Dart SDK. This makes Flutter's handling of DDS consistent with the standalone Dart VM. The second component of this PR is the initial work to prepare for the removal of instances of DevTools being served manually by the flutter_tool, instead relying on DDS to serve DevTools. This will be consistent with how the standalone Dart VM serves DevTools, tying the DevTools lifecycle to a live DDS instance. This will allow for the removal of much of the logic needed to properly manage the lifecycle of the DevTools server in a future PR. Also, by serving DevTools from DDS, users will no longer need to forward a secondary port in remote workflows as DevTools will be available on the DDS port. This code is currently commented out and will be enabled in a future PR. There's two remaining circumstances that will prevent us from removing DevtoolsRunner completely: - The daemon's `devtools.serve` endpoint - `flutter drive`'s `--profile-memory` flag used for recording memory profiles This PR also includes some refactoring around `DebuggingOptions` to reduce the number of debugging related arguments being passed as parameters adjacent to a `DebuggingOptions` instance.
1 parent 199b888 commit b0d1a1d

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

57 files changed

+920
-747
lines changed

packages/flutter_tools/lib/src/android/android_device.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ class AndroidDevice extends Device {
5959
this.productID,
6060
required this.modelID,
6161
this.deviceCodeName,
62-
required Logger logger,
62+
required super.logger,
6363
required ProcessManager processManager,
6464
required Platform platform,
6565
required AndroidSdk androidSdk,

0 commit comments

Comments
 (0)