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
feat: implement chrome.tabs.reload to fix PDF Viewer error #33560
Conversation
76e3e53
to
2dc485f
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.
LGTM on the navigation controller calls, will delegate to @nornagon for the extension api
API LGTM |
Release Notes Persisted
|
I have automatically backported this PR to "18-x-y", please check out #33711 |
I have automatically backported this PR to "19-x-y", please check out #33712 |
Description of Change
Closes #33519.
Addresses a failure introduced by CL:3501416, whereby if a PDF failed to load:
and a user then clicked the
Reload
button, the following error would be seen in console:This is fixed by implementing
chrome.tabs.reload
. Given it'ssemver-minor
as a change, I think we should likely document it as we would if we were implementing it standalone.Note: needs tests.
Checklist
npm test
passesRelease Notes
Notes: Fixed an issue where the PDF Viewer would fail if a user attempted to reload.