Open Bug 1597811 Opened 3 years ago Updated 2 months ago

Don't use "pageshow" and "pagehide" for page navigation listeners

Categories

(Remote Protocol :: CDP, defect, P3)

defect

Tracking

(Not tracked)

People

(Reporter: whimboo, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

As discovered by Maja when reviewing https://phabricator.services.mozilla.com/D53827#inline-325101 we should probably not use those events given that they are used for bfcache.

Instead 'unload' and 'load' seem to be the right choices.

But if it turns out that we have to use a webprogress listener as discussed on bug 1597762, the code in question here would require a huge refactoring.

Priority: -- → P3
Depends on: 1612174

In the browser chrome test environment, I see load, pageshow and pagehide reliably, but I only see unload under very narrow circumstances. I'll elaborate further when I dig in more, just want to make note of it for now.

Component: CDP: Page → CDP
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.