flutter/packages
1
0
Fork 0
mirror of https://github.com/flutter/packages.git synced 2025-08-24 03:18:54 +08:00
Code Issues Packages Projects Releases Wiki Activity

[tool] Adds --wasm flag to the drive-examples command (#7162)

Browse Source 
I was unsure if we wanted to add a new platform option for this or add a `--wasm` flag to the command. Adding a `--wasm` flag seems more complicated since we would probably want to add extra handling around proper use of the flag, whereas adding a `web-wasm` platform should require no additional checks.

This is in preparation for https://github.com/flutter/flutter/issues/151664
This commit is contained in:
Rexios
2024-07-25 19:30:57 -04:00
committed by GitHub
parent 50238e7dd3
commit 0489bdac9f
3 changed files with 89 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
script/tool/lib/src/drive_examples_command.dart
View File

@ -13,7 +13,7 @@ import 'common/package_looping_command.dart';
import 'common/plugin_utils.dart';
import 'common/repository_package.dart';
const int _exitNoPlatformFlags = 2;
const int _exitInvalidArgs = 2;
const int _exitNoAvailableDevice = 3;
// From https://flutter.dev/to/integration-test-on-web
@ -40,6 +40,8 @@ class DriveExamplesCommand extends PackageLoopingCommand {
help: 'Runs the web implementation of the examples');
argParser.addFlag(platformWindows,
help: 'Runs the Windows implementation of the examples');
argParser.addFlag(kWebWasmFlag,
help: 'Compile to WebAssembly rather than JavaScript');
argParser.addOption(
kEnableExperiment,
defaultsTo: '',
@ -84,7 +86,7 @@ class DriveExamplesCommand extends PackageLoopingCommand {
printError(
'Exactly one of ${platformSwitches.map((String platform) => '--$platform').join(', ')} '
'must be specified.');
throw ToolExit(_exitNoPlatformFlags);
throw ToolExit(_exitInvalidArgs);
}
String? androidDevice;
@ -107,22 +109,31 @@ class DriveExamplesCommand extends PackageLoopingCommand {
iOSDevice = devices.first;
}
final bool useWasm = getBoolArg(kWebWasmFlag);
final bool hasPlatformWeb = getBoolArg(platformWeb);
if (useWasm && !hasPlatformWeb) {
printError('--wasm is only supported on the web platform');
throw ToolExit(_exitInvalidArgs);
}
_targetDeviceFlags = <String, List<String>>{
if (getBoolArg(platformAndroid))
platformAndroid: <String>['-d', androidDevice!],
if (getBoolArg(platformIOS)) platformIOS: <String>['-d', iOSDevice!],
if (getBoolArg(platformLinux)) platformLinux: <String>['-d', 'linux'],
if (getBoolArg(platformMacOS)) platformMacOS: <String>['-d', 'macos'],
if (getBoolArg(platformWeb))
if (hasPlatformWeb)
platformWeb: <String>[
'-d',
'web-server',
'--web-port=7357',
'--browser-name=chrome',
if (useWasm)
'--wasm'
// TODO(dit): Clean this up, https://github.com/flutter/flutter/issues/151869
if (platform.environment['CHANNEL']?.toLowerCase() == 'master')
'--web-renderer=canvaskit',
if (platform.environment['CHANNEL']?.toLowerCase() != 'master')
else if (platform.environment['CHANNEL']?.toLowerCase() == 'master')
'--web-renderer=canvaskit'
else
'--web-renderer=html',
if (platform.environment.containsKey('CHROME_EXECUTABLE'))
'--chrome-binary=${platform.environment['CHROME_EXECUTABLE']}',