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

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

Merge "Lowered the TransactionProfiler threshold"

API: Add show=unread to ApiQueryWatchlist

This allows for retrieving only those pages that have changes since the
last visit.

Bug: 65246
Change-Id: I81d2fae39178708efc0f57e5acea93b50463bc7a

Merge "Update list item newline handling to follow Parsoid's model"

Update list item newline handling to follow Parsoid's model

This improves on commit 34bd573144883af9ace7ead32d3be8f692431718 by matching
Parsoid's newline handling in the PHP parser. It is the outcome of a
discussion with Erwin, where we agreed that

* foo
* bar

should produce

<ul><li>foo</li>
<li>bar</li></ul>

See the discussion in https://gerrit.wikimedia.org/r/#/c/94443/

The original rendering issue this tried to address is no longer present after
a change to the template. The pure CSS solution is now working.

Bug: 39617
Bug: 56809
Change-Id: Ib7aa9449bbd994cb23b83b3f23cff944b1cddadf

Merge "Vector: Restore old focus animation on the "More" dropdown"

Vector: Restore old focus animation on the "More" dropdown

Per post-merge review of I09c8031c, it's not consistent with anything
we have elsewhere.

Bug: 66363
Change-Id: I89cf29fcb92d29d782c4091f39879f3837579bdb

Merge "API: Add prop=unreadcount to ApiQueryUserInfo"

Merge "Installer: Re-use ResourceLoader methods instead of duplicating"

Merge "Remove backwards-compatibility modules from I3947eaac"

API: Add prop=unreadcount to ApiQueryUserInfo

People want to be able to fetch the count of unread pages on the
watchlist.

Bug: 65246
Change-Id: I890f21631ba074a7b3d660534c40e38773d8ea77

Merge "Upgrade jQuery UI from 1.8.24 to 1.9.2"

ResourceLoaderLESSFunctions: Quote CSS data: URIs if needed

Change-Id: I121cf6b410b17f2b8b880cb3b5cb786f5eeb23b6

Localisation updates from https://translatewiki.net.

Change-Id: I318f55bbd9ba3f99fd2061b61f663f79ec2ca7d6

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

Pulled some code from insertOneRow() to fix the update() function
and prevent the error warnings in bugzilla 64970 from showing up.
Note that since insertOneRow() and update() contains a lot of
duplicate code, they may be candidates for merging, but that is
outside the scope of this request.

Bug: 64970
Change-Id: I9ff9077d68e77f05f878fc541f87e0a5ce86ecc2

Localisation updates from https://translatewiki.net.

Change-Id: If92d90ab5d744d3b82beedbdac92ab3b5bf03a79

Remove unused jquery.cycle.all.js file

Originally added in r109828 / 2f1f0abc6b, but without any module
declaration. The only way this file could be used is if an
extension is referencing the file directly, which is unsupported.

Bug: 66292
Change-Id: I730c31cef40ff1cc1c12573ace9bec78ebbc1e7d

Installer: Re-use ResourceLoader methods instead of duplicating

Also:
* Fix incorrect documentation comment of ResourceLoaderModule::getStyles(),
  it doesn't return a string, it never has.
* Remove spurious space in count() call.
* Fix spelling of in "proper".

Change-Id: I000393636f7b9015ad1bacfbb3eb8a6ad8695d72

Minor clarification & tweak for Parsoid render testing

* Clarify comment

* Set some padding on body

Change-Id: I93c070e997cbf34bfa4a296f99c30d90ee67e59e

Bug 66287: Proper caption styling

* Make sure borders render properly even in Chrome & with zoom on high-dpi
  devices. overflow: auto is important for this.

* Set borders & background on imagecaption

* Enable border-collapsing; we are rendering this as a table & want the
  caption & figure to show a single border

* Add appropriate margin on the image to create some padding inside the figure
  despite border-collapsing

Tested in Chrome & Firefox.

Change-Id: Ibb46b682138ac703bea364efaeefc376e60c2d5f

Merge "Make sure images that don't have an explicit alignment get aligned right"

Make sure images that don't have an explicit alignment get aligned right

If the alignment of an image is not explicitly set, Parsoid does not
output an alignment class for the resultant figure element. This can lead
to the image not being aligned/floated.

To fix this we modify content.parsoid.less to apply the default
right-alignment styling to all images and then override this in cases
where the alignment is explicitly set.

Bug: 66282

Change-Id: I4f986f23b0c8ba6f6b66efc07eb38806744123e2

Localisation updates from https://translatewiki.net.

Change-Id: I2b1a64cae75faf8d201ba58305bb3e85fb2f4c4f

More optimizations to User::saveOptions()

* Avoid doing the DELETE for unchanged rows. This method is often
  called when one new preference (e.g. watchlist token) is set.
  These cases will now simple result in an INSERT, even if the
  user already has some preferences set.

bug: 36116
Change-Id: Ib7c558e7c06927875086b86ea719ac1323a7b4a8

Merge "Update outdated signup page-related message documentation"