Skip to content

Commit 94fe6f0

Browse files
authored
[CQ] migrate off deprecated descriptor, EDT, clientProperty accesses (#8192)
Migrate off deprecated APIs: * `getClientProperty` * `ExecutionManager.getInstance(project).getContentManager()` (for run content descriptors) * `EdtInvocationManager.getInstance()` (for `EDT` querying) See: #7718 --- - [x] I’ve reviewed the contributor guide and applied the relevant portions to this PR. <details> <summary>Contribution guidelines:</summary><br> - See our [contributor guide]([https://github.com/dart-lang/sdk/blob/main/CONTRIBUTING.md](https://github.com/flutter/flutter/blob/main/docs/contributing/Tree-hygiene.md#overview) for general expectations for PRs. - Larger or significant changes should be discussed in an issue before creating a PR. - Dart contributions to our repos should follow the [Dart style guide](https://dart.dev/guides/language/effective-dart) and use `dart format`. - Java and Kotlin contributions should strive to follow Java and Kotlin best practices ([discussion](#8098)). </details>
1 parent 1d8cd7b commit 94fe6f0

File tree

4 files changed

+26
-18
lines changed

4 files changed

+26
-18
lines changed

flutter-idea/src/io/flutter/actions/DeviceSelectorAction.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
import com.intellij.openapi.project.ProjectManager;
1717
import com.intellij.openapi.project.ProjectManagerListener;
1818
import com.intellij.openapi.util.Condition;
19+
import com.intellij.openapi.util.Key;
1920
import com.intellij.openapi.util.SystemInfo;
2021
import com.intellij.util.ModalityUiUtil;
2122
import icons.FlutterIcons;
@@ -132,10 +133,10 @@ private void update(@NotNull Project project, @NotNull Presentation presentation
132133
updateVisibility(project, presentation);
133134
}
134135

135-
private static void updateVisibility(final Project project, final Presentation presentation) {
136+
private static void updateVisibility(final Project project, final @NotNull Presentation presentation) {
136137
final boolean visible = isSelectorVisible(project);
137138

138-
final JComponent component = (JComponent)presentation.getClientProperty("customComponent");
139+
final JComponent component = presentation.getClientProperty(new Key<>("customComponent"));
139140
if (component != null) {
140141
component.setVisible(visible);
141142
if (component.getParent() != null) {

flutter-idea/src/io/flutter/run/LaunchState.java

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,10 @@
55
*/
66
package io.flutter.run;
77

8-
import com.intellij.execution.*;
8+
import com.intellij.execution.DefaultExecutionResult;
9+
import com.intellij.execution.ExecutionException;
10+
import com.intellij.execution.ExecutionResult;
11+
import com.intellij.execution.Executor;
912
import com.intellij.execution.configurations.CommandLineState;
1013
import com.intellij.execution.configurations.GeneralCommandLine;
1114
import com.intellij.execution.configurations.RunProfile;
@@ -21,6 +24,7 @@
2124
import com.intellij.execution.ui.ConsoleView;
2225
import com.intellij.execution.ui.ConsoleViewContentType;
2326
import com.intellij.execution.ui.RunContentDescriptor;
27+
import com.intellij.execution.ui.RunContentManager;
2428
import com.intellij.icons.AllIcons;
2529
import com.intellij.openapi.actionSystem.AnAction;
2630
import com.intellij.openapi.actionSystem.Separator;
@@ -389,7 +393,7 @@ protected RunContentDescriptor doExecute(@NotNull RunProfileState state, @NotNul
389393

390394
// See if we should issue a hot-reload.
391395
final List<RunContentDescriptor> runningProcesses =
392-
ExecutionManager.getInstance(env.getProject()).getContentManager().getAllDescriptors();
396+
RunContentManager.getInstance(env.getProject()).getAllDescriptors();
393397

394398
final ProcessHandler process = getRunningAppProcess(launchState.runConfig);
395399
if (process != null) {
@@ -446,15 +450,18 @@ private String getSelectedDeviceId(@NotNull Project project) {
446450
* Returns the currently running app for the given RunConfig, if any.
447451
*/
448452
@Nullable
449-
public static ProcessHandler getRunningAppProcess(RunConfig config) {
453+
public static ProcessHandler getRunningAppProcess(@NotNull RunConfig config) {
450454
final Project project = config.getProject();
451-
final List<RunContentDescriptor> runningProcesses =
452-
ExecutionManager.getInstance(project).getContentManager().getAllDescriptors();
455+
if (project != null) {
456+
final List<RunContentDescriptor> runningProcesses =
457+
RunContentManager.getInstance(project).getAllDescriptors();
453458

454-
for (RunContentDescriptor descriptor : runningProcesses) {
455-
final ProcessHandler process = descriptor.getProcessHandler();
456-
if (process != null && !process.isProcessTerminated() && process.getUserData(FLUTTER_RUN_CONFIG_KEY) == config) {
457-
return process;
459+
for (RunContentDescriptor descriptor : runningProcesses) {
460+
if (descriptor == null) continue;
461+
final ProcessHandler process = descriptor.getProcessHandler();
462+
if (process != null && !process.isProcessTerminated() && process.getUserData(FLUTTER_RUN_CONFIG_KEY) == config) {
463+
return process;
464+
}
458465
}
459466
}
460467

flutter-idea/src/io/flutter/run/daemon/FlutterApp.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
import com.google.common.base.Stopwatch;
99
import com.google.gson.JsonObject;
1010
import com.intellij.execution.ExecutionException;
11-
import com.intellij.execution.ExecutionManager;
1211
import com.intellij.execution.configurations.GeneralCommandLine;
1312
import com.intellij.execution.process.ProcessAdapter;
1413
import com.intellij.execution.process.ProcessEvent;
@@ -17,6 +16,7 @@
1716
import com.intellij.execution.ui.ConsoleView;
1817
import com.intellij.execution.ui.ConsoleViewContentType;
1918
import com.intellij.execution.ui.RunContentDescriptor;
19+
import com.intellij.execution.ui.RunContentManager;
2020
import com.intellij.history.LocalHistory;
2121
import com.intellij.openapi.Disposable;
2222
import com.intellij.openapi.diagnostic.Logger;
@@ -187,7 +187,7 @@ public static FlutterApp fromProcess(@NotNull ProcessHandler process) {
187187
public static List<FlutterApp> allFromProjectProcess(@NotNull Project project) {
188188
final List<FlutterApp> allRunningApps = new ArrayList<>();
189189
final List<RunContentDescriptor> runningProcesses =
190-
ExecutionManager.getInstance(project).getContentManager().getAllDescriptors();
190+
RunContentManager.getInstance(project).getAllDescriptors();
191191
for (RunContentDescriptor descriptor : runningProcesses) {
192192
final ProcessHandler process = descriptor.getProcessHandler();
193193
if (process != null) {
@@ -469,7 +469,7 @@ public StreamSubscription<Boolean> hasServiceExtension(String name, Consumer<Boo
469469
}
470470
return getVMServiceManager().hasServiceExtension(name, onData);
471471
}
472-
472+
473473
public void setConsole(@Nullable ConsoleView console) {
474474
myConsole = console;
475475
}

flutter-idea/src/io/flutter/sdk/FlutterSdk.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
import com.intellij.openapi.vfs.VfsUtil;
2727
import com.intellij.openapi.vfs.VirtualFile;
2828
import com.intellij.openapi.vfs.VirtualFileManager;
29-
import com.intellij.util.ui.EdtInvocationManager;
29+
import com.intellij.util.ui.EDT;
3030
import com.jetbrains.lang.dart.sdk.DartSdk;
3131
import git4idea.config.GitExecutableManager;
3232
import io.flutter.FlutterBundle;
@@ -159,7 +159,7 @@ private static FlutterSdk saveSdkInCache(@NotNull VirtualFile home) {
159159
@Nullable
160160
private static Library getDartSdkLibrary(@NotNull Project project) {
161161
LibraryTablesRegistrar registrar = LibraryTablesRegistrar.getInstance();
162-
if (registrar == null) return null;
162+
if (registrar == null) return null;
163163
final LibraryTable libraryTable = registrar.getLibraryTable(project);
164164
for (Library lib : libraryTable.getLibraries()) {
165165
if (lib != null && "Dart SDK".equals(lib.getName())) {
@@ -458,7 +458,7 @@ public PubRoot createFiles(@NotNull VirtualFile baseDir, @Nullable Module module
458458
return null;
459459
}
460460

461-
if (EdtInvocationManager.getInstance().isEventDispatchThread()) {
461+
if (EDT.isCurrentThreadEdt()) {
462462
VfsUtil.markDirtyAndRefresh(false, true, true, baseDir); // Need this for AS.
463463
}
464464
else {
@@ -670,7 +670,7 @@ public String queryFlutterConfig(String key, boolean useCachedValue) {
670670
}
671671

672672
final JsonObject obj = elem.getAsJsonObject();
673-
if (obj == null) return null;
673+
if (obj == null) return null;
674674

675675
for (String jsonKey : JsonUtils.getKeySet(obj)) {
676676
final JsonElement element = obj.get(jsonKey);

0 commit comments

Comments
 (0)