Render the "Import File" button.

Author: tugorez

packages/devtools_app/lib/src/app_size/file_import_container.dart

@@ -2,7 +2,10 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+import 'dart:convert';
 import 'package:flutter/material.dart';
+import 'package:flutter/foundation.dart';
+import 'package:file_selector/file_selector.dart';
 
 import '../common_widgets.dart';
 import '../config_specific/drag_and_drop/drag_and_drop.dart';
@@ -118,8 +121,7 @@ class _FileImportContainerState extends State<FileImportContainer> {
             child: _buildImportedFileDisplay(),
           ),
         ),
-        // TODO(kenz): uncomment once file picker support is added
-        // _buildImportButton(),
+        if (_shouldShowImportButton) _buildImportButton(),
         // Horizontal spacer with flex value of 1.
         const Flexible(
           child: SizedBox(height: rowHeight),
@@ -138,18 +140,33 @@ class _FileImportContainerState extends State<FileImportContainer> {
     );
   }
 
-// TODO(kenz): uncomment once file picker support is added
-//  Widget _buildImportButton() {
-//    return Row(
-//      mainAxisAlignment: MainAxisAlignment.center,
-//      children: [
-//        OutlinedButton(
-//          onPressed: () {},
-//          child: const MaterialIconLabel(Icons.file_upload, 'Import File'),
-//        ),
-//      ],
-//    );
-//  }
+  Widget _buildImportButton() {
+    return Row(
+      mainAxisAlignment: MainAxisAlignment.center,
+      children: [
+        OutlinedButton(
+          onPressed: () async {
+            final acceptedTypeGroups = [
+              XTypeGroup(extensions: ['.json']),
+            ];
+            final file = await openFile(acceptedTypeGroups: acceptedTypeGroups);
+            final json = jsonDecode(await file.readAsString());
+            final devToolsJsonFile = DevToolsJsonFile(
+              name: file.name,
+              lastModifiedTime: await file.lastModified(),
+              data: json,
+            );
+            _handleImportedFile(devToolsJsonFile);
+          },
+          child: const MaterialIconLabel(Icons.file_upload, 'Import File'),
+        ),
+      ],
+    );
+  }
+
+  // Only show the import button in web and macOS
+  bool get _shouldShowImportButton =>
+      kIsWeb || defaultTargetPlatform == TargetPlatform.macOS;
 
   Widget _buildActionButton() {
     return Column(

packages/devtools_app/pubspec.lock

@@ -176,6 +176,13 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "0.14.2"
+  cross_file:
+    dependency: transitive
+    description:
+      name: cross_file
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "0.1.0"
   crypto:
     dependency: transitive
     description:
@@ -232,6 +239,41 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "5.2.1"
+  file_selector:
+    dependency: "direct main"
+    description:
+      name: file_selector
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "0.7.0+2"
+  file_selector_linux:
+    dependency: "direct main"
+    description:
+      name: file_selector_linux
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "0.0.1"
+  file_selector_macos:
+    dependency: "direct main"
+    description:
+      name: file_selector_macos
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "0.0.1"
+  file_selector_platform_interface:
+    dependency: transitive
+    description:
+      name: file_selector_platform_interface
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "1.0.2"
+  file_selector_web:
+    dependency: "direct main"
+    description:
+      name: file_selector_web
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "0.7.0+1"
   fixnum:
     dependency: transitive
     description:
@@ -408,6 +450,13 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.10.0-nullsafety.3"
+  platform_detect:
+    dependency: transitive
+    description:
+      name: platform_detect
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "1.4.0"
   plugin_platform_interface:
     dependency: transitive
     description:

packages/devtools_app/pubspec.yaml

@@ -11,23 +11,26 @@ version: 0.9.6
 homepage: https://github.com/flutter/devtools
 
 environment:
-  sdk: '>=2.10.0 <3.0.0'
+  sdk: ">=2.10.0 <3.0.0"
   # The flutter desktop support interacts with build scripts on the Flutter
   # side that are not yet stable, so it requires a very recent version of
   # Flutter. This version will increase regularly as the build scripts change.
-  flutter: '>=1.10.0'
+  flutter: ">=1.10.0"
 
 dependencies:
-  ansi_up:
-    ^0.0.2
+  ansi_up: ^0.0.2
   #     path: ../../third_party/packages/ansi_up
   # Pin ansicolor to version before pre-NNBD version 1.1.0, should be ^1.0.5
-  # See https://github.com/flutter/devtools/issues/2530 
+  # See https://github.com/flutter/devtools/issues/2530
   ansicolor: 1.0.5
-  async: ^2.0.0 
+  async: ^2.0.0
   collection: ^1.15.0-nnbd
   devtools_shared: 0.9.6
   file: ^5.1.0
+  file_selector: ^0.7.0
+  file_selector_macos: ^0.0.1
+  file_selector_linux: ^0.0.1
+  file_selector_web: ^0.7.0
   flutter:
     sdk: flutter
   flutter_web_plugins:
@@ -59,8 +62,7 @@ dev_dependencies:
   # (temporary home for canonical lints)
   simple_lint_rules: ^1.0.0
   test: any # This version is pinned by Flutter so we don't need to set one explicitly.
-  webkit_inspection_protocol: '>=0.5.0 <0.8.0'
-
+  webkit_inspection_protocol: ">=0.5.0 <0.8.0"
 
 flutter:
   uses-material-design: true
@@ -113,12 +115,12 @@ flutter:
         - asset: fonts/Octicons.ttf
 
 dependency_overrides:
-# The "#OVERRIDE_FOR_DEVELOPMENT" lines are stripped out when we publish.
-# All overriden dependencies are published together so there is no harm
-# in treating them like they are part of a mono-repo while developing.
+  # The "#OVERRIDE_FOR_DEVELOPMENT" lines are stripped out when we publish.
+  # All overriden dependencies are published together so there is no harm
+  # in treating them like they are part of a mono-repo while developing.
   devtools_server: #OVERRIDE_FOR_DEVELOPMENT
-    path: ../devtools_server  #OVERRIDE_FOR_DEVELOPMENT
-  devtools_shared:  #OVERRIDE_FOR_DEVELOPMENT
-    path: ../devtools_shared  #OVERRIDE_FOR_DEVELOPMENT
-  devtools_testing:  #OVERRIDE_FOR_DEVELOPMENT
-    path: ../devtools_testing  #OVERRIDE_FOR_DEVELOPMENT
+    path: ../devtools_server #OVERRIDE_FOR_DEVELOPMENT
+  devtools_shared: #OVERRIDE_FOR_DEVELOPMENT
+    path: ../devtools_shared #OVERRIDE_FOR_DEVELOPMENT
+  devtools_testing: #OVERRIDE_FOR_DEVELOPMENT
+    path: ../devtools_testing #OVERRIDE_FOR_DEVELOPMENT