-
Notifications
You must be signed in to change notification settings - Fork 15.4k
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
fix: volume key globalShortcut registration #23782
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.
Patch application failed.
Side note: Can you also update the patch subject, it has outdated information about MPRemoteCommandCenter
fb7f05b
to
117485b
Compare
@deepak1556 that was just the automatic patch linter failing, not the patch itself. Fixed! |
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.
Thanks!
Release Notes Persisted
|
I was unable to backport this PR to "8-x-y" cleanly; |
I have automatically backported this PR to "10-x-y", please check out #23823 |
I have automatically backported this PR to "9-x-y", please check out #23824 |
@codebytere has manually backported this PR to "8-x-y", please check out #23948 |
Description of Change
Fixes a NOTREACHED DCHECK crash here borne of the fact that we patched
MediaKeysListener::IsMediaKeycode
and then did not add the three extra keys to the switch statement. Also adds the three missing keys to the Accessibility check.Checklist
npm test
passesRelease Notes
Notes: Fixed an issue with volume-related
globalShortcut
registration.