Closed
Bug 1646545
Opened 4 years ago
Closed 4 years ago
Figure out if GetInProcessScriptableTop usage in WebSocket.cpp is OK.
Categories
(Core :: DOM: Networking, task, P1)
Core
DOM: Networking
Tracking
()
Tracking | Status | |
---|---|---|
firefox82 | --- | fixed |
People
(Reporter: kmag, Assigned: valentin)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
It's used in a bunch of places, mostly (though not always) just to get the top inner window ID. That can be done (and more correctly) using WindowContext instead. I'm not sure what needs to happen with the other uses.
Updated•4 years ago
|
Severity: -- → N/A
Priority: -- → P3
Whiteboard: [necko-triaged]
Comment 1•4 years ago
|
||
Tracking "Figure out GetInProcessTop usage" bugs for Fission M6b.
Fission Milestone: --- → M6b
Updated•4 years ago
|
Flags: needinfo?(nhnguyen) → needinfo?(juhsu)
Comment 3•4 years ago
|
||
I'll take a look
Assignee: nobody → juhsu
Flags: needinfo?(juhsu)
Priority: P3 → P1
Updated•4 years ago
|
Flags: needinfo?(valentin.gosu)
Assignee | ||
Comment 4•4 years ago
|
||
Assignee | ||
Updated•4 years ago
|
Assignee: juhsu → valentin.gosu
Flags: needinfo?(valentin.gosu)
Pushed by valentin.gosu@gmail.com: https://hg.mozilla.org/integration/autoland/rev/75c91a941601 Replace GetInProcessScriptableTop usage with TopWindowContext() in WebSocket.cpp r=baku
Comment 6•4 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox82:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•