benchmarker: Implement setupEach for per-iteration setup

To bypass caches in legacy code where injection doesn't easily work,
support a per-iteration setup function, 'setupEach', which runs right
before the actual function is timed and called.

And use it in benchmarkTitleValue, where I mistakenly assumed that
'setup' did what I wanted.

Bug: T201992
Change-Id: I2d01d899bf63576df2833705667f1a16604ab4cc

Merge "Simplify redirect linking in InfoAction"

Localisation updates from https://translatewiki.net.

Change-Id: I0a824a0c317de91546f123d42158f818113f63b0

Merge "Improve StaticArrayWriterTest and add missing @covers tag"

Merge "Create and use PrefixingStatsdDataFactoryProxy in PerDbNameStatsdDataFactory"

Merge "CSP: In tests, use setMwGlobals() instead of manual set+undo"

Create and use PrefixingStatsdDataFactoryProxy in PerDbNameStatsdDataFactory

MediaWiki::emitBufferedStatsdData() is never called for the PerDbName factory,
so stats were being dropped. Instead of having two factories, turn the
PerDbName one into a proxy/wrapper around the main factory that just adds a
prefix in front of all of the keys.

Bug: T202144
Change-Id: I31e376446abb58e41353b4ca3814120d2e914104

Improve StaticArrayWriterTest and add missing @covers tag

Checking for proper quoting was suggested by Krinkle in CR.

Change-Id: I7fa599767017a5cfc82899bc4178de6cb34bdd02

CSP: In tests, use setMwGlobals() instead of manual set+undo

This is the idiom as used in other PHPUnit tests for MediaWiki.

Change-Id: I285e5abe57bad8b9a8a3b35d81bb3a9f5f3dbe85

Localisation updates from https://translatewiki.net.

Change-Id: Ia8ac63807e9d30d524349ab38e4c66f3f840ce57

Remove wfMakeStaticArrayFile()

Replaced by the StaticArrayWriter class.

Bug: T200626
Change-Id: I7c02cf3a7dda2804d2eb8d51716254bd78b7544c

Move wfMakeStaticArrayFile() into a class

And include tests :)

This code is independent of MediaWiki, but not really large enough to be
worth librarizing right now.

Bug: T200626
Change-Id: I022c074e8a708fb5219bc4ff4b53e7e31f60dc4b

Merge "Reduce calls to MediaWikiServices::getInstance()"

Merge "Use Title::makeName in Special:Export to simplify code"

Reduce calls to MediaWikiServices::getInstance()

In some functions MediaWikiServices::getInstance() was called twices or
in loops. Extract the variable to reduce calls.

Change-Id: I2705db11d7a9ea73efb9b5a5c40747ab0b3ea36f

Use Title::makeName in Special:Export to simplify code

Also adjust param docs

Change-Id: Ia938b38c86da25e73b91a55506d660bc9ca9384f

Merge "TitleFormatter: Remove indirection from getText()"

Merge "TitleFormatter: Don't look up namespace name for ns 0"

Merge "TitleValue: Avoid function call for internal property"

Merge "test: Assert that API generators have unique prefixes"

Merge "Turn ApiPrefixUniquenessTest into a structure test"

test: Assert that API generators have unique prefixes

Signed-off-by: Brad Jorsch <bjorsch@wikimedia.org>
Change-Id: I30758f2ac95fd8ae361ef8079abdfb0a82d92e34

TitleFormatter: Remove indirection from getText()

At the end of the day, the output is identical to LinkTarget::getText().

Change-Id: Ifddafb79d7602ba5e368cc40f99c56e22a56d506

TitleFormatter: Don't look up namespace name for ns 0

It's hardcoded to empty string, so we can skip some code.

Profiling (after switching the benchmark script to use NS_MAIN) showed
no noticable improvements in getPrefixedText().

Change-Id: Id6b044a51648d0a3f58331ac0427f9d5cd9d8f0f

TitleValue: Avoid function call for internal property

Change-Id: I16342469a8122bd46cada11fd1f1d3f88d2e8d1e

Merge "Inject SpecialPageFactory into Parser"

Merge "Pass through 'helpInline' to OOUI FieldLayout and make true by default"

Turn ApiPrefixUniquenessTest into a structure test

This way it gets run during extension test runs as well.

Temporarily add a hack allowing 'wbeu' duplicates until T196962 is fixed.

Change-Id: Ic89a22a2ff4525585de9e290a1d47d22cfaaac5e

Merge "mediawiki.util: Only call $.fn.updateTooltipAccessKeys if accesskey is set"

Localisation updates from https://translatewiki.net.

Change-Id: Ie7ab020ed7722038335b1a27c6faaaa7e07f8f3f

Merge "Mass conversion to SpecialPageFactory service"

Merge "Add index on rc_this_oldid"

Merge "Improve profileinfo.php documentation"

Pass through 'helpInline' to OOUI FieldLayout and make true by default

This change is meant to improve the layout of Special:Preferences in
particular, but it will affect any OOUI form using help messages. It
should be a harmless or beneficial change for most of them.

Previous behavior can be restored by passing `'help-inline' => false`
to the HTMLForm factory after 'help-message'/'help-messages'/'help'.

For example:

* Special:Preferences?ooui=1#mw-prefsection-watchlist
  * Before: https://phabricator.wikimedia.org/F25025213
  * After:  https://phabricator.wikimedia.org/F25025181

* Special:ChangeEmail
  * Before: https://phabricator.wikimedia.org/F25073327
  * After:  https://phabricator.wikimedia.org/F25073328

* Special:BotPasswords/foo
  * Before: https://phabricator.wikimedia.org/F25073324
  * After:  https://phabricator.wikimedia.org/F25073326

Bug: T181854
Change-Id: Ica67fe4081dfaa8eb9e8f56fdb93530750e47012

Merge "jquery.expandableField: Remove module unused since 1.22"

Merge "resourceloader: Increase minification cache version"

Merge "JavaScriptMinifier: Fix bad state after ternary in object literal"

Merge "jquery.suggestions: Remove dead code for returning options"

jquery.expandableField: Remove module unused since 1.22

Was originally created for Vector, which was originally part
of MediaWiki core. It hasn't been used for a while, and Vector
is in its own repository now.

Found 0 uses of this anywhere in Wikimedia Git, nor elsewhere
indexed by MediaWiki Codesearch.

Bug: T202154
Change-Id: I72ab5ad7f25be87b2f6d2f5f41a672b6a48b8005

Inject SpecialPageFactory into Parser

Change-Id: I6a6a94cbdafdc724ce02408cd9e744e7b3eda92b

Mass conversion to SpecialPageFactory service

Change-Id: Ia6e1e819ec6cbe8bf75b820109f51d47863e31fc

Merge "Make SpecialPageFactory a service"

Merge "Deprecate wfGetCache() and wfGetMainCache()"

Merge "Deprecate $wgParser"

Merge "Add release notes for $wgContLang deprecation"

Make SpecialPageFactory a service

Calling SpecialPageFactory methods statically is now soft-deprecated.

SpecialPageFactory::resetList() is a no-op, and I changed tests
in core to use overrideMwServices() instead.

Methods that fell back to $wgUser now require a User object being passed.

Depends-On: Ie1f80315871085b9fd4763a265b588849d94414d
Change-Id: Id8a92d57743f790b7d8c377c033cef38d1bb24de

Deprecate wfGetCache() and wfGetMainCache()

The former is already a wrapper around ObjectCache::getInstance().

The latter was identical to ObjectCache::getLocalClusterInstance().

Bug: T115890
Change-Id: Ib4e43bc8d3f4ac9f7a453e36dcce9b3d962666ba

Deprecate $wgParser

Bug: T160811
Change-Id: I7bcf4bbd14268ec069d97848c016e0098eaf0be4

jquery.suggestions: Remove dead code for returning options

    returnValue = (key in obj) ? undefined : obj[key];

By standard, this cannot return anything other undefined.
This code has been broken since the introduction of the module
in 2012 (32377424b, r72349).

The functionality also didn't have test (naturally), and isn't
used anywhere in Wikimedia Git. Simply remove it.

Also remove the confusing $() wrapping of the non-getter return
value that provides chainability.

The 'this' inside a jQuery method is already an instance of jQuery,
pulling it through $() again achieves very little.

Change-Id: Id13861d42b65a759d74670d8c5850aecee4daf7d

Add release notes for $wgContLang deprecation

Bug: T160815
Change-Id: I4dfdd8516b243f3eeccb69fe518a932178534aa2

mediawiki.util: Only call $.fn.updateTooltipAccessKeys if accesskey is set

Avoids overhead of mw.Message#parse, jqueryMsg.parser, and various
other DOM operations.

Bug: T202151
Change-Id: Id6304c9223806af6d469d9b4c67e60263f4b17ac

Simplify redirect linking in InfoAction

- Only call WikiPage::getRedirectTarget once
- Use the result of getRedirectTarget to determine if the page info must
be shown,
  because sometimes Title::isRedirect gets wrong state, see T172016 for
more information

Bug: T172016
Change-Id: Ic789207ba00d057a35e82a4c22628207eeb7b5fd

Message: Fix error on getTitle() without language

Steps to reproduce:

Old:
> $msg = new Message( 'brackets' )
> print $msg->getTitle()
Error from line 478 of /.../includes/Message.php: Call to a member function equals() on boolean

New:
> $msg = new Message( 'brackets' )
> print $msg->getTitle()
MediaWiki:Brackets

This got broken in 796d62d0.

Change-Id: I52cde36787e348378c4df18d90aa577a5600e06b

Improve profileinfo.php documentation

* Move $wgEnableProfileInfo to DefaultSettings.php

* The configuration variable to enable the entry point was added
  together with the entry point itself in 9af3c09e5c (r9846).

* Change references to StartProfiler.php to refer to LocalSettings.php,
  given the former is deprecated since 1.31 (I4e8dd9558132).

Change-Id: I7ca5f2deace8645f06bebd915630c1de05c84bc5

Merge "Html: Add test coverage for inlineScript()"

Merge "languages: Add coverage for 'ar' and 'ml' normalize()"

Merge "mediawiki.jqueryMsg: Combine the two generated data scripts"

Merge "mw.ForeignStructuredUpload: Use 'helpInline' instead of custom styles"

Merge "Fix eslint warnings and switch to error code"

Merge "Warn against certain jQuery methods"

Merge "benchmarks: Create $normBenchs before the run instead of during"

mw.ForeignStructuredUpload: Use 'helpInline' instead of custom styles

Before: https://phabricator.wikimedia.org/F25025765
After: https://phabricator.wikimedia.org/F25025764

Change-Id: I423c786e30570b8febc888b61450e8fce9185ba6

Merge "Remove incorrect timezone conversion from date parameters"

mediawiki.jqueryMsg: Combine the two generated data scripts

No need to output two mw.jqueryMsg.setParserDefaults() calls.
Follows-up 7e65b6b3a766c1.

Also:
* Fix an inline comment that referred to a non-existent method.
* Document why we pass deep=true.

Change-Id: I7414d4188a4d933e2132f80135cbda87241580c0

Localisation updates from https://translatewiki.net.

Change-Id: Icc72d5230b3c3c09c5095b52cc3d2191751a109f

Merge "rdbms: remove some dead code in LoadBalancer::setDomainPrefix"

benchmarks: Create $normBenchs before the run instead of during

Separates the concerns a bit better, and also makes the code easier
to debug with less distracting steps during the running of the
benchmark.

Change-Id: Ia5a18216cb77d39fd60cd76b23ebceee7324a250

resourceloader: Increase minification cache version

Follows-up 1f5f6fc204.

Bug: T201606
Change-Id: I0b5af067d1d44880c8122343f8e4a1b47e998619

JavaScriptMinifier: Fix bad state after ternary in object literal

The following pattern of input (found in jquery.js) triggered
this bug:

    call( {
        key: 1 ? 0 : function () {
            return this;
        }
    } );

The open brace changes state to PROPERTY_ASSIGNMENT (for object literals).
The colon after 'key' sets state to PROPERTY_EXPRESSION.

Each individual parts of an expression (identifiers and literal values)
is recognised with state *_EXPRESSION_OP, such as PROPERTY_EXPRESSION_OP.

The '1' after 'key:' correctly sets the state to PROPERTY_EXPRESSION_OP.
Upto there it goes well, but after that it goes wrong.

The question mark (TYPE_HOOK) in this context was wrongly switching
back to PROPERTY_EXPRESSION. That is a problem because that does not
handle TYPE_COLON, which meant '0: function' was seen together as a
sequence of continuous PROPERTY_EXPRESSION_OP where TYPE_FUNC may
not be handled.

Fixed by changing handling of TYPE_HOOK in PROPERTY_EXPRESSION to
switch states to EXPRESSION_TERNARY, and also performing a push
so that ternary handling can pop back to the property expression.

This mirrors the handling that already exists for ternaries in
the regular handling of EXPRESSION/EXPRESSION_OP (as opposed to
the variant for object literal properties).

Bug: T201606
Change-Id: I6104c839cfc3416257543b54a91b74cb4aa4193b

rdbms: remove some dead code in LoadBalancer::setDomainPrefix

Change-Id: I594a390b446c00eb604b0be63f67b2c8cbd45758

title: Disable the failing tests from TitlePermissionTest

Bug: T201776
Change-Id: I088bd797225e0c60c66de4d4d1aa12d0b5bf67d8

Merge "Set getEditConflictMainTextBox parameter type to array"

Merge "selenium: undo page"

Set getEditConflictMainTextBox parameter type to array

This is only called at one place in core and used in the TwoColConf
extention. The passed argument is always an array.

Change-Id: I4ca94442b79dc8871ed72943be733d100e093e44

Unbreak Special:JavaScriptTest

Follow-up to dec8009

Change-Id: I812253cc1a504a8bb876700026dbfd35a727458b

Merge "rdbms: use Database::getDomainId for transaction logging calls"

Merge "Fix spelling of word "necessary""

Merge "rdbms: Disable DBConnRef::selectDB() for sanity"

Fix spelling of word "necessary"

Bug: T201491
Change-Id: Ic983889a5702b8b8552c88ebc4d676f35e9fe61e

Fix some of the common typos

* supress -> suppress (Except in backup_LogTest.php)
* recomend -> recommend
* becuase -> because
* accross -> across

Bug: T201491
Change-Id: I8faa4e6cc688b3ee204b3f79ab55eb7b65cc1fdd

Merge "Have WANCacheReapUpdate::getEventAffectedKeys() accept LinkTarget"

Merge "Actually inject InterwikiLookup into MediaWikiTitleCodec"

Merge "Have LinkCache::getMutableCacheKeys() accept LinkTarget"

Merge "Title: Access properties directly instead of wrapper functions"

Localisation updates from https://translatewiki.net.

Change-Id: I4b07075f2c8b2965f542f08cfbb62d71b23d9099

Merge "Make LinkBatch::setCaller chainable"

Merge "Update OOUI to v0.28.0"

Update OOUI to v0.28.0

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

Bug: T109529
Bug: T183601
Bug: T183645
Bug: T185751
Bug: T189113
Bug: T189953
Bug: T190441
Bug: T190581
Bug: T194104
Bug: T197004
Bug: T198845
Bug: T201329
Bug: T201871
Depends-on: I3c000504661e190ac8752f69419bed28785601b5
Change-Id: I0c101a942011f6095678ffbd90f9bd536132091c

Merge "Add option to populateChangeTagDef not to update the count"

Merge "Remove dead code from Title::secureAndSplit()"

Merge "TitleValue: Don't use Assert for basic type checks"

Merge "Have TitleParser::parseTitle() default to NS_MAIN to match Title"

Add option to populateChangeTagDef not to update the count

I need this when I'm picking this up to continue populating

Change-Id: I52c74d3c1c59ee966027d5d9cb1a2bd3cf6dfc7a

Have WANCacheReapUpdate::getEventAffectedKeys() accept LinkTarget

Nothing specifically requires TitleValue here.

Change-Id: I62ee1995c49071db2871149b941277b29836a94a

Make LinkBatch::setCaller chainable

To allow for one-liners such as $lb->setCaller( __METHOD__ )->execute();

Change-Id: I45cd81d9e583f09aacb578ffb87d7c11b0e13d57

rdbms: use Database::getDomainId for transaction logging calls

Change-Id: I360ca633b145dc8e510bb00b542ac44933283943

Actually inject InterwikiLookup into MediaWikiTitleCodec

Change-Id: Ic469f0fb8068bfb06b741d957b21e66dfa839cd6

Merge "rdbms: split out private LoadBalancer::openLocalConnection method"

Have LinkCache::getMutableCacheKeys() accept LinkTarget

Change-Id: Id84590b3aa285e31a358877167fe8ff502077f3c

Title: Access properties directly instead of wrapper functions

Since this is pretty hot code, avoid the performance hit of a function
call when we can directly access the property.

Change-Id: I287de5eb003582b5704e9d93e2fc99d16c0ffb2a