Remove these two rights autoreview and torunblocked from mediawiki

Bug: T134027
Change-Id: I47f064f7e47bb25d9956f7836b3718fd6ede25e6

Merge "WatchedItemStore: Use callable type hint instead of Assert library"

Merge "LoginSignupSpecialPage: Load return and returnto params as early as possible"

Merge "RollbackAction: Don't return true, causes '1' to be output"

Localisation updates from https://translatewiki.net.

Change-Id: I8038db5de26e5efb0b801902e8ce51536af5e47b

Merge "mw.ForeignStructuredUpload.BookletLayout: Make licensing links clickable again"

Merge "Clean up updatedmarker styling mess"

SpecialSearch: Use OOUI\ActionFieldLayout for search input and button

This prevents the button from flipping to a new line if the window or
the available width at all is smaller as the width of the input and the
button together. Now, the input will be as small as needed for the button
to be on the same line.

Bug: T134475
Change-Id: Icc68a8e83fc7037b96a5efec2766ffdf453c5413

LoginSignupSpecialPage: Load return and returnto params as early as possible

They're are needed for a redirect to the target page after
a successful login, which is made before the SpecialPage::execute()
function is called. Loading basic request varaibles in the execute()
function is therefore too late to take effect for the redirect after
a successul authentication with a primary provider, which needs to
redirect the user to another site.

Bug: T135924
Change-Id: I6ded7f9bb255cbb332a5810e7ed3cb3ecfdb2c04

Merge "OOUI HTMLForm: Allow specifying whether buttons are framed"

Merge "SpecialSearch: Remove obsolete styles"

Merge "Separate Special:Search styling info into separate module"

Clean up updatedmarker styling mess

This is the follow-up to d629541076c29ce4a7ec50d69a5e5969b409f16d
that b09bfb7bf6b811f9657a606d611a7f889a5e013e meant to be but failed.
It updates the right file this time, and removes the old override for
old skins.

Bug: T134515
Change-Id: I074688562e4da8c7c32d9f2bae67cd601e9b5f65

Update mediawiki/mediawiki-codesniffer to 0.7.2

Change-Id: I9371faef4b8a0b9c80ec41d8ba08f4ff4936791e

Improve @covers tags for LinkerTest

These hooks are now called from LinkRenderer, so make sure it is
covering the right code.

Change-Id: Ifaa28d471f585dce9d968cc1173c7fdceb408239

WatchedItemStore: Use callable type hint instead of Assert library

And remove an unused use statement.

Change-Id: Idb74a564088744d73f71f5ef17e8f84f6e484c69

SpecialSearch: Remove obsolete styles

Remove old Advanced PowerSearch table styles, which seem to not have
been in use for quite some time now.

Change-Id: I4c3013b035e146a8f8ad9622662491934c0c5389

OOUI HTMLForm: Allow specifying whether buttons are framed

Bug: T136361
Change-Id: Ic31f857c749d62a32cafae68dc3f1cbd86e1e382

RollbackAction: Don't return true, causes '1' to be output

Bug: T136375
Change-Id: Id994b7ee7044ce18cf245a219ba290970511ea0f

Separate Special:Search styling info into separate module

Almost all styling of this page has little to do with the JS module.
Separating it into a styles module.
Also made the selector for the JS elements far simpler, as there was
no reason for being overly specific here.

Bug: T136363
Change-Id: I96a35f930be26b26f139120f3e5b23c33a5d019e

Localisation updates from https://translatewiki.net.

Change-Id: I47fe8c2cc2ff6457d934627dbe71842a34e4c48d

Merge "Fix @covers tags for LinkRenderer tests"

Merge "Avoid system styling issue of <textarea>s in Firefox+Linux"

Merge "Revert "RollbackAction: Implement AJAX interface and require POST""

Fix @covers tags for LinkRenderer tests

phpunit's coverage report needs class names that include the namespace,
otherwise it'll just fail.

Bug: T136420
Change-Id: Ie748237176ea1363b35d73084e63e6fafe808286

Avoid system styling issue of <textarea>s in Firefox+Linux

Bug: T136415
Change-Id: I5ebfeb65f4eb5607b161b4f95822e7dbfeea6d8b

Merge "Fix ConfirmLinkSecondaryAuthenticationProvider error message name"

Revert "RollbackAction: Implement AJAX interface and require POST"

This partially reverts commit 9af38c046c.

* The new JS modules continue to exist. While not used by default,
  the mediawiki.api.rollback may be used elsewhere by now.
  The mediawiki.page.rollback module may still be used on cached
  pages and should continue to work.

* The new API behaviour remains.

* To avoid breaking mid-air rollbacks again, the token remains
  the same (plain "rollback" salt). We're not re-introducing the
  web-specific salt again.

Bug: T136375
Change-Id: Iba17ce55ff9506e838bfc6e70ca280e5a20b77b6

Fix ConfirmLinkSecondaryAuthenticationProvider error message name

Change-Id: I49f149004b3ad5c3eefbcbfcd0cfed64d9cbf522

Enable AuthManager by default

Second attempt, after fixing T135884.
(First attempt was I6695aa3da42fb2b088eaa8d1883ccbb67f2c0c38.)

Depends on:
https://gerrit.wikimedia.org/r/#/q/topic:authmanager-selenium,n,z

Bug: T135498
Change-Id: I756a82267cb9c44ec35146fbb4599b183747c741

Merge "Special:UserLogin: Consolidate create account buttons"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit b0d48342e0c540c17b2c073833d4a7ab147e2852

Change-Id: I9eb00fdc854eac7ddfc37520bd6b3cf8913db523

Merge "Use HTMLForm style sheets on mobile"

Merge "Make the call match the function definition."

Merge "Add LinksUpdate::getRevision()"

Merge "Parser: Make makeKnownLinkHolder() protected, and remove $query handling"

Add LinksUpdate::getRevision()

Similar to getTriggeringUser(). Also propagate it
to subjobs similarly.

Bug: T135959
Change-Id: I3d894acaf3d85b790e5034c7d9f76bf94672f445

Parser: Make makeKnownLinkHolder() protected, and remove $query handling

Extensions shouldn't be calling this, just the Parser, so make it
protected. And since the only caller passes an empty array for $query,
we can just remove it entirely.

Change-Id: I3adbcaabbb40870eb3df1495c3c2743ff21f0c64

LinkBatch: Use TitleValue instead of Title

Replace the remaining instances of Title with TitleValue, and use
services from MediaWikiServices instead of calling deprecated singleton
functions.

Change-Id: I5d13939a76380fff6b787cea8d4a5f90c1a31a5d

Have Linker::getLinkColour() accept LinkTarget objects

Change-Id: I7d7318099a822f9ddd88c538274511e6d8a8f1bc

LinkCache: Don't try looking up NS_SPECIAL

Change-Id: I1afe8c780c3b12ae806ac19c4efb37c7033951ca

Add tests for Linker::getLinkColour()

Change-Id: Ic1553e21def47f5c4923ba747146b36b0b3ffdfc

ChangesList: Use LinkRenderer instead of Linker::link()

Change-Id: Iae32a9e365aad268d2671df6a0b916e4d9c0a801

Merge "Parser: Replace Linker::link() with LinkRenderer"

Merge "Send edit stash metrics for cache attempts"

Merge "LinkRenderer: Re-implement noclasses as makePreloadedLink function"

Parser: Replace Linker::link() with LinkRenderer

Replaces usage of Linker::link() in Parser and LinkHolderArray with the
new LinkRenderer.

Change-Id: Icb796ef08d70926728732ab5468940c09ba5eaf8

Send edit stash metrics for cache attempts

Change-Id: I22d5626cca55c2cc35e1b33fced5902e8d364d40

Merge "TitleParser: In formatTitle(), don't throw exceptions on bad namespaces"

Whenever possible, reuse User objects in unit tests

The unit tests spend nearly half of their run time resetting the user table for
each test. But the majority of tests do not depend on the user table having the
exact value that the setup code resets it to, and do not need to modify the
user objects they require to run.

Fix that by providing an API for tests to get User objects, and to indicate
whether the User object will be subject to destructive modification or not.
This allows User objects to be reused across multiple unit tests.

Change-Id: I17ef1f519759c5e7796c259282afe730ef722e96

TitleParser: In formatTitle(), don't throw exceptions on bad namespaces

This ocassionally happens for whatever reason, but it doesn't really
make sense to throw an exception when creating a broken-looking link
would also work. We already do this for TitleParser::getPrefixedDBkey(),
and this also matches the behavior of Title::getNsText().

Bug: T136352
Bug: T136356
Change-Id: Ic7eb17f8917f7fbb28b11d94b742dac1fe5582a1

Merge "Revert "build: Bump grunt-karma and related tools to 1.0.x""

Merge "Bail out in ApiStashEdit for bots for sanity"

Localisation updates from https://translatewiki.net.

Change-Id: I04fc4e9de4d4d0f4c8a13d559d6eafa91f1d70ac

Bail out in ApiStashEdit for bots for sanity

checkCache() is disabled in this case anyway.

Change-Id: I1c18585eecc1a7c4c0e24546799c7ee448b3ea57

Revert "build: Bump grunt-karma and related tools to 1.0.x"

Karma is suspected to have an issue disconnecting Chromium.
That causes the qunit job to idle even after all tests have
been completed.

This reverts commit 72b0d9da08d6ea8c1842d76aaefe8e57046a2026.

Bug: T136188
Change-Id: Id818f5a0d1983d0bffa545f7731a780bf2b9bf25

Merge "Document thumbinner's block formatting context"

Merge "mediawiki.special: Remove unused mediawiki.special.js"

Document thumbinner's block formatting context

Document the side effect of setting overflow:hidden on thumbinner.
It's primary utility here is to clear the background of the block from
any floating content, avoiding potential overlap.

The property sets a new block formatting context.

Change-Id: I91b759cdd386cb648b1b704b0e7df708f6c4fa92

Exclude bots from edit stash stats

Change-Id: Id309879eaaf039de7ac5f531930fa8e758a959c7

Merge "registration: Add --config-prefix to convertExtensionToRegistration.php"

Merge "Revert "Thumbnails: allow overflow inside thumbnails""

Merge "mediawiki.special.apisandbox: Use module.exports instead of mw.special"

Merge "mediawiki.special.recentchanges: Use module.exports instead of mw.special"

registration: Add --config-prefix to convertExtensionToRegistration.php

For converting extensions that use custom prefixes.

Change-Id: I4fcf7ff2af04e9a0367de368503f2c70d8fed959

Merge "Follow-up 276c30e: Use the correct hex for the green we want"

mediawiki.special: Remove unused mediawiki.special.js

'mediawiki.special' violates T92459 due to having both script and styles and
being loaded as a styles-only module. This avoids the warning from I8b6c6a10d.

> Unexpected general module 'mediawiki.special' in styles queue.

mw.special was first introduced in r90943 (d4abcb526806). I regretted recommending it.

Change-Id: I7bb000c9946d194fd1ae7f9a18fbc031ba225f25

mediawiki.special.apisandbox: Use module.exports instead of mw.special

mw.special.ApiSandbox is not used anywhere, and it's an easy candidate for
converting to the new module export pattern.

This removes the last use of the 'mediawiki.special.js' file, which violates
T92459 due to having styles.

Change-Id: I9b6972d763f34d50479099d4d726d1c77a91c223

mediawiki.special.recentchanges: Use module.exports instead of mw.special

This isn't generally used as a public method. The only reason it's exposed is
for an integration test to access it.

This commit and others prepare for removal of the 'mediawiki.special.js' file.
This script does nothing other than create "mw.special = {};". While that init
pattern is common in extensions and not wrong, 'mediawiki.special' violates
T92459 due to having styles. Given there's only script uses of it, it's easiest
to remove it in favour of the new module export pattern.

Change-Id: I2e78828828601e1160550efe02c07172ac32e985

Merge "collation: Remove suppressWarnings() for getSortKey()"

Merge "Avoid DBPerformance warnings on PURGE/TRACE requests"

collation: Remove suppressWarnings() for getSortKey()

Small optimization to IcuCollation::fetchFirstLetterData().

This used to suppress / restore warnings once per every letter of
every alphabet. The workaround for string casting and error
suppression is no longer needed as of PHP 5.3, in which the
bug was fixed.

Change-Id: Idd41a509858c0887df4f632b480b387bd74027b2

Follow-up 276c30e: Use the correct hex for the green we want

Bug: T136175
Change-Id: I4ce6010d55c361c56584a36ac252a68f0d2ee363

Special:UserLogin: Consolidate create account buttons

When logged in and not logged in. Both now shares the same styles
and DOM structure. The differences:

* When logged in, you don't get a "Don't have an account?" message
* When logged in, the button's label is "Create another account"
  instead of "Join {wikiname}"

Bug: T135925
Change-Id: I90c7a1288af927a03da3ccf9499f625935742125

Use HTMLForm style sheets on mobile

Bug: T135779
Change-Id: I7152da816b979a2d41b57cd4c117ec8087c3ff7a

Change button text for *Credentials special pages

Bug: T135779
Change-Id: If6c2edab6a574bc961e47c7ea04ec0ef18e36491

LinkRenderer: Re-implement noclasses as makePreloadedLink function

'noclasses' makes more sense as a per-link option rather than an
instance member of the LinkRenderer instance, since it depends entirely
on whether the calling code has preloaded the link classes.

Introduce LinkRenderer::makePreloadedLink() which makes this clear and
requires passing in the classes as a separate parameter. As a side-
effect, due to the way LinkRenderer::mergeAttribs() is implemented, the
'class' attribute will always appear before the 'title' attribute in the
final output.

Change-Id: I0545aa9d7139794bc22f9d3d6d6eccde003b2982

resourceloader: Merge 'user.groups' into 'user' module

This is with T92459 in mind to simplify the process of splitting
the 'user' module for the styles-only queue.

Consequences:

* Cached HTML isn't relevant in practice since there is no caching for logged-in
  users and this module is only for logged-in users. Even then, cached HTML will
  work and may happen as browsers re-use HTML responses when revisiting a
  privately cached page (after 304 Not Modified).
  Note that OutputPage (via isKnownEmpty) only actually tries to load 'user.groups'
  if the wiki has 'MediaWiki:Group-*.{js,css}' pages for the current user's groups.

  - Old style queue request will continue to ask for user.groups which is now a
    FileModule with no styles (simply concats the empty string to the bundle)
  - Old load() request will resolve with an empty function.

* The are no known dependants of 'user.groups'. If there are, they will work
  by proxy of it now being an empty module that just ensures 'user' is loaded.

* The security origin of 'user.groups' was USER_SITEWIDE. The origin of 'user'
  is lower (USER_INDIVIDUAL). Pages that are restricted to USER_SITEWIDE
  previously received user.groups, but won't anymore. This should be fine as
  OutputPage::reduceAllowedModules() is mainly used to either allow everything
  or restrict all the way down to CORE. The only exception is disallowUserJs()
  if $wgAllowSiteCSSOnRestrictedPages is enabled (T73621) but that edge case was
  made for Common.css, not Group-*.css.

Change-Id: I74cd2368ebd2989c5e1c22bea491a80beb0319dc

Avoid DBPerformance warnings on PURGE/TRACE requests

The former sometimes show up in the logs as they were causing
CentralAuth to use the master but the expectations treated
the request as a GET request. This makes things more
consistent.

Bug: T92357
Change-Id: I55bf3139c68f5926fe67a51cf0eb1b2ffe55d17b

Merge "Use correct module name for stats in executeActionWithErrorHandling()"

Merge "Restore Userlogin error/warning URL parameter"

Revert "Thumbnails: allow overflow inside thumbnails"

This reverts commit 2964c2799f0c19ac4c765f025a2998a0f3baeff0.

Bug: T133439
Change-Id: Id16420e8f5fd73e31cba2d5415ae9c1730a5a32c

Merge "Param &$html for the "LinkBegin" hook needs to be null if $text is null"

Restore Userlogin error/warning URL parameter

I accidentally removed this when rewriting for AuthManager.

Also change an ID into a CSS class for now since HTMLInfoField
does not support IDs.

Bug: T135775
Change-Id: I5663a02c9809dd573087a4a528702c949b938970

Merge "Do not use $wgUser in Linker/LinkRenderer when sessions are disabled"

Localisation updates from https://translatewiki.net.

Change-Id: I7108f988a2c97e99bb3f889d0ecea55fceb0d95f

Do not use $wgUser in Linker/LinkRenderer when sessions are disabled

When MW_NO_SESSION is set, getting the user options for $wgUser will
result in an error. Check for that in the service definition, and
reuse that check in Linker.

Also removed an unused use clause.

Bug: T136124
Change-Id: Id6448d12e8f4a8a94852980e5eb6ccb202066822

Make the call match the function definition.

Change-Id: I381fbc9b6ddbf27f4f7a79dc19eec0547e0fddbb

Use correct module name for stats in executeActionWithErrorHandling()

Change-Id: I7d300525c32877d6fcc940962c93b932dc4c9181

Param &$html for the "LinkBegin" hook needs to be null if $text is null

To keep b/c with the old implementation.

Change-Id: I5fff92f47babe890fe0d3726dba6f60cc75ee708

Merge "filerepo: Add missing quote back to LocalFileLockError message"

filerepo: Add missing quote back to LocalFileLockError message

Follows-up e724d14495.

Change-Id: I48e304a307d3c0427f797a382dc23347bb7e914f

Fix regression: text color in `.mw-search-result-data`

Follow-up to I2ba04cc3ad9898c17fee3c65bb3bead834c3a1fd where `green` in
`.mw-search-result-data` was transcripted falsely into
`#0f0` instead of `#008000`.

Bug: T136175
Change-Id: I5d55718510650634a718d688a0e0698355097c6c

Update es5-shim to v4.5.8

Project site
* https://github.com/es-shims/es5-shim

File source
* https://github.com/es-shims/es5-shim/tree/v4.5.8

Changelog
* https://github.com/es-shims/es5-shim/blob/v4.5.8/CHANGES

Change-Id: I4a6a703d7db00d7eaaeb1365c1c19f3ae055c440

Merge "Remove explicit tabindex from diff links"

Add LBFactory::closeAll() method

Needed to allow long-running background tasks to close out database
connections in a way that allows them to reopen later when needed
again.

Bug: T97641
Change-Id: I4974b691392d4225b15e7b771a28a2cefeb9d8e4

Merge "Tidy: Add mw-empty-elt class"

Merge "Update OOjs UI to v0.17.3"

Tidy: Add mw-empty-elt class

In hindsight, it was pretty silly to put the element name in the class.
This is to support an upcoming Html5Depurate change.

Change-Id: I38ddcdb7daf8f0e2bde9e019457fd4192d86e02b

Update OOjs UI to v0.17.3

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.17.3

Change-Id: I3fe38754c18a83554593479ea1b01f8a60144435