resourceloader: Fix RLQ script to support IE8 quirk
authorTimo Tijhof <krinklemail@gmail.com>
Tue, 4 Aug 2015 23:07:45 +0000 (16:07 -0700)
committerTimo Tijhof <krinklemail@gmail.com>
Tue, 4 Aug 2015 23:25:11 +0000 (16:25 -0700)
commit7facd8b214d36ddd3105bb5ee0e56f6ec17beba1
treeafe28c4537e959b0ce557292329cee0e7d89c951
parent315855f2acb7db43d6e6f3563bca30c75c43ecc7
resourceloader: Fix RLQ script to support IE8 quirk

In IE8 there is a race condition. If the window.RLQ runs first (that is, the
startup module loads first) then 'var RLQ' does not get associated
properly to our custom window.RLQ object.

The other way around is fine, but never happens because <script async>
isn't supported in IE8.

Consistently use 'window.RLQ' (never 'var RLQ') to bypass this bug.

Also updating the startup.js file for consistency (though not strictly
needed since the implicit global reference without 'var' also works fine
in IE8.

Bug: T107954
Change-Id: I3f46fee7c4528abf806bb9c51fc767eceb795009
includes/resourceloader/ResourceLoader.php
resources/src/startup.js
tests/phpunit/includes/OutputPageTest.php