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

Conversation

@dcharkes
Copy link
Contributor

@dcharkes dcharkes commented Jan 26, 2021

After #23808 and #23000 ffi patch files were still not included in the Flutter build.

This patch is tested locally with a Flutter app (on Android) using the extension method in ffi_allocation_patch and changes the result from

When the exception was thrown, this was the stack:
#0      NoSuchMethodError._throwNew (dart:core-patch/errors_patch.dart:212:5)
#1      AllocatorAlloc.call (dart:ffi/allocation.dart)
...

to a succeeding run.

If the branch cut is done after this is merged, this requires a cherry pick for AllocatorAlloc.call to work in the next Flutter release.

@dcharkes dcharkes merged commit a6119a3 into master Jan 26, 2021
@dcharkes dcharkes deleted the ffi-patch-files-2 branch January 26, 2021 13:44
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 26, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 26, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 26, 2021
dcharkes added a commit to dart-archive/ffi that referenced this pull request Feb 3, 2021
This Dart version contains the cherry pick for dart-lang/sdk#44822 (in dart-lang/sdk@e5dd92c) in Dart standalone.

This Dart version will contain and flutter/engine#23954 (in flutter/engine#24142).
christopherfujino added a commit that referenced this pull request Feb 3, 2021
…24142)

* add ffi_allocation_patch.dart to libraries.yaml 2 (#23954)

* Use `runes` to get code units in CanvasKit. (#24024)

* [windows] Honor only valid resize requests (#23990)

* Load FlutterLoader when creating FlutterEngineGroup (#23980)

* [canvaskit] remove the DOM node of unrendered platform view (#24001)

* update dart to 2.12.0-259.8.beta

* [web] Fix svg based stroke rendering. (#23969)

* [canvaskit] fix text background, foreground, color; add text style tests (#23800)

* [canvaskit] fix text background, foreground, familyFallback; add style tests

* add leak test

* update goldens_lock.yaml

* remove solo

* Warn when popping out of empty text style stack

* [web] Fix alignment issue in rich paragraphs (#23965)

* [web] Implement CanvasParagraph.getLineBoundary (#24037)

* [web] Fix text alignment when transform + tight constraints + DOM rendering (#24036)

* update licenses_third_party golden

Co-authored-by: Daco Harkes <[email protected]>
Co-authored-by: Harry Terkelsen <[email protected]>
Co-authored-by: Kaushik Iska <[email protected]>
Co-authored-by: xster <[email protected]>
Co-authored-by: Yegor <[email protected]>
Co-authored-by: Ferhat <[email protected]>
Co-authored-by: Mouad Debbar <[email protected]>
hjfreyer pushed a commit to hjfreyer/engine that referenced this pull request Mar 22, 2021
@dcharkes dcharkes mentioned this pull request Mar 4, 2022
8 tasks
dcharkes added a commit to dart-lang/native that referenced this pull request Jan 16, 2024
This Dart version contains the cherry pick for dart-lang/sdk#44822 (in dart-lang/sdk@e5dd92c) in Dart standalone.

This Dart version will contain and flutter/engine#23954 (in flutter/engine#24142).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants