Skip to content

send supportsRestart flag when starting an app - #4513

Merged
devoncarew merged 4 commits into
flutter:masterfrom
devoncarew:send_supportsRestart
Jun 13, 2016
Merged

devoncarew merged 4 commits into
flutter:masterfrom
devoncarew:send_supportsRestart

Conversation

@devoncarew

Copy link
Copy Markdown
Contributor
  • have devices know whether they support a restart command (they already have a restart method)
  • when starting an app via the daemon app.start command, have the result include whether the target device supports the restart operation
  • include info in the log events about whether the events is a progress start or progress end event (use this info to show good progress in the IDE UI)
  • only show the restart option in flutter run if the target device supports the restart command

@devoncarew

Copy link
Copy Markdown
Contributor Author

@abarth @yjbanov @Hixie for review :)

}

void _sendLogEvent(Map<String, dynamic> event) {
domain?._sendAppEvent(app, 'log', event);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this lead to us missing log events and not knowing why?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll log events to the flutter_tools log if they happen after the close() is called. This would be an unexpected condition however - the logs would be happening after the app has completed.

@yjbanov

yjbanov commented Jun 13, 2016

Copy link
Copy Markdown
Contributor

LGTM

@devoncarew
devoncarew merged commit b6c3ae3 into flutter:master Jun 13, 2016
a-siva added a commit to a-siva/flutter that referenced this pull request Jan 4, 2018
includes the following CLs since the last roll
- Further improve painting.dart documentation. (flutter#4505)
- Add support for --strong option in the engine, create a strong mode
  version of platform file (flutter#4504)
- Apply texture transform on Android (flutter#4513)
- [fuchsia] Removed the flutter_app template. (flutter#4511)
- This adds RPC call to set asset path. (flutter#4323)
- Fix windows build by using lambda capture workaround (flutter#4514)
- Add missing include for sk_malloc_throw
a-siva added a commit that referenced this pull request Jan 5, 2018
includes the following CLs since the last roll
- Further improve painting.dart documentation. (#4505)
- Add support for --strong option in the engine, create a strong mode
  version of platform file (#4504)
- Apply texture transform on Android (#4513)
- [fuchsia] Removed the flutter_app template. (#4511)
- This adds RPC call to set asset path. (#4323)
- Fix windows build by using lambda capture workaround (#4514)
- Add missing include for sk_malloc_throw
cdotstout pushed a commit to cdotstout/flutter that referenced this pull request Apr 3, 2018
* Apply transform to textures on android
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
includes the following CLs since the last roll
- Further improve painting.dart documentation. (flutter#4505)
- Add support for --strong option in the engine, create a strong mode
  version of platform file (flutter#4504)
- Apply texture transform on Android (flutter#4513)
- [fuchsia] Removed the flutter_app template. (flutter#4511)
- This adds RPC call to set asset path. (flutter#4323)
- Fix windows build by using lambda capture workaround (flutter#4514)
- Add missing include for sk_malloc_throw
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants