1367286 - Remove preference "dom.userHTMLBindings.uri"

Status: RESOLVED FIXED

(Core :: XBL, enhancement)

Description

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Per bug 1363640 comment 29, in practice it's unlikely someone ever uses  "dom.userHTMLBindings.uri" preference [1]. We should remove it.

[1] http://searchfox.org/mozilla-central/rev/2933592c4a01b634ab53315ce2d0e43fccb82181/dom/xbl/nsXBLWindowKeyHandler.cpp#115

Comment 1

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Boris, I guess you mean "remove the ability of loading user binding" in [1], not only just the preference, right?

[1] http://searchfox.org/mozilla-central/rev/2933592c4a01b634ab53315ce2d0e43fccb82181/dom/xbl/nsXBLWindowKeyHandler.cpp#114-127

Comment 2

[Boris Zbarsky [:bzbarsky]]

Yeah, I think we can just remove that whole bit.

Comment 3

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Attachment: Bug 1367286 - Remove loading user html binding via "dom.userHTMLBindings.uri".

Review commit: https://reviewboard.mozilla.org/r/142490/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/142490/

Comment 4

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Boris, perhaps you'd want to review this?

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8870920 [details]
Bug 1367286 - Remove loading user html binding via "dom.userHTMLBindings.uri".

https://reviewboard.mozilla.org/r/142490/#review146080

r=me

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b8d146db72f5
Remove loading user html binding via "dom.userHTMLBindings.uri". r=bz

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/b8d146db72f5