-
Notifications
You must be signed in to change notification settings - Fork 326
Wait for the Dart plugin to start the DevTools server with retries #8177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some nits really. This looks great. Looking forward to testing it out! 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks! looks great!
AsyncUtils.whenCompleteUiThread( | ||
DevToolsService.getInstance(project).getDevToolsInstance(), | ||
(instance, error) -> { | ||
viewUtils.presentLabel(toolWindow, "Loading " + getToolWindowTitle() + "..."); | ||
|
||
// Skip displaying if the project has been closed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rewrite this comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops autosubmit beat me to it! I will open a follow up PR monday
Fixes #8144
This PR:
executeOnPooledThread
(logic pulled intoDevToolsServerTask.java
). This allows us to display the progress and cancel the server startup onforceRestart
.Server start-up status:
Message if waiting for the server timed out:
Loaded Property Editor: