Fix track-widget-creation on Windows

#2205
There were two bugs in how we handled windows file paths that were converted to URIs.

Author: jacob314

src/io/flutter/inspector/InspectorService.java

@@ -9,6 +9,7 @@
 import com.google.gson.*;
 import com.intellij.openapi.Disposable;
 import com.intellij.openapi.application.ApplicationManager;
+import com.intellij.openapi.util.SystemInfo;
 import com.intellij.xdebugger.XSourcePosition;
 import com.jetbrains.lang.dart.ide.runner.server.vmService.VmServiceConsumers;
 import com.jetbrains.lang.dart.ide.runner.server.vmService.frame.DartVmServiceValue;
@@ -118,7 +119,14 @@ public void connectionClosed() {
         }
         final ArrayList<String> rootDirectories = new ArrayList<>();
         for (PubRoot root : app.getPubRoots()) {
-          rootDirectories.add(root.getRoot().getCanonicalPath());
+          String path = root.getRoot().getCanonicalPath();
+          if (SystemInfo.isWindows) {
+            // TODO(jacobr): remove after https://github.com/flutter/flutter-intellij/issues/2217.
+            // The problem is setPubRootDirectories is currently expecting
+            // valid URIs as opposed to windows paths.
+            path = "file:///" + path;
+          }
+          rootDirectories.add(path);
         }
         setPubRootDirectories(rootDirectories);
       });

src/io/flutter/inspector/InspectorSourceLocation.java

@@ -7,6 +7,7 @@
 
 import com.google.gson.JsonArray;
 import com.google.gson.JsonObject;
+import com.intellij.openapi.util.SystemInfo;
 import com.intellij.openapi.vfs.LocalFileSystem;
 import com.intellij.openapi.vfs.VirtualFile;
 import com.intellij.xdebugger.XSourcePosition;
@@ -30,8 +31,13 @@ public VirtualFile getFile() {
       return parent != null ? parent.getFile() : null;
     }
 
-    // We have to strip the file:// prefix from the Dart file path for compatibility.
-    final String filePrefix = "file://";
+    // We have to strip the file:// or file:/// prefix depending on the
+    // operating system to convert from paths stored as URIs to local operating
+    // system paths.
+    // TODO(jacobr): remove this workaround after the code in package:flutter
+    // is fixed to return operating system paths instead of URIs.
+    // https://github.com/flutter/flutter-intellij/issues/2217
+    final String filePrefix = SystemInfo.isWindows ? "file:///" : "file://";
     if (fileName.startsWith(filePrefix)) {
       fileName = fileName.substring(filePrefix.length());
     }

src/io/flutter/view/InspectorPanel.java

@@ -21,6 +21,7 @@
 import com.intellij.util.ui.ColumnInfo;
 import com.intellij.util.ui.JBUI;
 import com.intellij.util.ui.tree.TreeUtil;
+import com.intellij.xdebugger.XSourcePosition;
 import io.flutter.FlutterBundle;
 import io.flutter.editor.FlutterMaterialIcons;
 import io.flutter.inspector.*;
@@ -1097,8 +1098,10 @@ private void syncSelectionHelper(boolean maybeRerootSubtree, DiagnosticsNode det
     final DiagnosticsNode diagnostic = getSelectedDiagnostic();
     if (diagnostic != null) {
       if (isCreatedByLocalProject(diagnostic)) {
-        diagnostic.getCreationLocation().getXSourcePosition().createNavigatable(getFlutterApp().getProject())
-          .navigate(false);
+        XSourcePosition position = diagnostic.getCreationLocation().getXSourcePosition();
+        if (position != null) {
+          position.createNavigatable(getFlutterApp().getProject()).navigate(false);
+        }
       }
     }
     if (myPropertiesPanel != null) {