resourceloader: Remove top/bottom queue distinction
authorTimo Tijhof <krinklemail@gmail.com>
Mon, 7 Nov 2016 23:47:15 +0000 (23:47 +0000)
committerKrinkle <krinklemail@gmail.com>
Wed, 16 Nov 2016 19:29:16 +0000 (19:29 +0000)
commitbc374082fa1a0e459ef816908e4d3cb7956ef1aa
tree677b525021b1e17377fcba021aeb38a22d179ec6
parent2196833990de3c0c96e3c4ad2364d9baae5eb28e
resourceloader: Remove top/bottom queue distinction

* The styles queue has always been top-only
  (except for a few months in 2015).
* The top queue loads asynchronous since mid-2015. (T107399)
  And LocalStorage eval, previously the last remaining non-async part
  of module loading, is also async as of October 2016. (T142129)

* This change merges the bottom 'mw.loader.load()' queue with the top queue.
  It also moves any other snippets potentially in the bottom queue still:
  - embed: I couldn't find any private modules with position=bottom
     (doesn't make sense due to their blocking nature). If any do exist,
     (third-party extensions?), they'll now be embedded in the <head>.
  - scripts: Any legacy 'only=scripts' requests will now initiate
     from the <head>.

Bug: T109837
Change-Id: I6c21e3e47c23df33a04c42ce94bd4c1964599c7f
includes/resourceloader/ResourceLoaderClientHtml.php
includes/resourceloader/ResourceLoaderModule.php
tests/phpunit/includes/resourceloader/ResourceLoaderClientHtmlTest.php