1379927 - about:networking isn't auto refreshed after restoring session

Status: RESOLVED FIXED

(Core :: Networking, enhancement)

Description

[Michal Novotny [:michal]]

After restoring session, the page about:networking#rcwn doesn't auto refresh when autorefresh is checked. The "refresh" button is enabled. Auto-refreshing starts working after re-enabling it or after switching to another tab in about:networking. In the latter case the "refresh" button is still enabled.

Comment 1

[Valentin Gosu [:valentin] (he/him)]

Attachment: Make sure init gets called when about:networking is loaded from the bfcache

MozReview-Commit-ID: 8DdhaEm88Nb

Comment 2

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8885288 [details] [diff] [review]
Make sure init gets called when about:networking is loaded from the bfcache

Review of attachment 8885288 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/content/aboutNetworking.js
@@ +437,5 @@
>  }, {once: true});
>  
> +window.onunload = function() {
> +  // We need this to ensure that init is called when the page is loaded
> +  // from the bfcache.

Can you use "pageshow" and "pagehide" instead? Using "onunload" disables the bfcache making it slower to go back and forward etc.

Comment 3

[Valentin Gosu [:valentin] (he/him)]

Attachment: Make sure init gets called when about:networking is loaded from the bfcache

Thanks for the suggestion.

Comment 4

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8885531 [details] [diff] [review]
Make sure init gets called when about:networking is loaded from the bfcache

Review of attachment 8885531 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/content/aboutNetworking.js
@@ +435,5 @@
> +// We use the pageshow event instead of onload. This is needed because sometimes
> +// the page is loaded via session-restore/bfcache. In such cases we need to call
> +// init() to keep the page behaviour consistent with the ticked checkboxes.
> +// Mostly the issue is with the autorefresh checkbox.
> +window.addEventListener("pageshow", function() {

pageshow can be called multiple times, for example switching away and back to this tab. Can you add an early return to init() in the case that init() was already called?

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/50bb4bb1902f
Make sure init gets called when about:networking is loaded from the bfcache r=jaws

Comment 6

[Valentin Gosu [:valentin] (he/him)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/50bb4bb1902f84a7335754f62e7a124ca9b5c732
Bug 1379927 - Make sure init gets called when about:networking is loaded from the bfcache r=jaws

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/50bb4bb1902f