Closed Bug 1676709 Opened 4 years ago Closed 4 years ago

Dispatch "browsing-context-attached" notification when attaching a BrowsingContext

Categories

(Core :: DOM: Content Processes, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
85 Branch
Tracking Status
firefox85 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

(Whiteboard: [marionette-fission-mvp])

Attachments

(1 file)

Similar to bug 1590239, which added the browsing-context-discarded notification, there should be one when a browsing context gets attached.

This should be send from BrowsingContext::Attach:

https://searchfox.org/mozilla-central/rev/50215d649d4854812837f1343e8f47bd998dacb5/docshell/base/BrowsingContext.cpp#607

And as Nika mentioned on Matrix: "though make sure to use NotifyWhenScriptSafe as Attach is run when it's not safe to run script".

Blocks: 1674329

Given that we need this functionality in Marionette I'm trying to get this implemented.

Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [marionette-fission-mvp]
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/45221414fcf8
Send a "browsing-context-attached" notification when attached, r=nika
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: