mw.loader: Fix regression that caused CSS load after scripts.

Follows-up 705d50c which introduced cssText buffer (yielding
one tick of the event queue to reduce the number of forced
repaints).

However since that made CSS loading asynchronoous (be it only for
a split second) it caused some nasty side-effects.

This was also reflected in the unit tests that had to resort
to doing setTimeout from within the mw.loader implemented script
to assert the styles. This is now sane again: Scripts execute
after styles are inserted.

Bug: 46401
Change-Id: Ib54a3e78710b7f798c6730d3f540d3284001e2de

Revert "mw.loader: Fix regression that caused CSS load after scripts."

Broke master tests since it causes a JSDuck issue. See bug 47018.

This reverts commit be5f73b595b8fef0697bf83e9ebd13e85756c099

Change-Id: I2d945b7ffb7cfbcc5024a1c3bdcf0c9506b0cf76

Add missing wfProfileOut before throwing an exception

Change-Id: I1d830da0597f19efd0b2ae48642389975e736e23

Add missing wfProfileOut to LocalisationCache

Follow up Ib94a8c18c4e270a7bd46faa17eb27a22ff950f75

Moved some empty lines to match the places of other profile calls in the
file.

Change-Id: I6616b18027cdb9e70b43948aaff999a2e25b07e0

Merge "LocalisationCache: add profiling, fix doxygen warnings"

Merge "FormatJson: microoptimizations for UTF8_OK mode"

Merge "mw.loader: Fix regression that caused CSS load after scripts."

Merge "Call ApiPageSet::getFinalParamDescription"

Merge "Normalize as and or language names"

Merge "Avoid fatal errors with uselang=s"

FormatJson: microoptimizations for UTF8_OK mode

* Replace strtr with str_replace where faster.
* Use addcslashes to escape json_encode's output. Because no control
  characters are included, the only characters that have to be
  escaped are \ and ". (irrelevant for PHP 5.4+ installations)

Re-encoding a ~1.5 MB API response from the Chinese Wikipedia:
* PHP 5.3: 32% faster (from 347 ms to 239 ms)
* PHP 5.4: 70% faster (from 51 ms to 15 ms)
* HHVM: 42% faster (from 326 ms to 191 ms)

Change-Id: I7c9342682986d40a2f2436ac978390b6018a3521

Fix release notes for I5e6f7d48d6904.

Change-Id: Id442e6fcbdc410798579bc0bf1088ccede08e21b

Localisation updates from translatewiki.net.

Change-Id: I182906ee394da739d6b8fb64bc7743c05b3b6ac6

Merge "Update formatting and docs"

Update formatting and docs

Also remove some unneeded else{} statements and reverse if/else if that
reduces indentation.

Change-Id: I1f7bd26119850fc944fbe84ae78bdd95cafc7bbb

Merge "startup: Add unit tests and add Firefox < 4 to blacklist."

Remove the "Quickbar" feature and all things related to it

"Quickbar" was a feature of the Standard and CologneBlue skins that
allowed the sidebar to be displayed on left or right side of the page,
floated or fixed, and hidden on diff pages.

Standard was removed (Ia6d73c2d), and CologneBlue doesn't support this
anymore (bug 41246), so all things quickbar can now be safely removed.

* Removed user prefs option + interface
* Removed related messages
* Removed code for this in SkinLegacy
* Removed dead code in DifferenceEngine and Language

Change-Id: I5e6f7d48d6904a052a3a11547d3ebe6161463018

Localisation updates from translatewiki.net.

Change-Id: Icab7dc96cc3e81918512ea4f4b55fea2bd6cf7d5

Merge "remove references to removed skins from mediawiki.util.js"

remove references to removed skins from mediawiki.util.js

Dead code removal. Followup to Ia6d73c2d.

Change-Id: Ic43462d6775af682626e43a1bdfab3f8077f0101

Merge "Remove link to Special:ActiveUsers from Special:Statistics"

Merge "fixed php doc in ApiBase"

Remove link to Special:ActiveUsers from Special:Statistics

Besides being broken, causes PHP warnings.

Bug: 46719
Change-Id: I23d9c9bdc9d38eb4256d0f7061e225cf86b576f8

Localisation updates from translatewiki.net.

Change-Id: I222f66b12c0d6bdb03c7105b3d598304f9ffd61c

LocalisationCache: add profiling, fix doxygen warnings

Change-Id: Ib94a8c18c4e270a7bd46faa17eb27a22ff950f75

Merge "Protected function UploadBase->validateName changed to public"

Protected function UploadBase->validateName changed to public

*Any class that is not part of the UploadBase class
(so it can't call protected functions or members on an UploadBase object).
If you, for example,build an extension that uses an UploadBase instance,
there is no way to get the title validation error.

Bug: 38221

Change-Id: Ie6c3f95a09ef84777313c3ec2edd31c50c397e28

Merge "Fixed fatal when using fake load balancers."

startup: Add unit tests and add Firefox < 4 to blacklist.

Bug: 35906
Change-Id: Id8214c6a54f1bed2f9e1abb668248c46dcf7317e

Merge "New parserTest: check interaction of italics and English possessives."

Merge "Move table parsing tests from Parsoid's whitelist into upstream parserTests."

Merge "Move definition list parser tests from Parsoid's whitelist into upstream."

Merge "Move bold/italic parser tests from Parsoid's whitelist into upstream."

New parserTest: check interaction of italics and English possessives.

This documents a current bug in the Parsoid parser.

Change-Id: I242324237640dd21fb24c50fa164a45091e388b8

Move table parsing tests from Parsoid's whitelist into upstream parserTests.

A few more differences between Parsoid and the PHP parser.

Change-Id: Ic12c93fef905b65ffa459bb15920ea7edd9f2238

Move definition list parser tests from Parsoid's whitelist into upstream.

Parsoid parses certain uses of definition lists differently than the PHP
parser.

Change-Id: I185a805e117fc1b9ee3c0cabc657b48cf39af52b

Move bold/italic parser tests from Parsoid's whitelist into upstream.

Parsoid has been maintaining a whitelist for tests where its output
diverges from the PHP parser.  This patch upstreams part of that whitelist,
creating separate PHP and Parsoid test cases to document in one place
where the parsed output diverges and why.  (Uses the recently-added
'php' and 'parsoid' options in parserTests.)

Change-Id: I07ca6ec1e039a2842c641fe543b2d92eb964d932

Call ApiPageSet::getFinalParamDescription

This allows extensions to modify the param description of the ApiPageSet
params at once for each module, which used the ApiPageSet.

Change-Id: I2ed6f3ad38d3d84182b8525a6b247f721be7f460

Avoid fatal errors with uselang=s

languages is not a subclass of Language

Force valid built-in language codes to be at least two letters long.

Bug: 46710
Change-Id: Ife8885ed4bdb3a017a54e063f5dc58468ba231d5

fixed php doc in ApiBase

Change-Id: I9c893de8c067be3e926f5de5f0b87ac47b81772d

Merge "Added "cluster" parameter to wfWaitForSlaves() for external DBs."

Merge changes Ic22a190c,Ia5b37f99

* changes:
  Removed duplicate class registration
  Removed pointless static declaration

Merge "mergeMessageFileList.php: abort on read error"

Removed useless clear() call in WikiPage::doPurge().

Change-Id: I3bc251ea987f089342c054afd5a02ccf321f4fe7

mergeMessageFileList.php: abort on read error

If there is a parse error or if one of the files in the extension list
doesn't exist, exit with an error. mw-update-l10n has set -e so it will
abort without syncing the new file.

Change-Id: Idaad65783127b075626c102a8dc02e22df1588b7

Added "cluster" parameter to wfWaitForSlaves() for external DBs.

Change-Id: I226a1793dfb901de8e1c0ac40c1dc3ad890de95a

Merge "Localisation updates from translatewiki.net."

Merge "Set lang in api createaccount regardless of $wgLoginLanguageSelector"

Localisation updates from translatewiki.net.

Change-Id: Id8c6772f24ba3a569be66e210a99b89c3091e9a3

Fixed fatal when using fake load balancers.

The error was:
Catchable fatal error: Argument 1 passed to DatabaseMysql::masterPosWait()
must implement interface DBMasterPos, boolean given, called in
/var/www/DevWiki/core/includes/db/LoadBalancer.php on line 419

Change-Id: I230585930c1424b0868f9aa290167c66bc6848bd

Merge "Adding wfLogWarning for production warnings."

Remove ExternalUser authentication code

This was an experimental authentication system intoduced a couple
of years ago with a pretty narrow use-case. It's been pretty much
ignored since introduction, and makes login more complicated than
it needs to be.

I didn't drop the external_user table on the off-chance someone
out there actually has data in it, but they should use AuthPlugin
for their external authentication needs.

Change-Id: I794338dbb75961ee033d41fa44bb7aa22e54f447

Normalize as and or language names

Compared which of these differ if run through UtfNormal::cleanup.
Not doing this causes all kind of nasty bugs if they end up used
for page titles. See the linked bug.

Bug: 38712

Change-Id: Ieaa429b921bc93e3a40b70a76da481c11dc9c3f6

Merge "Drop ClickTracking integration from mw.user.bucket"

Merge "Add to/tweak mediawiki and jqueryMsg JS tests, fix comment"

Improve mediawiki.js docs:

* Fix an error in the Map.set documentation.
* Use JSDuck formatting in more places.

Change-Id: I5e2d271b45c1dbb33fa67abe4a2cc79380831a23

HTMLForm button behaviour modifications

- Buttons can now be marked disabled and they will not be clickable.
- The default submit button may now be opted out of.

Change-Id: I8293a3c09da51152bfc7f6f6951416c71ce0bb38

Drop ClickTracking integration from mw.user.bucket

This change set drops the 'tracked' option from mw.user.bucket, which
previous triggered logging via the ClickTracking extension.

Change-Id: I1a57df2ba60dba6041ecad7dd5c87519eb5d143e

Most important documentation fix

Change-Id: I0482144112345ca76ed3d95e3ad3b846cce4d87b

Merge "(bug 46615) updateCollation.php: sanity check the collation before proceeding"

Merge "Remove non-functional handling of curl_error()"

Merge "ResourceLoader: Honor $wgAllow* settings in site/user modules"

Merge "Test: Assert that modules loaded correctly."

Merge "OutputPage: Set empty modules to state 'ready' instead of 'missing'."

Add to/tweak mediawiki and jqueryMsg JS tests, fix comment

* Remove getOuterHtml.
* Use htmlEqual where getOuterHtml was used, among other places.
* The comment reflected a change I considered making, but did not in the end.
* Add a few link tests
* Minor tweaks to other test code for consistency
* Replace wikiGetlink with explicit output

Change-Id: Id6a857c9686901f8db4944fcc081d4b9c4457fa8

Specify the utf-8 charset on error messages.

Although the contents of error messages are English text,
and thus it's irrelevant that it gets interpreted as
iso-8859-1 instead of utf-8, $wgSitename is printed by
DBConnectionError::searchForm(), and that can contain utf-8.

Cached pages may also be in utf-8.

Change-Id: I81c02493f0824cb010c80f4356f338787d78df6b

Merge "jquery.makeCollapsible: events for collapsing/expanding, tests"

Merge changes Icdd8f392,Ied13e5ab

* changes:
  jquery.makeCollapsible: basic test suite
  jquery.makeCollapsible: use 'mw-collapsible' event namespace

Merge "(bug 36400) API: Fix sorting for iwlinks, langlinks"

ResourceLoader: Honor $wgAllow* settings in site/user modules

Bug: 46858
Change-Id: I17b3a2c1df5d52458f92c715242003b34111432c

Test: Assert that modules loaded correctly.

Since mw.loader catches any uncaught exceptions that modules
throw at run time, it is hard for QUnit and/or the browser
(e.g. PhantomJS) to know about a failure because there is
no assert failure and no uncaught exception (window.onerror).

VisualEditor's init module, for example, has a problem (bug 45175)
that sometimes causes an error, causing none of the tests to run,
yet the QUnit run would finish successful with simply none of the
VE tests included in the results.

Bug: 44299
Change-Id: Ib6e2b8d1be3e38fd9f1b948407c62da550fce0b4

Added Job::hasRootJobParams() convenience function.

* Added a missing @since tag.

Change-Id: I887d55074007551040eee486dbb9e8e44106e5a9

OutputPage: Set empty modules to state 'ready' instead of 'missing'.

Some cases were already covered by I595b3af190, but in the case
where the module is disabled entirely it was setting it to missing.

This is inconsistent with the behaviour of load.php (which will implement
an empty module resulting in state=ready) and confusing as it
implies there is a problem (a missing module) whereas in fact it
isn't a problem. Modules that depend on it to allow customization
wouldn't load as it will fail on missing dependencies.

Also makes asserting problems (bug 44299) hard as it would have
to whitelist state 'missing' for user/site/user.groups.

Bug: 46857
Change-Id: I46b5bc6f714abe4b66186d7b7a14c69195f00ee1

Added some missing @since tags.

Change-Id: I31c264aebf63f4bcaa3042f624e91dc27f5687e8

Merge "Localisation updates from translatewiki.net."

Merge "[FileBackend] Clarified the noAccess/noListing docs a bit."

Cleaned up return types for getAllQueuedJobs() and getAllDelayedJobs().

Change-Id: I305941c817dd13427d291b9265a71f9df9130f1c

Localisation updates from translatewiki.net.

Change-Id: I3cdedadb10249e5ea330e89e4d7f2ab187f804cb

(bug 36400) API: Fix sorting for iwlinks, langlinks

The iwlinks and langlinks modules continue parameters imply ordering by
page then prefix then title. But in certain modes, the actual queries
use a different ordering, which may result in skipped or repeated
results.

This changeset fixes that. To do so, it needs to re-add an index
iwl_prefix_from_title which was mistakenly removed in 2010 (r69721). And
while it's doing that, it cleans up errors in the sqlite and postgresql
handling of the iwlinks indexes too.

Also, per Asher, make the iwl_prefix_from_title and
iwl_prefix_title_from indexes non-UNIQUE.

Change-Id: I607e8bf9183a2d8152a6127a81c83a0b5bba0c61

Improve API 'origin' parameter description

When making CORS pre-flight requests, the origin parameter must still be
included. This means that it cannot be in the POST body, as that is not
sent for pre-flight.

Document that.

Change-Id: I3fc809cf2e4ca9c2566f62d32f96d24377c72863

jquery.makeCollapsible: events for collapsing/expanding, tests

Trigger a custom event to allow callers to hook to the
collapsing/expanding, allowing the module to be testable and making
it possible to e.g. implement persistence via cookies.

Added tests for the triggers themselves and for some basic
asynchronous collapsing functionality.

The following four events are fired (the namespace used is the same
one as the one used for firing the click event for initial toggle):
* beforeExpand.mw-collapsible
* beforeCollapse.mw-collapsible
* afterExpand.mw-collapsible
* afterCollapse.mw-collapsible

Change-Id: I580365770d061418a6734fcb54b2d86f2a821748

jquery.makeCollapsible: basic test suite

For now it just verifies that the very basic synchronous functionality
of the module works; not much more is possible without triggering some
events to hook to, and currently jquery.makeCollapsible doesn't
trigger any.

Change-Id: Icdd8f392e4921007a6bdfb3ae934949e4c875232

Remove spaces around ::

Change-Id: I24fe5ad907bcf5235aa91b650785c571502456a1

Merge "phpcs: Pass includes/resourceloader/"

phpcs: Pass includes/resourceloader/

- A cast statement must not be followed by a space.
- The method parameter $context is never used.
- Avoid function calls in a FOR loop test part.
- Opening brace should be on the same line as closing parenthesis.

Change-Id: I0eba7fcc9ceab372003d1134857346690c525e87

jquery.makeCollapsible: use 'mw-collapsible' event namespace

For consistency with the module name.

Change-Id: Ied13e5abdec107771e80418feb4bb3b2f0d54567

Deferred page_touched update via onTransactionIdle.

* This should reduce deadlocks and lock wait timeouts.

Change-Id: I595bc33d7643e7964d796b1d3da31f7cfab55024

Merge "In SqlBagOStuff::getMulti(), catch connect errors"

In SqlBagOStuff::getMulti(), catch connect errors

This case was somehow neglected in Ifba0d276. The other methods work
correctly.

Change-Id: If29f0212f07c13bd72469c580bb096a2a41a96b5

Merge "[LockManager] Added timeout to the lock() function."

Merge "Avoid duplicated effort in JobQueueAggregatorRedis::doGetAllReadyWikiQueues()."

[LockManager] Added timeout to the lock() function.

Change-Id: I32f97b813fc6036b2774d67c14c574420e7a1437

Merge "Add wfProfileOut to the new returns added by 19ecb69f"

Avoid duplicated effort in JobQueueAggregatorRedis::doGetAllReadyWikiQueues().

Change-Id: I76bfc23ecf94c974a9eefc7d430b26d70187409e

Merge "Use canonical class name."

Merge changes I8590352b,I164d60e0

* changes:
  ResourceLoader: Change $module by ref instead of resetting the key
  QUnit: Ensure test modules are top loading

Use canonical class name.

Change-Id: I7dc9980d334bc5ad6421135d713d890a8588281c

Add wfProfileOut to the new returns added by 19ecb69f

Change-Id: I97def12da054db7204955b95c3d590a3ab24ba55

ResourceLoader: Change $module by ref instead of resetting the key

Change-Id: I8590352b52ecf51def21a9c52af85aba0f050ff8

QUnit: Ensure test modules are top loading

Jenkins tests for extensions were exiting with a false positive.
They were marked "success" but in fact were wrong because
it only ran the core tests, the extensions' test suite was
loaded after QUnit already stopped listening.

Change-Id: I164d60e06118b64a1c6a17665a0fb1faeff744e5