Revert "[flutter_tools] default tree-shake-icons to enabled and improve...

Revert "[flutter_tools] default tree-shake-icons to enabled and improve performance (#54923)" (#55413)

This reverts commit 0b93a921.

dev/benchmarks/macrobenchmarks/pubspec.yaml

@@ -16,7 +16,6 @@ dependencies:
   # and run
   #   flutter update-packages --force-upgrade
   flutter_gallery_assets: 0.1.9+2
-  cupertino_icons: 0.1.3
 
   archive: 2.0.13 # THIS LINE IS AUTOGENERATED - TO UPDATE USE "flutter update-packages --force-upgrade"
   args: 1.6.0 # THIS LINE IS AUTOGENERATED - TO UPDATE USE "flutter update-packages --force-upgrade"
@@ -92,4 +91,4 @@ flutter:
   - packages/flutter_gallery_assets/food/cherry_pie.png
   - assets/999x1000.png
 
-# PUBSPEC CHECKSUM: 49fa
+# PUBSPEC CHECKSUM: 3a55

packages/flutter_tools/lib/src/build_system/targets/assets.dart

@@ -60,9 +60,9 @@ Future<Depfile> copyAssets(Environment environment, Directory outputDirectory) a
         file.parent.createSync(recursive: true);
         final DevFSContent content = entry.value;
         if (content is DevFSFileContent && content.file is File) {
-          inputs.add(content.file as File);
+          inputs.add(globals.fs.file(content.file.path));
           if (!await iconTreeShaker.subsetFont(
-            input: content.file as File,
+            inputPath: content.file.path,
             outputPath: file.path,
             relativePath: entry.key,
           )) {

packages/flutter_tools/lib/src/build_system/targets/icon_tree_shaker.dart

@@ -4,7 +4,6 @@
 
 import 'package:meta/meta.dart';
 import 'package:process/process.dart';
-import 'package:mime/mime.dart' as mime;
 
 import '../../artifacts.dart';
 import '../../base/common.dart';
@@ -21,7 +20,7 @@ import 'dart.dart';
 const String kIconTreeShakerFlag = 'TreeShakeIcons';
 
 /// Whether icon font subsetting is enabled by default.
-const bool kIconTreeShakerEnabledDefault = true;
+const bool kIconTreeShakerEnabledDefault = false;
 
 List<Map<String, dynamic>> _getList(dynamic object, String errorMessage) {
   try {
@@ -67,12 +66,6 @@ class IconTreeShaker {
     }
   }
 
-  /// The MIME type for ttf fonts.
-  static const Set<String> kTtfMimeTypes = <String>{
-    'font/ttf', // based on internet search
-    'application/x-font-ttf', // based on running locally.
-  };
-
   /// The [Source] inputs that targets using this should depend on.
   ///
   /// See [Target.inputs].
@@ -84,7 +77,6 @@ class IconTreeShaker {
 
   final Environment _environment;
   final String _fontManifest;
-  Future<void> _iconDataProcessing;
   Map<String, _IconTreeShakerData> _iconData;
 
   final ProcessManager _processManager;
@@ -97,10 +89,10 @@ class IconTreeShaker {
                    && _environment.defines[kIconTreeShakerFlag] == 'true'
                    && _environment.defines[kBuildMode] != 'debug';
 
-  // Fills the [_iconData] map.
-  Future<void> _getIconData(Environment environment) async {
+  /// Fills the [_iconData] map.
+  Future<Map<String, _IconTreeShakerData>> _getIconData(Environment environment) async {
     if (!enabled) {
-      return;
+      return null;
     }
 
     final File appDill = environment.buildDir.childFile('app.dill');
@@ -143,11 +135,13 @@ class IconTreeShaker {
         codePoints: iconData[entry.key],
       );
     }
-    _iconData = result;
+    return result;
   }
 
-  /// Calls font-subset, which transforms the [input] font file to a
-  /// subsetted version at [outputPath].
+  /// Calls font-subset, which transforms the `inputPath` font file to a
+  /// subsetted version at `outputPath`.
+  ///
+  /// The `relativePath` parameter
   ///
   /// All parameters are required.
   ///
@@ -156,24 +150,15 @@ class IconTreeShaker {
   /// If the font-subset subprocess fails, it will [throwToolExit].
   /// Otherwise, it will return true.
   Future<bool> subsetFont({
-    @required File input,
+    @required String inputPath,
     @required String outputPath,
     @required String relativePath,
   }) async {
     if (!enabled) {
       return false;
     }
-    if (input.lengthSync() < 12) {
-      return false;
-    }
-    final String mimeType = mime.lookupMimeType(
-      input.path,
-      headerBytes: await input.openRead(0, 12).first,
-    );
-    if (!kTtfMimeTypes.contains(mimeType)) {
-      return false;
-    }
-    await (_iconDataProcessing ??= _getIconData(_environment));
+
+    _iconData ??= await _getIconData(_environment);
     assert(_iconData != null);
 
     final _IconTreeShakerData iconTreeShakerData = _iconData[relativePath];
@@ -191,7 +176,7 @@ class IconTreeShaker {
     final List<String> cmd = <String>[
       fontSubset.path,
       outputPath,
-      input.path,
+      inputPath,
     ];
     final String codePoints = iconTreeShakerData.codePoints.join(' ');
     _logger.printTrace('Running font-subset: ${cmd.join(' ')}, '
@@ -201,7 +186,9 @@ class IconTreeShaker {
       fontSubsetProcess.stdin.writeln(codePoints);
       await fontSubsetProcess.stdin.flush();
       await fontSubsetProcess.stdin.close();
-    } on Exception {
+    } on Exception catch (_) {
+      // handled by checking the exit code.
+    } on OSError catch (_) {  // ignore: dead_code_on_catch_subtype
       // handled by checking the exit code.
     }
 

packages/flutter_tools/test/commands.shard/permeable/build_bundle_test.dart

@@ -219,7 +219,7 @@ void main() {
         kBuildMode: 'debug',
         kTargetPlatform: 'android-arm',
         kTrackWidgetCreation: 'true',
-        kIconTreeShakerFlag: 'true',
+        kIconTreeShakerFlag: null,
       });
 
       return BuildResult(success: true);

packages/flutter_tools/test/general.shard/commands/build_aot_test.dart

@@ -110,7 +110,7 @@ void main() {
 
   testUsingContext('build aot outputs timing info', () async {
     globals.fs
-      .file('.dart_tool/flutter_build/3f206b606f73e08587a94405f2e86fad/app.so')
+      .file('.dart_tool/flutter_build/0c21fd4ab3b8bde8b385ff01d08e0093/app.so')
       .createSync(recursive: true);
     when(globals.buildSystem.build(any, any))
       .thenAnswer((Invocation invocation) async {

packages/flutter_tools/test/general.shard/commands/build_apk_test.dart

@@ -238,7 +238,6 @@ void main() {
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=true',
           '-Pshrink=true',
-          '-Ptree-shake-icons=true',
           'assembleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -268,7 +267,6 @@ void main() {
           '-Ptrack-widget-creation=true',
           '-Pshrink=true',
           '-Psplit-debug-info=${tempDir.path}',
-          '-Ptree-shake-icons=true',
           'assembleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -301,7 +299,6 @@ void main() {
           '-Ptrack-widget-creation=true',
           '-Pextra-front-end-options=foo,bar',
           '-Pshrink=true',
-          '-Ptree-shake-icons=true',
           'assembleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -332,7 +329,6 @@ void main() {
           '-Ptarget-platform=android-arm,android-arm64,android-x64',
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=true',
-          '-Ptree-shake-icons=true',
           'assembleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -357,7 +353,6 @@ void main() {
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=true',
           '-Pshrink=true',
-          '-Ptree-shake-icons=true',
           'assembleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -419,7 +414,6 @@ void main() {
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=true',
           '-Pshrink=true',
-          '-Ptree-shake-icons=true',
           'assembleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -471,7 +465,6 @@ void main() {
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=true',
           '-Pshrink=true',
-          '-Ptree-shake-icons=true',
           'assembleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),

packages/flutter_tools/test/general.shard/commands/build_appbundle_test.dart

@@ -225,7 +225,6 @@ void main() {
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=false',
           '-Pshrink=true',
-          '-Ptree-shake-icons=true',
           'bundleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -258,7 +257,6 @@ void main() {
           '-Ptarget-platform=android-arm,android-arm64,android-x64',
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=false',
-          '-Ptree-shake-icons=true',
           'bundleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -283,7 +281,6 @@ void main() {
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=false',
           '-Pshrink=true',
-          '-Ptree-shake-icons=true',
           'bundleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -345,7 +342,6 @@ void main() {
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=false',
           '-Pshrink=true',
-          '-Ptree-shake-icons=true',
           'assembleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),
@@ -397,7 +393,6 @@ void main() {
           '-Ptarget=${globals.fs.path.join(tempDir.path, 'flutter_project', 'lib', 'main.dart')}',
           '-Ptrack-widget-creation=false',
           '-Pshrink=true',
-          '-Ptree-shake-icons=true',
           'assembleRelease',
         ],
         workingDirectory: anyNamed('workingDirectory'),