Closed Bug 1597449 Opened 5 years ago Closed 5 years ago

Fix uses of mozilla::dom::BrowserChild::RecvSetWindowName in dom/ipc/BrowserChild.cpp

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla75
Fission Milestone M6
Tracking Status
firefox75 --- fixed

People

(Reporter: djvj, Assigned: tetsuharu)

References

(Blocks 1 open bug)

Details

(Whiteboard: [rm-docshell-tree-item:simple])

Attachments

(1 file)

In file dom/ipc/BrowserChild.cpp

Only usage is to QueryInterface WebNavigation into a DocShellTreeItem, and then call SetName on it.

Implementation of SetName already forwards to mBrowsingContext.

Change to get the mBrowsingContext directly off of WebNavigation and update there.

Only implementors of nsIWebNavigation are nsDocShell and nsWebBrowser anyway.

Kannan says replacing nsIDocShellTreeItem calls should block enabling Fission in Nightly (M6).

Fission Milestone: --- → M6
Priority: -- → P3
Assignee: nobody → tetsuharu.ohzeki
Status: NEW → ASSIGNED
Attachment #9123775 - Attachment description: Bug 1597449 - Fix uses nsIDocShellTreeItem of mozilla::dom::BrowserChild::RecvSetWindowName(). → Bug 1597449 - Remove mozilla::dom::BrowserChild::Send/RecvSetWindowName() to remove usages of nsIDocShellTreeItem.
Pushed by tetsuharu.ohzeki@gmail.com: https://hg.mozilla.org/integration/autoland/rev/ef53e2f57eeb Remove mozilla::dom::BrowserChild::Send/RecvSetWindowName() to remove usages of nsIDocShellTreeItem. r=kmag
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: