Closed Bug 1571066 Opened 1 year ago Closed 3 months ago

nsDocShell::GetWindowProxy should return a WindowProxyHolder rather than a BrowsingContext*

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla72
Fission Milestone M4
Tracking Status
firefox72 --- fixed

People

(Reporter: kmag, Assigned: kmag)

Details

Attachments

(1 file)

No description provided.
Status: NEW → ASSIGNED
Fission Milestone: --- → M4
Priority: -- → P3

Returning BrowsingContext* from these sorts of getters tends to lead to
misuse. WindowProxyHolder is the semantically correct type to return here, in
any case.

Pushed by maglione.k@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/4ebefd304d2f
Return WindowProxyHolder from nsDocShell::GetWindowProxy. r=peterv
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.