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

Register native callback before running engine. #37600

Merged
merged 1 commit into from
Nov 15, 2022

Conversation

a-wallen
Copy link
Contributor

@a-wallen a-wallen commented Nov 14, 2022

Reorders the native callback registration before the engine is run. If the callback is registered after the engine runs then there is a race.

Fixes flutter/flutter#115245 (comment)

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@a-wallen a-wallen force-pushed the add_native_callback_correct branch from a367338 to bafb04e Compare November 14, 2022 23:37
@a-wallen a-wallen marked this pull request as ready for review November 15, 2022 00:35
@a-wallen a-wallen added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 15, 2022
@auto-submit auto-submit bot merged commit eead7b0 into flutter:main Nov 15, 2022
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Nov 15, 2022
…115369)

* 7a4b112d7 Roll Skia from 4d519a8ba97a to 1991515dd191 (1 revision) (flutter/engine#37622)

* eead7b0d8 Place native callback registration before running engine (flutter/engine#37600)
schwa423 pushed a commit to schwa423/engine that referenced this pull request Nov 16, 2022
shogohida pushed a commit to shogohida/flutter that referenced this pull request Dec 7, 2022
…lutter#115369)

* 7a4b112d7 Roll Skia from 4d519a8ba97a to 1991515dd191 (1 revision) (flutter/engine#37622)

* eead7b0d8 Place native callback registration before running engine (flutter/engine#37600)
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…lutter#115369)

* 7a4b112d7 Roll Skia from 4d519a8ba97a to 1991515dd191 (1 revision) (flutter/engine#37622)

* eead7b0d8 Place native callback registration before running engine (flutter/engine#37600)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App platform-macos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New flake in FlutterEngineTest.BackgroundIsBlack
2 participants