Remove nsIWebShellServices

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P2
normal
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: qdot, Assigned: qdot)

Tracking

(Blocks 2 bugs)

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

nsIWebShellServices is only implemented by nsDocShell, and only in C++ (via nsIDocShell). We can move the function definitions to the nsIDocShell interface and remove the need for the QI in the HTML parser.
nsIWebShellServices is only implemented by nsDocShell, and only used
in one place in C++. Move definitions to nsIDocShell, and rename
functions to show they are only used as part of Charset changes.

MozReview-Commit-ID: DOSeE3Doc51
Priority: -- → P2
Comment on attachment 8997229 [details]
Bug 1480628 - Remove nsIWebShellServices; r=nika

:Nika Layzell has approved the revision.

https://phabricator.services.mozilla.com/D2692
Attachment #8997229 - Flags: review+

Comment 5

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c629114f55f0
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.