Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove layoutContext Threading #39401

Closed
wants to merge 3 commits into from

Conversation

NickGerleman
Copy link
Contributor

Summary: kill_with_fire_flamethrower

Differential Revision: D49179244

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Sep 12, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 12, 2023
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Differential Revision: D49179244

fbshipit-source-id: ab8ced071b61425c0a13ef288d0ba560dd5633b1
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 12, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Differential Revision: D49179244

fbshipit-source-id: 7591bad4ce29955b19dc9d352b20178315d211f8
@analysis-bot
Copy link

analysis-bot commented Sep 12, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,969,050 +1,211
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 9,561,603 +1,295
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 5998320
Branch: main

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 12, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Differential Revision: D49179244

fbshipit-source-id: 0f33b4dcb832fa04dc96603224d25849f5baa79b
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 12, 2023
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Differential Revision: D49179244

fbshipit-source-id: 15610d13b4ba119dab99e031d5581d237a30661e
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 12, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Differential Revision: D49179244

fbshipit-source-id: be3d1fa8c4ff208a3adbe2589f23fafa6b88c74d
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 12, 2023
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Differential Revision: D49179244

fbshipit-source-id: dbf0f25cc1c8ec59d774d26f41efde12a290938b
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 12, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Differential Revision: D49179244

fbshipit-source-id: 375457b7bbcae78bb03f8e708a72b70e5ecb05f3
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 12, 2023
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Differential Revision: D49179244

fbshipit-source-id: db0a91d63b0a07445f2a6581a004f5f53266358c
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 12, 2023
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: c7a2c14dd3f934ed77d27841902e7565167e322b
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 12, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 76c184c1b72432179d481ef0463b197cdea5b44a
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 12, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 6b5ef24149dc5dfbe5c1dcb27048f4ed640e17b8
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 12, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 3094a63a273d7515a169420d3f1ab96fd55766e8
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 12, 2023
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 872eb72aca480817c92b69eaa62a249f225762d2
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 13, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 4f0ee3b3245b84d30b3b69f285ba97e1ee800d6b
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 13, 2023
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: d78310517fbc49adf7b9264b8d9ebed10aac08e1
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 13, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 2d14e44ed0ab19b85b3f3c65b4ff81fba3cc2ab1
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 13, 2023
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 0382def3a307db7187ec25a3c972f76e9a9d1915
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 13, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: e3d535f6fde65a956bb67cfbf1eb78d58305e86b
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Sep 13, 2023
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 820bbe5ec0ef795d04ac77bf37659e0150277e19
@github-actions
Copy link

github-actions bot commented Sep 13, 2023

Warnings
⚠️

E2E tests for ci/circleci: test_e2e_ios failed with errors. See the logs for details

⚠️

E2E tests for ci/circleci: test_e2e_ios failed with errors. See the logs for details

Generated by 🚫 dangerJS against 0aaf6f7

NickGerleman and others added 3 commits September 12, 2023 19:09
Summary: Moves `isBaselineLayout` out of `CalculateLayout` into `Baseline.h`. This function is called by flex line justification code, which I have been looking at extracting.

Differential Revision: D49177937

fbshipit-source-id: 94a45c6570bd073578802b0ba9aa945329d08eea
Summary:
To avoid keeping a per-node mapping on native Yoga nodes to Java nodes, a per-layout context was added, to be able to pass information from the start of the layout, to measure functions, log functions, etc.

The way this was done was super invasive, and added quite a few private APIs used only by the JNI functions.

This change removes the context-using functions from the JNI bindings in favor of it managing its own context. Next diff removes all the cruft.

Differential Revision: D49179243

fbshipit-source-id: 2b1433450b18ef5c5bce9800924504a142fe952c
Summary:
Pull Request resolved: facebook#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 96054b9b3a826dbea74deff94d37eb1f8e172757
NickGerleman added a commit to NickGerleman/yoga that referenced this pull request Sep 13, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: facebook#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 4f680ba32d0afb5b0a46621e39d5ad12e2bbbf04
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D49179244

facebook-github-bot pushed a commit to facebook/litho that referenced this pull request Sep 13, 2023
Summary:
X-link: facebook/react-native#39401

X-link: facebook/yoga#1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 9a827e1bd29205254fee5725449191726d6bcf5a
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 13, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in b955d8d.

facebook-github-bot pushed a commit to facebook/yoga that referenced this pull request Sep 13, 2023
Summary:
X-link: facebook/react-native#39401

Pull Request resolved: #1376

kill_with_fire_flamethrower

Reviewed By: rshest

Differential Revision: D49179244

fbshipit-source-id: 9a827e1bd29205254fee5725449191726d6bcf5a
adamaveray added a commit to adamaveray/react-native that referenced this pull request Sep 13, 2023
* main: (1012 commits)
  Add simple constructor for JSError (facebook#39415)
  Breaking: per-node pointScaleFactor (facebook#39403)
  Implement "tickleJS" for Hermes (facebook#39289)
  Add thread idle indicator (facebook#39206)
  Unblock `yarn android` on main (facebook#39413)
  Remove Codegen buck-oss scripts as they're unused (facebook#39422)
  Immediately dispatch events to the shared C++ infrastructure to support interruptability (facebook#39380)
  Fix race condition in Binding::reportMount (facebook#39420)
  Clone free state progression (facebook#39357)
  fix: return the correct default trait collection (facebook#38214)
  Read the React Native version and set the new arch flag properly (facebook#39388)
  Deprecate default_flags in Podfile (facebook#39389)
  Create Helper to read the package.json from Cocoapods (facebook#39390)
  Create helper to enforce the New Arch enabled for double released (facebook#39387)
  Remove layoutContext Drilling (facebook#39401)
  Remove JNI Binding usage of layoutContext (facebook#39402)
  Extract isBaselineLayout() (facebook#39400)
  Refactor and separate layout affected nodes react marker (facebook#39249)
  Bump AGP to 8.1.1 (facebook#39392)
  Fix broken Gradle Sync when opening the project with Android Studio (facebook#39412)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants