resourceloader: Remove wgResourceLoaderValidateStaticJS setting
authorTimo Tijhof <krinklemail@gmail.com>
Mon, 20 Aug 2018 19:19:24 +0000 (20:19 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Mon, 20 Aug 2018 19:25:01 +0000 (20:25 +0100)
commit3bd1910f95bc64ea88afa40ead5f934ca3581e29
tree3e3cbd32d39dcfeac18f75938f996420fcaa0d4b
parente8d99088916b1c4a4cde7a93730c1337b86ecc6a
resourceloader: Remove wgResourceLoaderValidateStaticJS setting

Not used since 2011 (MediaWiki 1.18).

In an early version of ResourceLoader, we ran JSMinPlus syntax
validation on-demand on all served JavaScript content. This
was identified as cause of slowdown and high memory use, and
generally not considered as useful in production.

The reason it was there originally was not for the purpose of
validating static files, but for user-generated content.

So in MediaWiki 1.18, the behaviour of wgResourceLoaderValidateJS
was changed to only apply to user-generated content, and the
rest was disabled behind wgResourceLoaderValidateStaticJS, which
we then never use. Not even in development, given that we now
have superior experience through ESLint, even within IDEs where
supported.

Follows-up 49d3d18033738 (r91914).

Change-Id: Ie25109a4fb23ee93fed0db4af5db4b11fe9ffe7f
includes/DefaultSettings.php
includes/resourceloader/ResourceLoaderFileModule.php