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

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Aug 27, 2019

CompleteEmpty causes an exception in the framework because it thinks the null response means there's no plugin/channel listening. This was introduced a while back now to improve the API usage.

Fixes flutter/flutter#39337

/cc @abhishekamit

@dnfield dnfield merged commit 4ceaafc into flutter:master Aug 28, 2019
@dnfield dnfield deleted the skia_response branch August 28, 2019 00:58
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 28, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Aug 28, 2019
[email protected]:flutter/engine.git/compare/f93b9cbff143...180adcb

git log f93b9cb..180adcb --no-merges --oneline
2019-08-28 [email protected] make engine, ui, and sdk rewriter inputs of dill construction (flutter/engine#11554)
2019-08-28 [email protected] Make Skia cache size channel respond with a value (flutter/engine#11550)
2019-08-27 [email protected] Roll src/third_party/dart f17d0563fa..2bc0f49c00 (10 commits)
2019-08-27 [email protected] Add check to enable metal for import (flutter/engine#11537)
2019-08-27 [email protected] Update label of Fuchsia FIDL targets. (flutter/engine#11514)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected] on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
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.

Return a non-empty response from setResourceCacheMaxBytes

3 participants