Revert "[flutter_tools] Enable fast start by default for Android (#48746)" (#48836)
This reverts commit d5b0b3c8d1c5fd89302089077ccabbcfaae045e4.
diff --git a/packages/flutter_tools/lib/src/commands/run.dart b/packages/flutter_tools/lib/src/commands/run.dart
index 10ae7f4..006d521 100644
--- a/packages/flutter_tools/lib/src/commands/run.dart
+++ b/packages/flutter_tools/lib/src/commands/run.dart
@@ -189,7 +189,8 @@
)
..addFlag('fast-start',
negatable: true,
- defaultsTo: true,
+ defaultsTo: false,
+ hide: true,
help: 'Whether to quickly bootstrap applications with a minimal app. '
'Currently this is only supported on Android devices. This option '
'cannot be paired with --use-application-binary.'
@@ -318,6 +319,10 @@
await super.validateCommand();
}
+ if (boolArg('fast-start') && runningWithPrebuiltApplication) {
+ throwToolExit('--fast-start is not supported with --use-application-binary');
+ }
+
devices = await findAllTargetDevices();
if (devices == null) {
throwToolExit(null);
@@ -360,9 +365,7 @@
vmserviceOutFile: stringArg('vmservice-out-file'),
// Allow forcing fast-start to off to prevent doing more work on devices that
// don't support it.
- fastStart: boolArg('fast-start')
- && !runningWithPrebuiltApplication
- && devices.every((Device device) => device.supportsFastStart),
+ fastStart: boolArg('fast-start') && devices.every((Device device) => device.supportsFastStart),
);
}
}
@@ -425,6 +428,12 @@
}
for (final Device device in devices) {
+ if (!device.supportsFastStart && boolArg('fast-start')) {
+ globals.printStatus(
+ 'Using --fast-start option with device ${device.name}, but this device '
+ 'does not support it. Overriding the setting to false.'
+ );
+ }
if (await device.isLocalEmulator) {
if (await device.supportsHardwareRendering) {
final bool enableSoftwareRendering = boolArg('enable-software-rendering') == true;
diff --git a/packages/flutter_tools/test/commands.shard/hermetic/run_test.dart b/packages/flutter_tools/test/commands.shard/hermetic/run_test.dart
index ccf8b2f..466f4c8 100644
--- a/packages/flutter_tools/test/commands.shard/hermetic/run_test.dart
+++ b/packages/flutter_tools/test/commands.shard/hermetic/run_test.dart
@@ -77,7 +77,7 @@
]);
fail('Expect exception');
} catch (e) {
- expect(e.toString(), isNot(contains('--fast-start is not supported with --use-application-binary')));
+ expect(e.toString(), contains('--fast-start is not supported with --use-application-binary'));
}
}, overrides: <Type, Generator>{
FileSystem: () => MemoryFileSystem(),
@@ -115,10 +115,10 @@
}
final BufferLogger bufferLogger = globals.logger as BufferLogger;
- expect(bufferLogger.statusText, isNot(contains(
+ expect(bufferLogger.statusText, contains(
'Using --fast-start option with device mockdevice, but this device '
'does not support it. Overriding the setting to false.'
- )));
+ ));
}, overrides: <Type, Generator>{
FileSystem: () => MemoryFileSystem(),
ProcessManager: () => FakeProcessManager.any(),