1480643 - Remove nsIWebBrowserSetup

Status: RESOLVED FIXED

(Toolkit :: General, enhancement)

Description

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Only used by TabChild in C++, we can fold this into nsWebBrowser.

Comment 2

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

We should just be able to make a AllowDNSPrefetch function on nsIWebBrowser and call that instead of SetProperty. There are no other usages of the nsIWebBrowserSetup enums, so they can be removed entirely with the interface.

Comment 3

[Kyle Machulis [:qdot] [:kmachulis] (INACTIVE)]

Attachment: Bug 1480643 - Remove nsIWebBrowserSetup; r=nika

We only use one branch of the property set method in
nsIWebBrowserSetup, in one place. Expose this setting in the C++ API
and remove the XPCOM interface.

This patch also exposes the nsWebBrowser.h header to the codebase,
meaning we can possibly start removing some uses of nsIWebBrowser
elsewhere.

MozReview-Commit-ID: G3gnRWJUx6M

Comment 4

[Phabricator Automation]

Comment on attachment 8997549 [details]
Bug 1480643 - Remove nsIWebBrowserSetup; r=nika

:Nika Layzell has approved the revision.

https://phabricator.services.mozilla.com/D2736

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/796b4331f06a
Remove nsIWebBrowserSetup; r=nika

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/796b4331f06a