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

Conversation

sbaranov
Copy link
Contributor

@sbaranov sbaranov commented Dec 28, 2018

This change introduces manifest properties that control when dynamic patches are downloaded and installed in the application lifecycle.

Application developer can choose whether between install on restart, install on resume, or immediate forced install of dynamic patches.

…led.

This change introduces manifest properties that control when dynamic patches are downloaded and installed in the application lifecycle.

Application developer can choose whether between install on restart, install on resume, or immediate forced install of dynamic patches.
@sbaranov
Copy link
Contributor Author

PTAL

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

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

LGTM

if (sIsPrecompiledAsSharedLibrary) {
sResourceExtractor
.addResource(sAotSharedLibraryPath);

Copy link
Contributor

Choose a reason for hiding this comment

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

nit: extra line here and below

@sbaranov sbaranov merged commit 8cca33a into flutter:master Jan 2, 2019
@sbaranov sbaranov deleted the onresume branch January 2, 2019 18:40
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 3, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jan 3, 2019
flutter/engine@2940812...134b65e

git log 2940812..134b65e --no-merges --oneline
134b65e Roll src/third_party/skia aae6ae39a356..fdde2926df92 (1 commits) (flutter/engine#7358)
f97edf2 Roll src/third_party/skia 96345a23dfa9..aae6ae39a356 (1 commits) (flutter/engine#7357)
d051c47 Roll src/third_party/skia bc84adceaada..96345a23dfa9 (2 commits) (flutter/engine#7356)
15fcba4 Roll src/third_party/skia d0d5bb8c8500..bc84adceaada (6 commits) (flutter/engine#7355)
4d353de Preparing to remove SK_SUPPORT_LEGACY_PAINT_TEXTMEASURE (flutter/engine#7354)
54d7c42 Paint all backgrounds first to prevent overlap (flutter/engine#7287)
0ad16dd Roll src/third_party/skia 279b65ddb5a3..d0d5bb8c8500 (12 commits) (flutter/engine#7353)
8cca33a Give more control over when dynamic patches get downloaded and installed. (flutter/engine#7327)
2d291c7 Roll src/third_party/skia 8f37ce5b9f2c..279b65ddb5a3 (6 commits) (flutter/engine#7350)

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

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 (amirha@chromium.org), and stop
the roller if necessary.
kangwang1988 pushed a commit to XianyuTech/flutter that referenced this pull request Feb 12, 2019
flutter/engine@2940812...134b65e

git log 2940812..134b65e --no-merges --oneline
134b65e Roll src/third_party/skia aae6ae39a356..fdde2926df92 (1 commits) (flutter/engine#7358)
f97edf2 Roll src/third_party/skia 96345a23dfa9..aae6ae39a356 (1 commits) (flutter/engine#7357)
d051c47 Roll src/third_party/skia bc84adceaada..96345a23dfa9 (2 commits) (flutter/engine#7356)
15fcba4 Roll src/third_party/skia d0d5bb8c8500..bc84adceaada (6 commits) (flutter/engine#7355)
4d353de Preparing to remove SK_SUPPORT_LEGACY_PAINT_TEXTMEASURE (flutter/engine#7354)
54d7c42 Paint all backgrounds first to prevent overlap (flutter/engine#7287)
0ad16dd Roll src/third_party/skia 279b65ddb5a3..d0d5bb8c8500 (12 commits) (flutter/engine#7353)
8cca33a Give more control over when dynamic patches get downloaded and installed. (flutter/engine#7327)
2d291c7 Roll src/third_party/skia 8f37ce5b9f2c..279b65ddb5a3 (6 commits) (flutter/engine#7350)

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

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 (amirha@chromium.org), 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.

3 participants