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

Conversation

@blasten
Copy link

@blasten blasten commented May 15, 2019

This enables a new mode in which we look at the snapshot from the lib/ directory instead of assets. This is required so that an app bundle .aab can contain AOT snapshots for arm and arm64 and users will only download the snapshots needed for the user's device ABI.

Tested=Manually.

As discussed, we need to add tests to the embedder, but there is some pending work here.

Related issue: flutter/flutter#31922

\cc @dnfield

@blasten blasten requested review from dnfield and jason-simmons May 16, 2019 21:45
@blasten blasten requested a review from jason-simmons May 17, 2019 07:36
@blasten blasten requested a review from jason-simmons May 17, 2019 19:35
@blasten blasten requested a review from matthew-carroll May 18, 2019 00:04
@blasten blasten requested a review from matthew-carroll May 18, 2019 00:32
@matthew-carroll
Copy link
Contributor

LGTM

@blasten blasten merged commit 20f5000 into flutter:master May 20, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 20, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request May 20, 2019
flutter/engine@3a50919...20f5000

git log 3a50919..20f5000 --no-merges --oneline
20f5000 Add mode to load AOT snapshots as a native lib (flutter/engine#8979)
ec347cf Roll src/third_party/dart ed169c4bb0..82c8c78739 (17 commits)

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 ([email protected]), and stop
the roller if necessary.
huqiuser pushed a commit to huqiuser/engine that referenced this pull request Jun 12, 2019
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.

5 participants