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
refactor(core): Simplify event handler extraction logic. #55752
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iteriani looks great 👍
7db61fc
to
33bf9fa
Compare
f09d338
to
e0d9f99
Compare
… extraction logic. This should have been part of an earlier commit, but was not merged.
e0d9f99
to
efdf25a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iteriani do we need a new presubmit (regular, not a TGP one) for this change? I'm removing the "merge" label for now, please re-add if we've ran a presubmit already.
this presubmit was actually done already, but for some reason only one of the commits got merged. |
FYI, I've started a new presubmit for the changes in this PR (since the other part of the change has landed separately). |
This PR was merged into the repository by commit d75adc5. |
This reuses information already recorded during hydration to remove jsaction attributes to also stash event handlers. This avoids a tree walk and looku.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Other information