Skip to content

refactor(core): Update method of grabbing native setTimeout and rAF#55092

Closed
atscott wants to merge 1 commit intoangular:mainfrom
atscott:unpatchedthing
Closed

refactor(core): Update method of grabbing native setTimeout and rAF#55092
atscott wants to merge 1 commit intoangular:mainfrom
atscott:unpatchedthing

Conversation

@atscott
Copy link
Copy Markdown
Contributor

@atscott atscott commented Mar 28, 2024

As mentioned in #54600 (comment) a more effective way of getting the unpatched version of a zone-patched API is to grab it from global[Zone.symbol('apiname')].

@atscott atscott added area: core Issues related to the framework runtime target: minor This PR is targeted for the next minor release labels Mar 28, 2024
@ngbot ngbot Bot modified the milestone: Backlog Mar 28, 2024
@atscott atscott requested a review from JiaLiPassion March 28, 2024 19:17
@atscott atscott force-pushed the unpatchedthing branch 2 times, most recently from c69033f to 02fd7ea Compare March 29, 2024 14:53
Copy link
Copy Markdown
Contributor

@JiaLiPassion JiaLiPassion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

As mentioned in angular#54600 (comment)
a more effective way of getting the unpatched version of a zone-patched
API is to grab it from global[Zone.__symbol__('apiname')].
@atscott atscott added the action: merge The PR is ready for merge by the caretaker label Apr 1, 2024
@jessicajaniuk
Copy link
Copy Markdown
Contributor

This PR was merged into the repository by commit 58942db.

atscott added a commit to atscott/angular that referenced this pull request Apr 5, 2024
Now that angular#55092 has merged, we now consistently get the `setTimeout` and
`rAF` implementations that are not patched by zone. There's no longer a
need to run them in the root zone.
atscott added a commit that referenced this pull request Apr 8, 2024
Now that #55092 has merged, we now consistently get the `setTimeout` and
`rAF` implementations that are not patched by zone. There's no longer a
need to run them in the root zone.

PR Close #55230
iteriani pushed a commit to iteriani/angular that referenced this pull request Apr 11, 2024
…lar#55230)

Now that angular#55092 has merged, we now consistently get the `setTimeout` and
`rAF` implementations that are not patched by zone. There's no longer a
need to run them in the root zone.

PR Close angular#55230
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators May 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants