Update wikimedia/xmp-reader from 0.6.1 to 0.6.2

Change-Id: Ie396666aae2122dc552f6f1a7f6fe6ff96387fd1
Depends-On: Ia5dcf97548ed1c1f17cce9701df5cf13eea814dd

Update wikimedia/php-session-serializer from v1.0.6 to v1.0.7

Change-Id: Id30bb862bec42e508b3551d39f5980d900ade206
Depends-On: Ia19d237198fd2a0a15db467ec9dd97e98a857e33

Merge "Revert "RCFilters: Export config vars in the RL modules where possible""

Revert "RCFilters: Export config vars in the RL modules where possible"

This reverts commit bf4f9d0313495dd67a53b9925edf9a3cc6ef36fe.

Reason for revert: causes issues with OAuth extension

The OAuth extension uses a hook to override certain tag messages, and
that hook uses RequestContext::getMain()->msg(), which tries to load
the current user. That's not allowed in load.php requests, so we get a
"Sessions are disabled for this entry point" error.

Change-Id: If58b7edfca116cd7a11f38556f74bed82609d96a

Update wikimedia/ip-set 2.0.0 to 2.0.1

Change-Id: I154a677bac28a451be4f203ff05ea4620260b54e
Depends-On: I1090817a5ff1e406cf51596a8b1558c7bd3ca7d7

Merge "ChangesList: Remove spaces from <td> in EnhancedChangesListGroup.mustache"

Merge "ProtectionForm: Move JS config var to RL packageFiles"

ChangesList: Remove spaces from <td> in EnhancedChangesListGroup.mustache

The special page RecentChanges can included in wiki content via

  {{Special:RecentChanges}}

In this case the generated content is parsed by a wikitext parser. The
wikitext parser converts

  <td>
    <span>Foo</span>
  </td>

to the HTML

  <td><p><span>Foo</span></p></td>

But the wikitext

  <td><span>Foo</span></td>

is converted to the HTML

  <td><span>Foo</span></td>

The module 'jquery.makeCollapsible' supports custom collapsible toggle
buttons but only when they are a direct child of the <td> element. When
the module does not find a custom collapsible toggle button it adds the
default button [Expand].

Because of the additional <p> element when including the recent changes
the module 'jquery.makeCollapsible' adds a default collapsible toggle
button.

This change removes the spaces between <td> and <span> to prevent the
<p> elements when including the recent changes.

Bug: T215737
Change-Id: Ida46956fc7f9ab923c542751ac13411380c02ab0

Merge "Job::factory should throw an InvalidArgumentException, not MWException"

Job::factory should throw an InvalidArgumentException, not MWException

Change-Id: I9a85562b205db604d2746ec5db0e72dacdb93c6d

Merge "mw.widgets: Use CSS grid for the calendar in supporting browsers"

mw.widgets: Use CSS grid for the calendar in supporting browsers

Bug: T214997
Change-Id: I05dca01585f3de86580e7ae94ceeae1716f56ba3

Merge "parser: Add new pcache metrics, split by page content model"

Merge "registration: Define packageFiles for v1 extension schema"

Localisation updates from https://translatewiki.net.

Change-Id: Id809e80a46547f7bf042eb6073efd7a519666ce7

parser: Add new pcache metrics, split by page content model

Change-Id: I31c3c5b863309ffcc4424c43891b577b3fb7a753

registration: Define packageFiles for v1 extension schema

Follows 56fa42d4332c.

Bug: T215756
Change-Id: Icbf433b6b147f53cf99533a7272fc83392a8999d

Merge "Add ability to define packageFiles in extension.json"

Merge "Change `.forEach()` to `for ()` loop with break"

Merge "mw.widgets: Make TitleSearchWidget read-only-able"

Change `.forEach()` to `for ()` loop with break

Follow-up to change 1edba8029, which replaced:
  $.each( array, function ( index, value ) { ... } )
by:
  array.forEach( function ( value, index ) { ... } )

`jQuery.each` makes a break of the loop on a `return false`.
`Array.prototype.forEach` ignores the return value and there
is no way to stop or break a `forEach()` loop other than by
throwing an exception. [1]

A `$.each` with a `return false` could be replaced by
`Array.prototype.find`, but `find()` is not part of ES5. [2]

  return this.getItems().find( function ( filterItem ) {
    return filterItem.isSelected() && filterItem.isConflicted();
  } );

So a simple `for ()` loop with a `break` or a `return` is
used in these cases.

Also explicitly specify `undefined` as possible return value
in FiltersViewModel.prototype.getFirstConflictedItem.

[1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/forEach
[2] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/find

Change-Id: I0af89d39ed789f49e06b25cac14ecd995ba58cbd

Merge "Exception is never thrown in the getConnection() method"

mw.widgets: Make TitleSearchWidget read-only-able

Change-Id: I3eed52438c1e416d4dcdedb564bb9df3745d0f77

Remove unused @throws anotation from PHPDoc comment

Change-Id: If60ad5c12edbf4ea96c5c337f0c05cf0a8d5e43d

ProtectionForm: Move JS config var to RL packageFiles

The check for type Array of the JavaScript config variable is obsolete.

Change-Id: I4a38cfc211360c831958ce8ca88584cdf56a727c

Add ability to define packageFiles in extension.json

Bug: T215756
Change-Id: I97c3ccc13c54b424952ce15509b98a217b2670b1

Merge "Add jakub-onderka/php-console-highlighter 0.3.2 explicitly"

Add jakub-onderka/php-console-highlighter 0.3.2 explicitly

We pin it in various libraries, might aswell in core for completenes

Change-Id: I659c880c9e16e4eab7050874f75c9eb5c9fa958c

Update cssjanus from 1.2.0 to 1.2.1

Change-Id: I99f11a24ff0a8b02b1c0103ad8cb8faeb165c84e
Depends-On: Iee4c24b6b5dd92493160caafa4764b6d6e427147

Merge "Updated pear/net_smtp from 1.8.0 to 1.8.1"

Merge "ParserTestPrinter: use Shell class"

Merge "JpegPixelFormatTest: use Shell class"

Merge "Linker: Remove some else statements, and unnecessary temporary variables"

Merge "Make attachLatest.php wait for DB replica lag"

Updated pear/net_smtp from 1.8.0 to 1.8.1

Depends-On: I6543ac3e6482da197ce2d644823861d1db4624d8
Change-Id: I91b3c2864536a53fcc829a1e10a3bc0d4638db2b

Linker: Remove some else statements, and unnecessary temporary variables

Change-Id: I9b9383459edf06280135d9709da889fc5a688c98

Update dev dependancies

* Updated psy/psysh from 0.9.6 to 0.9.9 (dev-only).
* Updated nikic/php-parser from 3.1.3 to 3.1.5 (dev-only).

Change-Id: I05e45327e93d3bc733db73c0efe573b3223cc902
Depends-On: I896ac6c5afa8221ceca726eb7895e05d87d7e6f3

Localisation updates from https://translatewiki.net.

Change-Id: I26dda2663489848ffc75fc54b766fb827eeff4de

Make attachLatest.php wait for DB replica lag

Change-Id: I4f402ba772d0b0dda8360eaacf11e318f659efc0

build: Use headless Chrome and Firefox for QUnit testing

Also:

* Remove "watch" task from Gruntfile.
  This wasn't exposed via npm-run and afaik rarely if ever used.
  It seems mostly obsoleted by modern editors which perform
  linting and style guidance as-you-type and on-save.

* Remove "chromium" target from karma config, the default
  target supports both Chromium and Chrome.

* Remove unused "default" alias in Gruntfile. Was only
  accessible if developers installed grunt-cli globally, which
  we don't require and shouldn't encourage. The new minify task
  is now exposed as "npm run build".

* Remove "test" alias in Gruntfile. Was only used by "npm run test",
  which now uses "grunt lint" directly, which makes it more clear
  that these are only linters, and not unit tests.

This references T211784 because ensuring no headed browsers are used
(which would need Xvfb to start), is a prerequisite for the new Node 10
images. We may still need to provide a slower Xvfb-wrapped variant for
some repos, but would be nice of mw-core wasn't one of them.

Bug: T211784
Change-Id: I270a16d2b4f799ac7c9f84e9e2ee23806bdbc57e

ParserTestPrinter: use Shell class

Change-Id: I9e2eb3b01a9907d72e3edaa6660a4057d513942e

JpegPixelFormatTest: use Shell class

Change-Id: I53099827dc2b2b26b7f3cbc9f0e12030868fdafd

Merge "build: Update karma to 3.1.4"

Merge "Hard deprecate Language::setCode"

build: Update karma to 3.1.4

This fixes the bug where HeadlessChrome was always shown as
version 0.0.0 in the console output, which makes it rather
difficult to debug which Chrome version is installed.

Upstream https://github.com/karma-runner/karma/issues/2762,
which updates the ua-parser library to a version that fixes
that bug.

Change-Id: I0b7fe7f8a90649592c4d31fb3055cdc9e8d34814

RCFilters: Export config vars in the RL modules where possible

Right now there are only a few settings that don't depend on the request
context, but we can at least export those. The tag data in
particular can get pretty big.

Downgrade the type hint for ChangeTags::tagDescription() and
tagLongDescriptionMessage() from IContextSource to MessageLocalizer, so
we can pass a ResourceLoaderContext to these functions. (Since
IContextSource extends MessageLocalizer, this won't break any callers.)

truncateTagDescription() can't be downgraded to MessageLocalizer because
it needs a Language object, so hack up a fake RequestContext when
calling it. It would be nice if this weren't necessary; perhaps we can
move getLanguage() from IContextSource up into MessageLocalizer?

Bug: T201574
Change-Id: I9b66e35de826a07aa9551ba285e64e4852293229

Merge "resourceloader: Reduce 'implement' overhead for modules without scripts"

Merge "Move migrateActors.php to includes"

Merge "RCFilters: Convert RL modules to packageFiles"

RCFilters: Convert RL modules to packageFiles

* Use per-file require() to get classes' dependencies
* Export all DM classes to the global mw.rcfilters.dm object because the
  unit tests need that
* Don't make any of the UI classes globally available except
  MainWrapperWidget (and we can probably drop that one too)
* Rename mw.rcfilters.{dm,ui}.XYZ.js to XYZ.js

Change-Id: I8e541abf15c6987827a29eecc6a6f19f27378abb

Merge "wdio-mediawiki: Rename internal "Front page" ref to "Main page""

Merge "resources: Update jquery-client from v2.0.1 to v2.0.2"

Merge "mediawiki.user: Remove deprecated stickyRandomId()"

resources: Update jquery-client from v2.0.1 to v2.0.2

Bug: T101483
Change-Id: Ib1537cd92d733998818fe28b04ee81bec48a75d3

Merge "wdio-mediawiki: Move non-public methods to local scope"

wdio-mediawiki: Rename internal "Front page" ref to "Main page"

Also remove redundant 'new Promise' indirection, in favour
of native chaining through "then()".

This avoids a situation where wdio could hang indefinitely (until
timeout) when MWBot.request fails because we did not attach any
"catch()" handler to the promise. By always keeping a chain, the
failures are (by default) forwarded and thus propagated and thus
caught eventually.

Change-Id: If2f09479600029db2fa967c57082031744a06d8f

Merge "Follow-up I5bcae848: Remove from 1.33 RL, backported to 1.32"

wdio-mediawiki: Move non-public methods to local scope

Change-Id: I56258290d7c731b913f4df21c0bf4adc7fe4c69f

Merge "mediawiki.language: Deprecate mw.language.specialCharacters"

Merge "mediawiki.libs.jpegmeta: Remove deprecated mw.libs property"

Follow-up I5bcae848: Remove from 1.33 RL, backported to 1.32

Change-Id: I84fa5e2fae785f80833a3128c18790d6ad3d1042

resourceloader: Reduce 'implement' overhead for modules without scripts

Change-Id: I55728c526711545be3923d3e1e2f276cbfa52224

mediawiki.language: Deprecate mw.language.specialCharacters

Use require( 'mediawiki.language.specialCharacters' ) instead.

Remove mw.language.setSpecialCharacters(), because it was only used by
the specialCharacters module. There already was no getter.

Depends-On: I9cf525a1e1aadc12d72284d08b98882b0ab95f5e
Depends-On: I74e706a6603bd8fa899193de91e49f246608c4c1
Change-Id: I02e0e84b79309caa314954925cc9c0c2874b9bde

Merge "Drop SessionManager::autoCreateUser(), deprecated in 1.27 and unused"

mediawiki.user: Remove deprecated stickyRandomId()

Introduced and renamed/deprecated in the same cycle (MW 1.32), was
meant to be removed before release but forgotten.

Change-Id: I4bb219a1e38e15651115b70277e1968886dca690

mediawiki.libs.jpegmeta: Remove deprecated mw.libs property

Deprecated since 1.31.

Also fix existing uses to not use the private mw.loader access
pattern, which is unsupported and not needed here.

Depends-On: If0a125320af15b92216d98f54348f8b375cc1988
Change-Id: I403379ebf710077b77a02854c22071014d1a6bc4

Merge "Drop b/c reading of wgSessionsInObjectCache and wgSessionHandler, deprecated in 1.27"

Merge "resourceloader: Misc optimisations in startup module"

Drop SessionManager::autoCreateUser(), deprecated in 1.27 and unused

Depends-On: I261ea755c1d273ccc7fcc7c3b041623ccf4e331b
Change-Id: Ice2d755bee694b97e5ddbfc72caddf2b87ba0fc9

Drop b/c reading of wgSessionsInObjectCache and wgSessionHandler, deprecated in 1.27

Depends-On: I0d861a24b6cfd5c881112f15c682d97cef715cf0
Change-Id: I6b69d9ad3863512879f287f960828c1fdf7b214c

Drop User::getGroup{Name|Member}(), deprecated in 1.29 and unused

Except in the Configure extension.

Change-Id: Id6e2511cc2c5f86afcf0af2faa69d4a712efbe26

Drop EmailNotification::updateWatchlistTimestamp(), deprecated in 1.27 and unused

Change-Id: Idad6f5aa579e41e941d1074465dd2f0e0f4577dc

Move migrateActors.php to includes

This way it can be subclassed in extensions, like AbuseFilter
(Ic755526d5f989c4a66b1d37527cda235f61cb437).

Bug: T188180
Change-Id: Idf320232011c72e39267b1f3c39848aea35d37fe

Drop SearchEngine::getNearMatchResultSet(), deprecated in 1.27 and unused

Change-Id: I87987af702b9b92c4d1b3578e69f1b2e68529b52

Drop MWExceptionHandler::getLogId(), deprecated in 1.27 and unused

Change-Id: Id03aaf4f2ea5c910e13274826e44033e12adcd2e

Drop JsonContent->getJsonData(), deprecated in 1.25 and unused

Change-Id: I2cac3a1d2cda799c302ca2364fb91916c3240c11

Drop LinkCache->addLink(), deprecated in 1.27 and unused

Change-Id: Id6395cc9a122375349b7a13c051049bca3f5ab08

Merge "Drop EventRelayerGroup::singleton(), deprecated in 1.27 and unused"

Merge "Drop Linker::formatTemplates, deprecated in 1.28 and unused"

Merge "Drop Linker::formatSize, deprecated in 1.28 and unused"

Drop EventRelayerGroup::singleton(), deprecated in 1.27 and unused

Change-Id: I9e63fc255a575086a6c0934e51e593b5ee0a31d8

Merge "Persist sessions pre-send instead of post-send"

Merge "resourceloader: Require logName parameter in mw.log.deprecate()"

resourceloader: Misc optimisations in startup module

* Remove redundant trackQueue var.
* Simplify logError() code, and remove some local vars.
* Simplify Map#set() code.
* Simplify performance.now check. There are no known browsers,
  supported and unsupported, in which this property is existant
  as anything other than a function.
* Optimised baseModules iteration in sortDependencies().
  Slightly shorter and slightly faster as for-loop.
* Omit trailing dot from error message in one case.
  (None of the other error messages had trailing dots.)
* Add missing space to error message for mainScript.
* Simplify splitModuleKey() code.
* Remove redundant quotes in mw.loader.store.set() for encodedScript.

Differences in response body size on stock MediaWiki.

> uncomp 42152 - 41984 = 168 bytes saved.
> gzip-1 15437 - 15364 =  73 bytes saved.
> gzip-9 13506 - 13419 =  87 bytes saved.

Change-Id: I54c9ab30522c36ece054b915809fe20bff6867b3

resourceloader: Require logName parameter in mw.log.deprecate()

When omitted it, it previously defaulted to the key string, which
would automatically lead to the creation of confusing and
non-descript Graphite metrics. (Via the WikimediaEvents extension
via the 'mw.deprecate' tracking topic.)

If Graphite tracking is intended, require the logName parameter
to be set. For example, in wikibits, a deprecated 'write' method
is set on the 'document' object. Its Graphite property under
mw/js/deprecate/ will be document_write because logName is set.
Without logName, it would previously create 'mw/js/deprecate/write'
as metric, which easily clashes and is not very understandable
in Grafana.

An exception is kept for properties of the window object as their
fully-qualified property name would always be identical due to
them being available as global variables.

Change-Id: I9d99fe0395ec5309eac0895f2419f7cd16caeb94

Drop Linker::formatTemplates, deprecated in 1.28 and unused

Change-Id: I02e677021cdc1fc7ac6da4839a23b15afcfb2296

Drop Linker::formatSize, deprecated in 1.28 and unused

Change-Id: Id119e9f3ff0d27b59654876bad9212cc0f3c5a2d

Drop SiteSQLStore, deprecated in 1.27, unused

Change-Id: I6743417c3aafbba59ac02350e0190fbf688a5a40

Merge "Remove old Title CSS/JS methods, deprecated in 1.31, unused"

Merge "Remove Title->canTalk, deprecated in 1.30, unused"

Merge "Remove Title::escapeFragmentForURL, deprecated in 1.30, unused"

Use packageFiles feature to replace special-purpose RL modules

For config vars, add a virtual package file called 'config.js', and use
require( './config.js' ) in the module. For most data modules, add a
virtual package file called 'data.js', use require( './data.js' ) in the
module. Where needed, add wrapper files that put the data in the
relevant global object and in module.exports.

LanguageDataModule is the only special-purpose module not being removed
in this commit, because it uses languageScripts, and those are not
compatible with packageFiles (yet).

Also merge mediawiki.ForeignStructuredUpload.config into
mediawiki.ForeignStructuredUpload, since that was the only thing that
used it.

Change-Id: I203d4e3ecdeeeb16729eba2dcf40d11a41d2e582

Remove old Title CSS/JS methods, deprecated in 1.31, unused

Change-Id: I500b5efbdc2ee3b4ca09662ed0df89e9d7514bbd

Merge "Remove Title->getSquidURLs, deprecated in 1.27, unused"

Merge "Fix mw-ui-quiet+progressive/destructive selectors"

MWHttpRequest: Include the request ID in outgoing HTTP requests

In order to be able to trace requests across the production environment,
we attach the X-Request-Id to outgoing requests.

Bug: T201409
Change-Id: Iadd6cbd51a78a1876a1a318783b51635717d054f

NoLocalSettings: Remove style for the html element

There is no need to style the html element. A style of the body element
is enough.

Change-Id: I4ab90824de393de3569ff55cf0138a7aede5770c

Fix mw-ui-quiet+progressive/destructive selectors

Bonus: Fix color of disabled button text

Bug: T215610
Change-Id: I6e38355b46d87920605604beb0141cf097efab08

Merge "Collapse some else statements and nested if statements"

Merge "Expose external link additions and deletions"