Closed
Bug 2037172
Opened 6 months ago
Closed 6 months ago
Remove FrameLoader wrapper reparenting
Categories
(Core :: DOM: Core & HTML, defect, P3)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
152 Branch
| Tracking | Status | |
|---|---|---|
| firefox152 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(2 external files)
No description provided.
| Assignee | ||
Comment 1•6 months ago
|
||
This is the last usage of UpdateReflectorGlobal. It was added to bug
1391110 with this commit message:
We allow swapping frameloaders between unrelated documents, so we need
to reparent wrappers when the owner content changes.
It seems better to not parent them to a window object and parent them to
the privileged junk scope just like BrowsingContext objects are for
example.
| Assignee | ||
Comment 2•6 months ago
|
||
Dead code after the last patch.
Pushed by ealvarez@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/7f3a41026c71
https://hg.mozilla.org/integration/autoland/rev/f37fa772419f
Remove frameloader reflector reparenting. r=smaug
Comment 4•6 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
status-firefox152:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 152 Branch
| Assignee | ||
Updated•6 months ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Pushed by ealvarez@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/dd911ae7c0b2
https://hg.mozilla.org/integration/autoland/rev/cf7b259fb24e
Remove UpdateReflectorGlobal and related transplanting code. r=dom-core,mccr8
Updated•6 months ago
|
Status: REOPENED → RESOLVED
Closed: 6 months ago → 6 months ago
Resolution: --- → FIXED
Updated•5 months ago
|
QA Whiteboard: [qa-triage-done-c153/b152]
You need to log in
before you can comment on or make changes to this bug.
Description
•