Skip to content
This repository has been archived by the owner on Feb 25, 2025. It is now read-only.

Removed callback for HeadlessDartRunner #5983

Merged
merged 1 commit into from
Aug 9, 2018
Merged

Removed callback for HeadlessDartRunner #5983

merged 1 commit into from
Aug 9, 2018

Conversation

bkonyi
Copy link
Contributor

@bkonyi bkonyi commented Aug 9, 2018

This is to be somewhat consistent with the Android interface where we decided against having a callback for when the created by a background shell is entered.

@bkonyi bkonyi merged commit 764884b into master Aug 9, 2018
@bkonyi bkonyi deleted the remove_callback branch August 9, 2018 18:21
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 9, 2018
flutter/engine@3777931...764884b

git log 3777931..764884b --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-09 bkonyi@google.com Removed callback for HeadlessDartRunner (flutter/engine#5983)
2018-08-08 liyuqian@google.com Revert "Temporarily add travis/analyze.sh back for Chrome bot (flutter#5961)" (flutter/engine#5966)
2018-08-08 37626415+skia-flutter-autoroll@users.noreply.github.com Roll src/third_party/skia 9c9611fcc1bb..0d5d0659a684 (7 commits) (flutter/engine#5980)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
Copy link
Contributor

@jamesderlin jamesderlin left a comment

Choose a reason for hiding this comment

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

Line 114 still references runWithEntrypointAndCallback:libraryUri:completion:.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 9, 2018
flutter/engine@3777931...e54bc4e

git log 3777931..e54bc4e --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-09 bkonyi@google.com Fixed invalid call site of runWithEntrypointAndCallback (flutter/engine#5984)
2018-08-09 bkonyi@google.com Removed callback for HeadlessDartRunner (flutter/engine#5983)
2018-08-08 liyuqian@google.com Revert "Temporarily add travis/analyze.sh back for Chrome bot (flutter#5961)" (flutter/engine#5966)
2018-08-08 37626415+skia-flutter-autoroll@users.noreply.github.com Roll src/third_party/skia 9c9611fcc1bb..0d5d0659a684 (7 commits) (flutter/engine#5980)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 10, 2018
flutter/engine@3777931...d90223f

git log 3777931..d90223f --date=short --no-merges --format='%%ad %%ae %%s'
2018-08-09 a-siva@users.noreply.github.com Roll Dart to version 1be785ae2ddb1754a184cd638ab719e94d86b4e9 (flutter/engine#5985)
2018-08-09 bkonyi@google.com Fixed invalid call site of runWithEntrypointAndCallback (flutter/engine#5984)
2018-08-09 bkonyi@google.com Removed callback for HeadlessDartRunner (flutter/engine#5983)
2018-08-08 liyuqian@google.com Revert "Temporarily add travis/analyze.sh back for Chrome bot (flutter#5961)" (flutter/engine#5966)
2018-08-08 37626415+skia-flutter-autoroll@users.noreply.github.com Roll src/third_party/skia 9c9611fcc1bb..0d5d0659a684 (7 commits) (flutter/engine#5980)

The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants