Merge "Add tests for API's assert={user|bot}"

Revert "Revert "Officially deprecate skin autodiscovery""

The CI issue that made the revert necessary is fixed now.

This reverts commit d672aa3123a612583ef9ec67115992aafce5b2af.

Bug: 66429
Change-Id: Ib2b4a895c5d01e5ccc7e4edfdbfdcaf0a93c267c

Merge "Fix some line mis-wrapping and typos in RELEASE-NOTES-1.24"

Merge "Update jquery.jStorage.js"

Merge "Collapse duplicate $existingExpiry == 'infinity' case into else"

Collapse duplicate $existingExpiry == 'infinity' case into else

Change-Id: I1cc612e9f646dd75f58cc6ae0ae073ebffa7d6d3

API: Allow 'infinity' as a valid protection expiry

Change-Id: If9eb1f3e835579f5c8b8be22297a1eb26beda4e7

Switch API to use Config classes

Only done where globals are config (so not $wgParser, $wgContLang etc)

Change-Id: Ic39cdd858cfb9096a2bc09618f97e64270d76f13

Localisation updates from https://translatewiki.net.

Change-Id: Id7f62d5c54ac1d476f37d238759a28ef803347fb

Update jquery.jStorage.js

Project site

* http://www.jstorage.info/ and https://github.com/andris9/jStorage

jquery.jStorage.js file

* https://github.com/andris9/jStorage/blob/master/jstorage.js

Author and contact

* Andris Reinman and email andris.reinman@gmail.com

README

* https://github.com/andris9/jStorage/blob/master/README.md

Change-Id: I0ca42e909cc08f8f0b67cec0a7c0aedd0ebf91d7

Merge "mw-ui mixins: More colors, more browser helper mixins"

mw-ui mixins: More colors, more browser helper mixins

Change-Id: I895679ffe2f5eb1f002ff110c750e77824004820

Localisation updates from https://translatewiki.net.

Change-Id: Id1349f603b9e9c7d7c039a7fbea03ccf69238d89

Fix some line mis-wrapping and typos in RELEASE-NOTES-1.24

Change-Id: I08c513ed6418f58b1d269456affd02d882ab8e9b

Localisation updates from https://translatewiki.net.

Change-Id: I70982e6a517a7fd472acb8577adde727cabd7a21

Add SpecialEditWatchlist::prefixSearchSubpages

Also add 'clear' to SpecialWatchlist::prefixSearchSubpages

Change-Id: Ibc1ca44a9472cd86a6dc5df04274089b8fcf30a2

Merge "Change MostLinkedTemplates to MostTranscludedPages"

Merge "Made FSFileBackend use closures for doExecuteOpHandlesInternal()"

Change MostLinkedTemplates to MostTranscludedPages

Rename Special:MostLinkedTemplates to Special:MostTranscludedPages, and
allow it to return pages from any namespace.

Change-Id: I27c2fb50f77aef760f8c7006ae48297451230c6f

Merge RELEASE-NOTES-1.23 into HISTORY

Used the REL1_23 version of that file, and moved the note added in
5e48ad2e78ce to RELEASE-NOTES-1.24, where it belongs.

Change-Id: Id5f4726152fc2f4c90117762f6dc4904f5fa9f94

Localisation updates from https://translatewiki.net.

Change-Id: Ib84e5a1bc0740a2948a2a902e6916fa3f9c1627a

Merge "Remove $wgDBClusterTimeout"

Made FSFileBackend use closures for doExecuteOpHandlesInternal()

Change-Id: I8401a92384cb454eb79a4607b350c50d432a9510

Merge "Apparently for certain (API) requests $this->getTitle() doesn't return a valid Title."

Remove $wgDBClusterTimeout

Also added release notes that I forgot to include in
change I4b409ef

Change-Id: I9416c0dc7dea1c32c3e1f6d7177ee9bc4e34e1de

Merge "Deprecate $wgMasterWaitTimeout"

Use type hinting in ApiBase

For ideological reasons this was not included in Ie6bf1915.

Change-Id: I5f7119665746eb6fcf86c3a403caa2dcb67904eb

Autocomplete special pages' subpages in search suggestions

Includes basic "example" implementations for:
* Special:Watchlist (predefined subpage list)
* Special:Log (based on config variable)

Bug: 20210
Change-Id: I2b0505df244b9d23bd23d5fb4d3405dc28d57739

Merge "Vector: Workaround for legacy IE font-sizing issues"

Merge "Create PostLoginRedirect Hook for changing the redirect behavior"

Vector: Workaround for legacy IE font-sizing issues

In IE <= 7, 'Ctrl' + '+' scales the font size in far too large steps if the
root element's font size is given in the 'em' unit. Everything becomes normal
if the root element's font-size is given in '%'. This should not have any
effect in other browsers.

Bug: 24515
Change-Id: I5cc864d73935a02713075840b7ff80000b3ae88a

Deprecate $wgMasterWaitTimeout

Not only was this global unused, it also caused the LoadBalancer
to look for the wrong array key, which would always set
$mWaitTimeout to 10 in the conditional. There is also
a manual function LoadBalancer::waitTimeout() for this.

Change-Id: I4b409ef890bec5aac7f666a2ed3dff9ea8bc14cc

resourceloader: Implement "skip function" feature

A module can be registered with a skip function. Such function,
if provided, will be invoked by the client when a module is
queued for loading. If the function returns true, the client will
bypass any further loading action and mark the module as 'ready'.

This can be used to implement a feature test for a module
providing a shim or polyfill.

* Change visibility of method ResourceLoader::filter to public.

So that it can be invoked by ResourceLoaderStartupModule.

* Add option to suppress the cache key report in ResourceLoader::filter.

We usually only call the minifier once on an entire request
reponse (because it's all concatenated javascript or embedded
javascript in various different closures, still valid as one
large script) and only add a little bottom line for the cache
key. When embedding the skip function we have to run the minifier
on them separately as they're output as strings (not actual
functions). These strings are typically quite small and blowing
up the response with loads of cache keys is not desirable in
production.

* Add method to clear the static cache of ResourceLoader::inDebugMode.

Global static state is evil but, as long as we have it, we at
least need to clear it after switching contexts in the test suite.

Also:
* Remove obsolete setting of 'debug=true' in the FauxRequest in
  ResourceLoaderTestCase. It already sets global wgResourceLoaderDebug
  in the setUp() method.

Bug: 66390
Change-Id: I87a0ea888d791ad39f114380c42e2daeca470961

Update OOjs UI to v0.1.0-pre (c5ae888f67)

New changes:
9eec766 Create 'next' and 'previous' indicators

Change-Id: I3f091f25be73fe35a55b113630c253c9a3cf5b38

Update OOjs UI to v0.1.0-pre (9f4f250f88)

New changes:
bf4ba74 Localisation updates from https://translatewiki.net.
a99acae Remove use of imaginary config option in OutlineControlsWidget
8db7499 Localisation updates from https://translatewiki.net.
7d7b9ce [BREAKING CHANGE] Window process cleanup

Change-Id: Ib9ae3fdfe14bbeb1f687b26f552cf4fb49db2d69

Merge "Remove $wgProxyKey"

Merge "RCFeed: Add log_id property (exposes rc_logid attribute)"

RCFeed: Add log_id property (exposes rc_logid attribute)

Bug: 20394
Change-Id: I005be8e41a5cbea934d4dac27184f03e9af7c648

Remove $wgProxyKey

Change-Id: I1748164f108fab701c79532ede13ae387b12769d

Localisation updates from https://translatewiki.net.

Change-Id: Ib84999f8508bc7bd8843eae28a8db1a35b1628be

Apparently for certain (API) requests $this->getTitle() doesn't return a valid Title.

Usually these are API siteinfo requests (api.php?action=query&meta=siteinfo). In that case, this code used to produce a fatal ("Call to a member function getPageLanguage() on a non-object").

Change-Id: Ifb6f99fe554890ff2719220f58d1b6c1a7a95ddc

Reduce deadlocks adding log rows to the RC table on delete

bug: 56776
Change-Id: Ic75530b25f3fa5668019c76ce59a1dda2768f389

Merge "Fix misnamed SVGHandler::canAnimateThumb()"

Merge "Remove isset() check in Sanitizer::setupAttributeWhitelist()"

Merge "Cleanups to WebRequest::getIP logic"

Merge "Add a maintenance script to list variants and languages with then"

Merge "MimeMagic: Don't seek before BOF"

Cleanups to WebRequest::getIP logic

* Throw an error if there is no immediate server IP, otherwise the
  XFF logic is wonky.
* Refactored the loop to be a bit easier to read.
* Better handle "unknown" entries in the XFF chain.

Change-Id: I9541afa408d895c3fd337a883ecfe4ce0ba57090

MimeMagic: Don't seek before BOF

This has weird side effects like only extracting the tail of the
file partially or not at all.

Bug: 66428
Change-Id: I182128c6958244f1515227ee742c3206a7484aee

History: Simplify checkboxes script on History pages

This should reduce the amount of variables constructed and also switches
to a class based implementation, which allows us to do the same with
fewer calls touching the DOM.

Not supported by IE6.

Bug: 51561
Change-Id: I4596b3667f990ef1d8c7ca753e1d80fe285d5614

Remove obsolete RC_MOVE and RC_MOVE_OVER_REDIRECT rc types

RC_MOVE and RC_MOVE_OVER_REDIRECT are obsolete, since at least 2006.
So it is considered safe to remove.

Bug: 63755
Change-Id: I0f17c4d164585a48fb9f0d40b90a7d3b975c7ab8

Merge "Final page of search results sometimes having erroneous "next" link"

Fix misnamed SVGHandler::canAnimateThumb()

Caused the animation warning to not be present on
image description pages

Also add unit tests. Since I'm adding an animated svg file for
this unit test, also add a metadata extraction test for animated
svgs, since we're missing one.

Change-Id: Id03e1a0e1c151d3c575a695a42c54b709187d10a
follow-up: 876128f8c9f0ce6a8424336f

Merge "LoginForm: Pass username as first parameter to error message if aborted"

Merge "Fetch string to pass through writeTemporary() in DatabaseOracle.php"

Merge "Removed undeclared var from wfDebug()"

Merge "Revert "Fix "CSSMin url() value remapping not working in certain obscure cases"""

Revert "Fix "CSSMin url() value remapping not working in certain obscure cases""

This reverts commit fa1053a16a5b3ff2004f6c06c69e87f8570a35d7.

Breaks CSS output, making the skin and extensions look unstyled.

Change-Id: I297150f775dd5559f91f2c9c25c64b134459c6b6

LoginForm: Pass username as first parameter to error message if aborted

If a hook is aborted, pass the username as the first parameter to allow
for more customized messages.

Change-Id: Ifb55a2e180c619f7d06f728dd8eea2c9863b8f82

Remove <div id="mw-js-message">

This is a remnant of the mw.util.jsMessage() utility, replaced by
mw.notify() several versions ago.

Change-Id: Ifd2765581f06dcc2ca057bc30b27b336df2d066b

Merge "Fix "CSSMin url() value remapping not working in certain obscure cases""

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: I8f904f32657c9b380fffe31de30b18b3a164583d

Use anonymous function in Linker::formatAutocomments/formatLinksInComment

A anonym function has no own scope, so it is not possible to call the
private callback, so moved the code of the callback into the function.

This avoids passing parameters with static variables to the callback.

Change-Id: I41ad85545301f6699817135b7e8ceba0f8b26851

Merge "LinkerTest: A bunch of tests for Linker::formatComment and friends"

LinkerTest: A bunch of tests for Linker::formatComment and friends

Change-Id: Ie805d2945ddc1596545ef3f22d32b429caca8a27

Merge "Change 'delete' to 'rollback' in action=rollback params description"

Change 'delete' to 'rollback' in action=rollback params description

Bug: 66406
Change-Id: Iab874f45b5baaa89a6345e351cc45ed16f5af6e8

Remove isset() check in Sanitizer::setupAttributeWhitelist()

Declared variables have a default value of null, so there is
no point to use isset() to check only if it has the value null.

Also changed the location of the line break to group variable
declarations.

Change-Id: Ic36fc95db86909d8b5075954a72afa479fa20d0d

Remove $dbr from ChangeTags::addTags()

$dbr (DB_SLAVE) is completely unused now.

Change-Id: I02bbde8366480b841d8028403e9a0efe88fa46fd

Merge "Make Minerva skin opt out of parsoid style module"

Make Minerva skin opt out of parsoid style module

These styles are mostly Vectorish, so shouldn't be applied to the
Minerva skin that is provided in MobileFrontend.

We set our own parsoid styles in
VisualEditorOverlay.less.

Change-Id: I11a75c5552fa14ad119ad60bb5fa3f6185582429
Dependency: I5de12c97108dcb6e41ac8c34b17ff993d2e65249

Update OOjs UI to v0.1.0-pre (99068355d6)

New changes:
9906835 Make menu toolgroup take up less space and look more like a menu

Change-Id: Ie5e5f3d8f3f6bccdbacd85f1ea8423bfeaa8e2f2

Merge "Bug 51245: Add auto-numbered link CSS to Parsoid styles"

Merge "Revert "Officially deprecate skin autodiscovery""

Revert "Officially deprecate skin autodiscovery"

The warnings interfere with Jenkins and stop code from being merged
where Jenkins is voting. See bug 66429 - we'll need to fix this before
merging this. Sorry!

This reverts commit 7fe6e4422467fa0c701f3ffb395e66de9cd0d67f.

Bug: 66429
Change-Id: I649462b0ccc22954ba4fbad8087dd55a7ad910bd

Bug 51245: Add auto-numbered link CSS to Parsoid styles

Also use the mw-body-content class to scope styles to the content area.

Change-Id: Id765da7bfb60726eead86e67693b86c82bdb5610

Removed undeclared var from wfDebug()

Change-Id: Ie8a0ff2f9347c6ed2a04f7c99de6c43958dfd1f7

Merge "Vector: Update comments in vector.js"

Officially deprecate skin autodiscovery

Corresponding change in REL1_23 (different, because core skins still use
autodiscovery in that version): If9125ef587ce11f0d69f12eacd00ae7236139798.

Bug: 65748
Change-Id: I6fc30f97d4b35820c132c4892abf6d2c86f9482e

Don't use autodiscovery for core skins, move them to separate directories

Step one on the way to killing the autodiscovery mechanism and making
the core skins less intertwined with core.

This only moves the files and fixes hardcoded paths throughout core.
Any further changes will be done in separate patch(es).

Moved files:
* skins/MonoBook.php → skins/monobook/MonoBook.php
* skins/Vector.php   → skins/vector/Vector.php

Bug: 65748
Change-Id: Ib4bdda5ed3c133fce0113eb17fa39950aa812f87

Merge "Remove unused variable"

Remove unused variable

Follows up Ie47649ef45f9eebdfe

Change-Id: I2c6f2a0b4a8f15056ad0b314da7f8086168b1863

Do not show page navigation on included Special:Contributions page

Bug: 65780
Change-Id: Ie1f96d62e0f7b8c1066aaecc5e8ba528fb94cbf7

Merge "Update OOjs UI to v0.1.0-pre (15f4156bac)"

Merge "More optimizations to User::saveOptions()"

Update OOjs UI to v0.1.0-pre (15f4156bac)

New changes:
15f4156 Follow-up 2334482: actually clear items in clearItems()

Change-Id: I7d4570364690b82a6fb7de3beb7e7e68ff451366

Add mw.track calls for jquery.migrate

Can't use mw.log.deprecate because many of these are fired
for specific behaviour within a method, not for access to the method
itself.

And mustn't fire the events with topic "mw.deprecate" as we can't satisfy
all the requires properties of that event schema (such as having a search
phrase to find offending code). These tracking hits would be for hit
counts only, not for finding where it comes from (we'll still have wikiid,
pagename, revision id and referral information from EventLogging though).

Change-Id: I6c868f2ce831d74889ed58b6ab4a69b233b3923f

Fix "CSSMin url() value remapping not working in certain obscure cases"

This patch extends the regex used to find URLs by css block quote subpatterns.

Bug: 60077
Change-Id: Ife00d01b78c6369f20ffc49d20aa58e354a61ee7

Update OOjs UI to v0.1.0-pre (098f84f8a0)

New changes:
371fcff Correct TextInputMenuWidget position for RTL
05467d3 Localisation updates from https://translatewiki.net.
2334482 GroupElement: Improve performance by avoiding .add() overhead

Change-Id: I6701dd1bc08343e14f6b3cacdfe0c8d80b350394

Merge "ResourceLoaderLESSFunctions: Quote CSS data: URIs if needed"

Add PPFrame::getTTL() and setTTL()

Add functions to frames to control the TTL of their output, and expose
this via expandtemplates in the API.

Bug: 49803
Change-Id: I412febf3469503bf4839fb1ef4dca098a8c79457

Localisation updates from https://translatewiki.net.

Change-Id: I7323b1280bf812e5043d528dea4f32d4b4cde26f

Vector: Update comments in vector.js

Change-Id: Ife1364c6f3c7d2dfde2275287e2f85d10c19c526

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit d6666187bac33cbdb0ad5f67d20be98680845b24

Change-Id: Ib4fd31e9d1c14e6a171d39510b5621baf1642b9e

Merge "Remove unused jquery.cycle.all.js file"

Vector: Hide the superfluous "Variants" menu label again

Follow-up to I09c8031c.

Also removed a piece of temporary hacky CSS from Iabda8155.
The other hacks from it were removed, this one persisted.

Bug: 66364
Change-Id: I4d5ae0858e1685a60c42d8080fec2753f25ea9ad

Merge "Add HKDF as a fast, random number source"

Merge "API: Add show=unread to ApiQueryWatchlist"

Add HKDF as a fast, random number source

This adds an RFC 5869 compatible library for quickly generating
cryptographically secure random keys/numbers. An attacker should be
unable to predict the next number generated, as long as the secret
key (set to wgSecretKey if not explicitly defined for a wiki) remains
secret.

In my testing, this is generating random numbers about 7x faster than
MWCryptRand for 32 hex characters, and shouldn't drain openssl's
entropy source.

This will hopefully speed up maintenance tasks that require secure
random numbers, such as resetting login token, and wrapping user's
passwords in a new format.

Change-Id: I437d9aec0122a0533b0d8723fe8a8624898af909