Unverified Commit e48b7e99 authored by Michael Goderbauer's avatar Michael Goderbauer Committed by GitHub

More word substiturions (#59497)

parent 4173881b
...@@ -644,8 +644,8 @@ class AndroidDevice extends Device { ...@@ -644,8 +644,8 @@ class AndroidDevice extends Device {
...<String>['--ez', 'skia-deterministic-rendering', 'true'], ...<String>['--ez', 'skia-deterministic-rendering', 'true'],
if (debuggingOptions.traceSkia) if (debuggingOptions.traceSkia)
...<String>['--ez', 'trace-skia', 'true'], ...<String>['--ez', 'trace-skia', 'true'],
if (debuggingOptions.traceWhitelist != null) if (debuggingOptions.traceAllowlist != null)
...<String>['--ez', 'trace-whitelist', debuggingOptions.traceWhitelist], ...<String>['--ez', 'trace-allowlist', debuggingOptions.traceAllowlist],
if (debuggingOptions.traceSystrace) if (debuggingOptions.traceSystrace)
...<String>['--ez', 'trace-systrace', 'true'], ...<String>['--ez', 'trace-systrace', 'true'],
if (debuggingOptions.endlessTraceBuffer) if (debuggingOptions.endlessTraceBuffer)
......
...@@ -107,8 +107,14 @@ class RunCommand extends RunCommandBase { ...@@ -107,8 +107,14 @@ class RunCommand extends RunCommandBase {
'By default, Flutter will not log skia code.', 'By default, Flutter will not log skia code.',
) )
..addOption('trace-whitelist', ..addOption('trace-whitelist',
hide: true,
help: '(deprecated) Use --trace-allowlist instead',
valueHelp: 'foo,bar',
)
..addOption('trace-allowlist',
hide: true,
help: 'Filters out all trace events except those that are specified in ' help: 'Filters out all trace events except those that are specified in '
'this comma separated list of whitelisted prefixes.', 'this comma separated list of allowed prefixes.',
valueHelp: 'foo,bar', valueHelp: 'foo,bar',
) )
..addFlag('endless-trace-buffer', ..addFlag('endless-trace-buffer',
...@@ -147,8 +153,8 @@ class RunCommand extends RunCommandBase { ...@@ -147,8 +153,8 @@ class RunCommand extends RunCommandBase {
hide: !verboseHelp, hide: !verboseHelp,
help: 'Pass a list of comma separated flags to the Dart instance at ' help: 'Pass a list of comma separated flags to the Dart instance at '
'application startup. Flags passed through this option must be ' 'application startup. Flags passed through this option must be '
'present on the whitelist defined within the Flutter engine. If ' 'present on the allowlist defined within the Flutter engine. If '
'a non-whitelisted flag is encountered, the process will be ' 'a disallowed flag is encountered, the process will be '
'terminated immediately.\n\n' 'terminated immediately.\n\n'
'This flag is not available on the stable channel and is only ' 'This flag is not available on the stable channel and is only '
'applied in debug and profile modes. This option should only ' 'applied in debug and profile modes. This option should only '
...@@ -348,6 +354,14 @@ class RunCommand extends RunCommandBase { ...@@ -348,6 +354,14 @@ class RunCommand extends RunCommandBase {
} }
} }
String get _traceAllowlist {
final String deprecatedValue = stringArg('trace-whitelist');
if (deprecatedValue != null) {
globals.printError('--trace-whitelist has been deprecated, use --trace-allowlist instead');
}
return stringArg('trace-allowlist') ?? deprecatedValue;
}
DebuggingOptions _createDebuggingOptions() { DebuggingOptions _createDebuggingOptions() {
final BuildInfo buildInfo = getBuildInfo(); final BuildInfo buildInfo = getBuildInfo();
final int browserDebugPort = featureFlags.isWebEnabled && argResults.wasParsed('web-browser-debug-port') final int browserDebugPort = featureFlags.isWebEnabled && argResults.wasParsed('web-browser-debug-port')
...@@ -374,7 +388,7 @@ class RunCommand extends RunCommandBase { ...@@ -374,7 +388,7 @@ class RunCommand extends RunCommandBase {
enableSoftwareRendering: boolArg('enable-software-rendering'), enableSoftwareRendering: boolArg('enable-software-rendering'),
skiaDeterministicRendering: boolArg('skia-deterministic-rendering'), skiaDeterministicRendering: boolArg('skia-deterministic-rendering'),
traceSkia: boolArg('trace-skia'), traceSkia: boolArg('trace-skia'),
traceWhitelist: stringArg('trace-whitelist'), traceAllowlist: _traceAllowlist,
traceSystrace: boolArg('trace-systrace'), traceSystrace: boolArg('trace-systrace'),
endlessTraceBuffer: boolArg('endless-trace-buffer'), endlessTraceBuffer: boolArg('endless-trace-buffer'),
dumpSkpOnShaderCompilation: dumpSkpOnShaderCompilation, dumpSkpOnShaderCompilation: dumpSkpOnShaderCompilation,
......
...@@ -646,7 +646,7 @@ class DebuggingOptions { ...@@ -646,7 +646,7 @@ class DebuggingOptions {
this.enableSoftwareRendering = false, this.enableSoftwareRendering = false,
this.skiaDeterministicRendering = false, this.skiaDeterministicRendering = false,
this.traceSkia = false, this.traceSkia = false,
this.traceWhitelist, this.traceAllowlist,
this.traceSystrace = false, this.traceSystrace = false,
this.endlessTraceBuffer = false, this.endlessTraceBuffer = false,
this.dumpSkpOnShaderCompilation = false, this.dumpSkpOnShaderCompilation = false,
...@@ -676,7 +676,7 @@ class DebuggingOptions { ...@@ -676,7 +676,7 @@ class DebuggingOptions {
this.webRunHeadless = false, this.webRunHeadless = false,
this.webBrowserDebugPort, this.webBrowserDebugPort,
this.cacheSkSL = false, this.cacheSkSL = false,
this.traceWhitelist, this.traceAllowlist,
}) : debuggingEnabled = false, }) : debuggingEnabled = false,
useTestFonts = false, useTestFonts = false,
startPaused = false, startPaused = false,
...@@ -704,7 +704,7 @@ class DebuggingOptions { ...@@ -704,7 +704,7 @@ class DebuggingOptions {
final bool enableSoftwareRendering; final bool enableSoftwareRendering;
final bool skiaDeterministicRendering; final bool skiaDeterministicRendering;
final bool traceSkia; final bool traceSkia;
final String traceWhitelist; final String traceAllowlist;
final bool traceSystrace; final bool traceSystrace;
final bool endlessTraceBuffer; final bool endlessTraceBuffer;
final bool dumpSkpOnShaderCompilation; final bool dumpSkpOnShaderCompilation;
......
...@@ -380,7 +380,7 @@ class IOSDevice extends Device { ...@@ -380,7 +380,7 @@ class IOSDevice extends Device {
if (debuggingOptions.enableSoftwareRendering) '--enable-software-rendering', if (debuggingOptions.enableSoftwareRendering) '--enable-software-rendering',
if (debuggingOptions.skiaDeterministicRendering) '--skia-deterministic-rendering', if (debuggingOptions.skiaDeterministicRendering) '--skia-deterministic-rendering',
if (debuggingOptions.traceSkia) '--trace-skia', if (debuggingOptions.traceSkia) '--trace-skia',
if (debuggingOptions.traceWhitelist != null) '--trace-whitelist="${debuggingOptions.traceWhitelist}"', if (debuggingOptions.traceAllowlist != null) '--trace-allowlist="${debuggingOptions.traceAllowlist}"',
if (debuggingOptions.endlessTraceBuffer) '--endless-trace-buffer', if (debuggingOptions.endlessTraceBuffer) '--endless-trace-buffer',
if (debuggingOptions.dumpSkpOnShaderCompilation) '--dump-skp-on-shader-compilation', if (debuggingOptions.dumpSkpOnShaderCompilation) '--dump-skp-on-shader-compilation',
if (debuggingOptions.verboseSystemLogs) '--verbose-logging', if (debuggingOptions.verboseSystemLogs) '--verbose-logging',
......
...@@ -422,7 +422,7 @@ class IOSSimulator extends Device { ...@@ -422,7 +422,7 @@ class IOSSimulator extends Device {
if (debuggingOptions.disableServiceAuthCodes) '--disable-service-auth-codes', if (debuggingOptions.disableServiceAuthCodes) '--disable-service-auth-codes',
if (debuggingOptions.skiaDeterministicRendering) '--skia-deterministic-rendering', if (debuggingOptions.skiaDeterministicRendering) '--skia-deterministic-rendering',
if (debuggingOptions.useTestFonts) '--use-test-fonts', if (debuggingOptions.useTestFonts) '--use-test-fonts',
if (debuggingOptions.traceWhitelist != null) '--trace-whitelist="${debuggingOptions.traceWhitelist}"', if (debuggingOptions.traceAllowlist != null) '--trace-allowlist="${debuggingOptions.traceAllowlist}"',
'--observatory-port=${debuggingOptions.hostVmServicePort ?? 0}', '--observatory-port=${debuggingOptions.hostVmServicePort ?? 0}',
], ],
]; ];
......
...@@ -244,7 +244,7 @@ void main() { ...@@ -244,7 +244,7 @@ void main() {
'--ez', 'enable-software-rendering', 'true', '--ez', 'enable-software-rendering', 'true',
'--ez', 'skia-deterministic-rendering', 'true', '--ez', 'skia-deterministic-rendering', 'true',
'--ez', 'trace-skia', 'true', '--ez', 'trace-skia', 'true',
'--ez', 'trace-whitelist', 'bar,baz', '--ez', 'trace-allowlist', 'bar,baz',
'--ez', 'trace-systrace', 'true', '--ez', 'trace-systrace', 'true',
'--ez', 'endless-trace-buffer', 'true', '--ez', 'endless-trace-buffer', 'true',
'--ez', 'dump-skp-on-shader-compilation', 'true', '--ez', 'dump-skp-on-shader-compilation', 'true',
...@@ -272,7 +272,7 @@ void main() { ...@@ -272,7 +272,7 @@ void main() {
enableSoftwareRendering: true, enableSoftwareRendering: true,
skiaDeterministicRendering: true, skiaDeterministicRendering: true,
traceSkia: true, traceSkia: true,
traceWhitelist: 'bar,baz', traceAllowlist: 'bar,baz',
traceSystrace: true, traceSystrace: true,
endlessTraceBuffer: true, endlessTraceBuffer: true,
dumpSkpOnShaderCompilation: true, dumpSkpOnShaderCompilation: true,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment