Skip to content

Remove bridge-mode branches from RCTDevSettings (#59260) - #59260

Closed
christophpurrer wants to merge 3 commits into
react:mainfrom
christophpurrer:export-D126574662
Closed

christophpurrer wants to merge 3 commits into
react:mainfrom
christophpurrer:export-D126574662

Conversation

@christophpurrer

@christophpurrer christophpurrer commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Reviewed By: shwanton

Differential Revision: D126574662

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 21, 2026
@meta-codesync

meta-codesync Bot commented Oct 21, 2026

Copy link
Copy Markdown

@christophpurrer has exported this pull request. If you are a Meta employee, you can view the originating Diff in D126574662.

christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:
Pull Request resolved: react#59260

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Differential Revision: D126574662
@meta-codesync meta-codesync Bot changed the title Remove bridge-mode branches from RCTDevSettings Remove bridge-mode branches from RCTDevSettings (#59260) Oct 21, 2026
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:
Pull Request resolved: react#59260

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Differential Revision: D126574662
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Differential Revision: D126574662
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Differential Revision: D126574662
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Differential Revision: D126574662
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:
Pull Request resolved: react#59260

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Differential Revision: D126574662
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Reviewed By: shwanton

Differential Revision: D126574662
Summary:

Synthesize the injected launchOptions property and read getInitialURL state from it instead of self.bridge. Drops the last bridge read in this module along with its direct bridge header import.

Changelog:
[Internal]

Reviewed By: shwanton

Differential Revision: D126567470
Summary:

Route JS-thread dispatch through the injected dispatchToJSThread block and read the injected performanceLogger. Delete the Views counter, which can only show zero since RCTViewRegistry exposes no enumeration, and reflow the HUD bar.

Changelog:
[iOS][Removed] - Remove the non-functional Views counter from the PerfMonitor HUD

Reviewed By: shwanton

Differential Revision: D126567926
Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Reviewed By: shwanton

Differential Revision: D126574662
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 22, 2026
Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Reviewed By: shwanton

Differential Revision: D126574662
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 22, 2026
Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Reviewed By: shwanton

Differential Revision: D126574662
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 22, 2026
Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Reviewed By: shwanton

Differential Revision: D126574662
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 22, 2026
Summary:

Delete the always-false _isBridgeMode helper and fold its call sites to the bridgeless paths. Remove the no-op profiling bridge calls with their synchronize cascade, and simplify setExecutorClass to persist locally plus reload.

Changelog:
[Internal]

Reviewed By: shwanton

Differential Revision: D126574662
@meta-codesync meta-codesync Bot closed this in 91d2f80 Oct 23, 2026
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @christophpurrer in 91d2f80

When will my fix make it into a release? | How to file a pick request?

@react-native-bot react-native-bot added the Merged This PR has been merged. label Oct 23, 2026
@meta-codesync

meta-codesync Bot commented Oct 23, 2026

Copy link
Copy Markdown

This pull request has been merged in 91d2f80.

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. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants