Closed Bug 1580764 Opened 5 years ago Closed 4 years ago

Replace frameIds, tabIds and windowIds with BrowsingContext id

Categories

(WebExtensions :: General, task, P2)

task

Tracking

(Fission Milestone:M6, firefox80 fixed)

RESOLVED FIXED
mozilla80
Fission Milestone M6
Tracking Status
firefox80 --- fixed

People

(Reporter: zombie, Assigned: kmag)

References

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

Details

Attachments

(3 files)

We use a lot of code to track and map 3 different types of IDs, and send them around to relevant processes. BrowsingContext info should be more stable and easier to work with, and is already automatically synced across (relevant) processes.

Tab’s top level browsing context might get replaced in some specific cases in the future, but the new concept of “Session” should be introduced that would be a stable representation of a “tab”.

Depends on: 1580766

Tentatively moving all bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to the "?" triage milestone.

This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:

0ee3c76a-bc79-4eb2-8d12-05dc0b68e732

Fission Milestone: --- → ?

I think this will not block any functionality, just a major cleanup of various peaces of code.

Fission Milestone: ? → M6
Depends on: 1641638
Depends on: 1642468
Blocks: 1642468
No longer depends on: 1642468
Assignee: nobody → kmaglione+bmo
Pushed by maglione.k@gmail.com: https://hg.mozilla.org/integration/autoland/rev/36cd8e6d1a15 Part 1 - Add ancestor BrowsingContext IDs to LoadInfo. r=nika https://hg.mozilla.org/integration/autoland/rev/48564fee92c7 Part 2 - Use BrowsingContext IDs rather than window IDs to compute frame IDs. r=zombie https://hg.mozilla.org/integration/autoland/rev/50d88c975e74 Part 3 - Remove unused AncestorOuterWindowIDs from LoadInfo. r=nika
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: