resourceloader: Un-deprecate makeLoaderConditionalScript()
authorTimo Tijhof <krinklemail@gmail.com>
Fri, 2 Mar 2018 01:12:03 +0000 (17:12 -0800)
committerTimo Tijhof <krinklemail@gmail.com>
Fri, 2 Mar 2018 01:12:03 +0000 (17:12 -0800)
commitb004fb874ba6b4588e46d76f867ff59eee63730c
treeacc77245c127d135090509b6af693f04047b812e
parent5e3b2469991ebfc3ba96bb377b9c3847410b2f86
resourceloader: Un-deprecate makeLoaderConditionalScript()

It was deprecated to encourage callers to use makeInlineScript()
instead, because before makeInlineScript() was introduced, callers
would often manually wrap it in a <script> tag, which we don't want
because 1) We want de-duplication through WrappedString and 2)
callers sometimes forget to use Html.php methods that ensure
proper escaping as-needed.

However, there is still use in getting thet wrapped script without
also the wrapped HTML script tag. Namely, for use in http responses
that produce JavaScript, rather than HTML, where there cannot be
an HTML script tag.

There is at least one such caller (in SpecialJavaScriptTest) and
it's fine for there to be others.

Change-Id: Icfc285e12293c76dc68a220deb6816e32b058209
includes/resourceloader/ResourceLoader.php