Remove nsIWebBrowserFocus

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P2
normal
RESOLVED FIXED
Last year
11 months ago

People

(Reporter: qdot, Assigned: qdot)

Tracking

(Blocks 1 bug)

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

Only implemented by nsWebBrowser, only accessed via C++ in TabChild. One less interface to QI to, especially if we just store a RefPtr<nsWebBrowser> ptr in TabChild instead of constantly routing through nsIWebBrowser.
Component: General → Document Navigation
Priority: -- → P2
Product: Toolkit → Core
Only implemented by nsWebBrowser, only 2 methods used in TabChild.
Move methods to nsWebBrowser implementation and remove unused methods,
change names to something more obvious, and remove interface.

MozReview-Commit-ID: 4WwBrVWQEVy
Comment on attachment 8997629 [details]
Bug 1480645 - Remove nsIWebBrowserFocus; r=nika

:Nika Layzell has approved the revision.

https://phabricator.services.mozilla.com/D2752
Attachment #8997629 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/82e7fc398926
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.