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
Reload WebXR sites when settings are changed #3031
Conversation
@@ -323,6 +320,9 @@ private void setWebXR(boolean value, boolean doApply) { | |||
|
|||
if (doApply) { | |||
SettingsStore.getInstance(getContext()).setWebXREnabled(value); | |||
for (WindowWidget window: mWidgetManager.getWindows().getCurrentWindows()) { | |||
window.getSession().reload(GeckoSession.LOAD_FLAGS_NONE); |
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.
Should probably be using GeckoSession.LOAD_FLAGS_BYPASS_CACHE
as it has been fixed in GV.
for (WindowWidget window: mWidgetManager.getWindows().getCurrentWindows()) { | ||
Session session = window.getSession(); | ||
if (aHost.equalsIgnoreCase(UrlUtils.getHost(session.getCurrentUri()))) { | ||
session.reload(GeckoSession.LOAD_FLAGS_NONE); |
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.
And here too should use GeckoSession.LOAD_FLAGS_BYPASS_CACHE
134d1f3
to
7f95037
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.
If you have the same site opened in two windows and Enable/Disable the WebXR status from the URL button, the other window is not refreshed.
Also related, if you have the same site opened in different windows and change the WebXR state in one of them the WebXR icon is not updated when the other window is focused.
@keianhzo Done |
@MortimerGoro I don't see the items from my last comment fixed, maybe it's not pushed? |
@MortimerGoro I see this fixed in the other PR so it's ok. |
No description provided.