[logging] Change DeviceDaemon to use separate log file (#8454)

This also removes one of the util warn methods that is no longer needed

Author: helin24

src/io/flutter/FlutterUtils.java

@@ -135,17 +135,6 @@ public static void warn(@NotNull Logger logger, @NotNull Exception e, boolean sa
     }
   }
 
-  /**
-   * Write a warning message to the IntelliJ log.
-   * <p>
-   * This is separate from LOG.warn() to allow us to decorate the behavior.
-   *
-   * This method is deprecated (as we are not decorating this behavior anywhere).
-   */
-  public static void warn(@NotNull Logger logger, @NotNull Throwable t) {
-    logger.warn(t);
-  }
-
   /**
    * Write a warning message to the IntelliJ log.
    * <p>

src/io/flutter/run/daemon/DeviceDaemon.java

@@ -24,16 +24,19 @@
 import io.flutter.android.IntelliJAndroidSdk;
 import io.flutter.bazel.Workspace;
 import io.flutter.bazel.WorkspaceCache;
+import io.flutter.logging.PluginLogger;
 import io.flutter.run.FlutterDevice;
 import io.flutter.sdk.FlutterSdk;
 import io.flutter.sdk.FlutterSdkUtil;
+import io.flutter.settings.FlutterSettings;
 import io.flutter.utils.FlutterModuleUtils;
 import io.flutter.utils.MostlySilentColoredProcessHandler;
 import io.flutter.utils.OpenApiUtils;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 import javax.swing.*;
+import java.io.File;
 import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.Comparator;
@@ -114,7 +117,7 @@ boolean needRestart(@NotNull Command next) {
    */
   void shutdown() {
     if (!process.isProcessTerminated()) {
-      LOG.info("shutting down Flutter device daemon #" + id + ": " + command);
+      LOG.info("shutting down Flutter device daemon #" + id + ": " + command.toString(FlutterSettings.getInstance().isFilePathLoggingEnabled()));
     }
     listener.running.set(false);
     process.destroyProcess();
@@ -207,7 +210,7 @@ DeviceDaemon start(Supplier<Boolean> isCancelled,
                        Consumer<String> processStopped) throws ExecutionException {
       final int daemonId = nextDaemonId.incrementAndGet();
       //noinspection UnnecessaryToStringCall
-      LOG.info("starting Flutter device daemon #" + daemonId + ": " + toString());
+      LOG.info("starting Flutter device daemon #" + daemonId + ": " + toString(FlutterSettings.getInstance().isFilePathLoggingEnabled()));
       // The mostly silent process handler reduces CPU usage of the daemon process.
       final ProcessHandler process = new MostlySilentColoredProcessHandler(toCommandLine());
 
@@ -268,7 +271,7 @@ else if (attempts == DeviceDaemon.RESTART_ATTEMPTS_BEFORE_WARNING + 4) {
           }
           catch (java.util.concurrent.ExecutionException e) {
             // This is not a user facing crash - we log (and no devices will be discovered).
-            FlutterUtils.warn(LOG, e.getCause());
+            FlutterUtils.warn(LOG, e, true);
           }
         }
       }
@@ -311,8 +314,12 @@ private GeneralCommandLine toCommandLine() {
 
     @Override
     public String toString() {
+      return toString(true);
+    }
+
+    public @NotNull String toString(boolean showFullPath) {
       final StringBuilder out = new StringBuilder();
-      out.append(command);
+      out.append(showFullPath ? command : command.substring(command.lastIndexOf(File.separatorChar) + 1));
       if (!parameters.isEmpty()) {
         out.append(' ');
         out.append(Joiner.on(' ').join(parameters));
@@ -426,7 +433,7 @@ private static ImmutableList<FlutterDevice> removeDevice(Stream<FlutterDevice> o
     }
   }
 
-  private static final @NotNull Logger LOG = Logger.getInstance(DeviceDaemon.class);
+  private static final @NotNull Logger LOG = PluginLogger.createLogger(DeviceDaemon.class);
 
   // If the daemon cannot be started, display a modal dialog with hopefully helpful
   // instructions on how to fix the problem. This is a big problem; we really do

src/io/flutter/sdk/FlutterSdk.java

@@ -444,11 +444,7 @@ public PubRoot createFiles(@NotNull VirtualFile baseDir, @Nullable Module module
       }
     }
     catch (InterruptedException e) {
-      if (FlutterSettings.getInstance().isFilePathLoggingEnabled()) {
-        LOG.warn(e);
-      } else {
-        LOG.warn(e.toString());
-      }
+      FlutterUtils.warn(LOG, e, true);
       return null;
     }