Open Bug 1712380 Opened 3 years ago Updated 3 years ago

Enable BFCache for pages with *unload event listeners on Desktop

Categories

(Core :: DOM: Navigation, enhancement, P3)

Unspecified
All
enhancement

Tracking

()

ASSIGNED

People

(Reporter: neha, Assigned: smaug)

References

(Depends on 2 open bugs)

Details

Enable BFCache enhancements in Desktop for:

  1. Pages with beforeunload or unload event listeners will now be eligible for BFCache.
  2. Pages with opener relationships will not be BFCache eligible.

This was enabled for Android in bug 1705734 .

Olli - should we enable this with Fission BFCache implementation?

Flags: needinfo?(bugs)

This is very much unrelated to any Fission work.
We could enable this before shipping Fission or after shipping Fission, but hopefully not when we ship Fission.

Opener relationship restriction is there always with bfcache-in-parent.

I assume the summary of the bug was going to be something else ;)

Flags: needinfo?(bugs)
Summary: Enable BFCache for pages with *unload event listeners on Android → Enable BFCache for pages with *unload event listeners on Desktop

(In reply to Olli Pettay [:smaug] from comment #2)

We could enable this before shipping Fission or after shipping Fission, but hopefully not when we ship Fission.
Opener relationship restriction is there always with bfcache-in-parent.
Yes, that's why I thought we can ship this with, but definitely before is better ;)
I assume the summary of the bug was going to be something else ;)
hah that's the job of bugzilla cloning and my oversight. I fixed it now. :)

Assigning to smaug to do after the assigned Fission M7a and M8 work.

Assignee: nobody → bugs
Status: NEW → ASSIGNED
Depends on: 1741670
Depends on: 1741671

Moving to Enhancement, and changing the priority to P3.

Severity: S2 → --
Priority: P2 → P3
See Also: → 1751542
You need to log in before you can comment on or make changes to this bug.