Set visibility for some methods and remove unused ones

* private: checkUnicodeCompliantBrowser(), makesafe(), unmakesafe().
* protected: safeUnicodeInput(), safeUnicodeOutput().
* unused and deleted: safeUnicodeText(), sectionAnchor().

Also updated documentation for touched methods.

Change-Id: I1a46113a6380505aeea71150b6d106ec68423002

Merge "Reduce connect timeout in MediaWiki::triggerJobs"

Reduce connect timeout in MediaWiki::triggerJobs

The fifth parameter to fsockopen is the number of seconds
to wait before declaring the connection a failure. When not
provided it defaults to the php ini value of 'default_socket_timeout'
which is 60s.  Under no circumstances should we wait 60s to
connect to ourselves,  100ms seems like a reasonable timeout
since its explicitly connecting to itself, but the exact timeout
could be slightly longer.

Change-Id: I18d328274ddf1e0848fce40ebf39e0466b5c4d5d

Merge "Remove wfDebugLog() call from wfSetupSession()"

Localisation updates from https://translatewiki.net.

Change-Id: I083069e8633deaa49055a255e05692dcb1072193

Merge "Sync up with Parsoid parserTests."

Merge "Handle exceptions when testing row validity"

Remove wfDebugLog() call from wfSetupSession()

Since Iffba121a99 (00b7f76) with the removal of wfHttpOnlySafe(),
session cookie's parameters are based only on configuration
settings, so there is no point to spam the "cookie" log group
with predicitible values.

Change-Id: I8b1cdea929cefc32dd8b01c2ecbf2d76bb64189f

Lowered high lock timeout in mergeViaLock() and lock() in BagOStuff

Change-Id: Ib5cd0a98ce6ec47edb24f8cc9f4df2c941ce1c38

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit b2e209c4d370a4a83f33bb1a9f6f7794e7fda3ca

Change-Id: I27e0373436cbfd16b1558ca2b2e998144b808f4b

Merge "Remove $wgHttpOnlyBlacklist"

Merge "Remove unused style for #loginend and #signupend"

Merge "Adding function documentation for Skin::footerLink()"

Remove unused style for #loginend and #signupend

loginend in skins/common/commonContent.css was added in
https://gerrit.wikimedia.org/r/53614
and is now unused.

clear:both for #loginend and #signupend in skins/monobook/main.css
is not necessary because there are no floating elements above.

Change-Id: Ic472f43f1d1a7ab4b7abb9f19067574f12ce6a6b

Merge "Fix documentation of wfDebugLog()"

Merge "Set up node-jscs via Grunt (and pass it)"

Merge "Update formatting for includes/specials/"

Merge "Autoloader incorrect case warnings fixed"

Update OOjs UI to v0.1.0

New changes:
4f4fc2c Localisation updates from https://translatewiki.net.
ea88cce Localisation updates from https://translatewiki.net.
37af34f Add the ability to clear the SearchWidget
1cd4b32 Make PopupWidget clippable.
81f2edc Localisation updates from https://translatewiki.net.
fa6e8e5 Make popup toolgroup labels a little closer to their icons

Change-Id: I63fb1f9edb24a86cc882b204075540427ac4d11b

Set up node-jscs via Grunt (and pass it)

* Set up Grunt via package.json (run `npm install` in mediawiki-core)
* Add grunt task for node-jscs (NEW)
  This is a style checker (as opposed to jshint, which is for
  code quality). There are a few small style-related things that
  JSHint can check (camelcase, onevar etc.) but those are being
  deprecated in JSHint v3, people should use more sophisticated
  tools like node-jscs for this instead. As such this commit
  removes moves of those options from our jshint configuration.
  See: http://jshint.com/blog/jshint-3-plans/
* Add grunt task for jshint
  This will use the same jshint configuration as we use on
  Jenkins but makes it easier to run locally from the command
  line by being part of the same `$ grunt test` task list.

Also:
* Changed jshintignore to use "dir/**"" instead of "/dir" or "dir"
  because the latter is not compatible with Grunt for some reason.
  See also https://github.com/gruntjs/grunt-contrib-jshint/issues/126.

Examples of coding style rules that were being violated that we
can now catch in node-jscs:
* Operator "," should stick to preceding expression
* Missing space after "if" keyword
* Multiple line break
* Empty block (in jquery.textSelection and mediawiki.language)

Bug: 54218
Change-Id: Ib9d7eab9f0d5cea5fb33f0b9f82e5554897fdfe0

Localisation updates from https://translatewiki.net.

Change-Id: Icad3f57cf182107ce7e813297f00473e75e4ad6f

Adding function documentation for Skin::footerLink()

Change-Id: Ib38173ed7bdc434274ff17dfd8c85e5ec95bfb29

Followup af4085a: reset $prev interwiki to null in between result sets

Change-Id: Ie2ba7b061e532e0ab749b0519a7e72bc018a1b5a

Merge "Automatically add a new line at the end of wfLogDBError()"

Allow interwiki searches to return arrays of interwiki results

This allows them to group by interwiki in more sane ways.
Also remove no longer used $terms paramter

Change-Id: I5d420595edfd35a1ce85662d2722ebbe2d357ce3

Improve docs for $wgExtensionCredits (and add 'license-name')

Follows-up 357eb3d488b9d8c which implemented 'license-name'.

Change-Id: I2b2c17965230558fd1650e3db92abafb082737d1

Autoloader incorrect case warnings fixed

Change-Id: I610de60b49840a26ddb7ceeb14264aec77dc8b96

Add getRobotPolicy()

Bug: 57764
Change-Id: I5d53b9b36b8ec5263bf7ebb13c7ef8c102c1ce68

Test against php 5.6 on travis

Change-Id: I7401f1d059437eadd3712c012a93a6a5b6ac4403

Merge "Use correct Creative Commons license abbreviation"

Merge "Removing Wikitravel from the default interwiki list"

Use correct Creative Commons license abbreviation

Change-Id: I7df989de9dca2ee056030ff6f5830d5f0124185e

Tweaked "latest" handling of filebackend stat entries

* Let "latest" stat entries override non "latest" so
  that future getFileStat() calls with the "latest" flag
  can actually have a cache hit.

Change-Id: I1e9391039537d608b89773b4d51575e3b364a751

Merge "[Regression] Delete autoblocks when removing autoblock flag"

Merge "DatabaseMysqlBase: Remove broken check for Percona Server"

Merge "Remove double wfDebug() call in Database.php"

Merge "Add support for specifying headers in FileRepo::quickImport"

Merge "Add Special:ListDuplicatedFiles expensive query special page."

Merge "Add default implementation of getLength to base MediaHandler class"

Add Special:ListDuplicatedFiles expensive query special page.

I saw some comments recently on commons suggesting that
better ways are needed to manage duplicate files (There are tools
for if a specific file is a duplicate, but no backlog of
outstanding duplicate files).

This seems like a fairly easy first step in that direction.

Wasn't sure if this should be an image gallery type
query page, or just a list. I think in this case a plain
list is more useful.

Change-Id: Ibe4b9da71ca6451ec4e6b0050feaf3ca70e1b888

Add support for specifying headers in FileRepo::quickImport

Previously only content-disposition headers were allowed.

This is needed so that TimedMediaHandler can put
X-Content-Duration headers on ogg transcodes.

Bug: 62928
Change-Id: Ic053be63b3edf348da95a88fc494fcf334430265

Add default implementation of getLength to base MediaHandler class

It was being called in File::getLength, but was only implemented
in a subclass, that's not even in core. Any method of a handler
class called from a method in File class should work on any
subclass of MediaHandler.

Change-Id: I94eda90ec3dd64b6ff2e3a5732aa539366cec521

Localisation updates from https://translatewiki.net.

Change-Id: I30f333c40b241dfeefea11e617f3f80c70a84bb1

Merge "Button group adjustments"

[Regression] Delete autoblocks when removing autoblock flag

Since I5403f6e1d7f8f07273cd5da8068b928fdddcdfc9 on reblock the block
will be updated and not deleted/inserted and that update did not trigger
a delete of existing autoblocks.

Bug: 62748
Change-Id: I50384a41d819689944907e653e2495391253831b

Merge "Misc fixes to UploadSourceField"

Update formatting for includes/specials/

Change-Id: I478dedd0b692eae002a0ca9b0ec15cb2c357411f

Link to bugzilla.wikimedia.org in README for consistency

Change-Id: I27a97cb89ba1981c931fe5fe2ae1422c832c4405

Remove double wfDebug() call in Database.php

Fix for I12adfb4fcb (54664be).

Change-Id: I6acfd35aae18af9d6e5ae29e5554ca7d04342af5

Automatically add a new line at the end of wfLogDBError()

I found two calls to wfLogDBError() that do not add a new line
at the end of the message. So instead of adding them to that
entries, I changed wfLogDBError() to automatically put it on
icoming messages; as for wfDebugLog().

Change-Id: Id014b5827a0aeef6873ebf08d78f0a3d7581d63b

Fix documentation of wfDebugLog()

- $public parameter does not exist anymore
- Put the type before the parameter name

Change-Id: I0f3a893803ab5466570ef45398f797e4714d9d37

Misc fixes to UploadSourceField

Follow-up I7c15fd4250 (00ca365).

- Added 'radio-id' configuration option to set the id of the radio button,
  and changed Special:Upload to use it. Had to keep the old way to set it
  since this class is used by some extensions.
- Always set the "for" attribute of the label to the field and the not the
  radio, since the text it contains does not describes the radio button.

Change-Id: I524f8743fd03546024f8c97852a2bcb3c23c6019

Merge "AutoLoader: Remove "Class ... not found" debug message"

Merge "Remove some unneeded local vars from EditPage.php"

Localisation updates from https://translatewiki.net.

Change-Id: I93194841aebc13bb2e6e147ae6170a3acf2186e7

Merge "Give name for oojs-ui i18n files"

Add a few @since to mediawiki.api

Change-Id: I0717d1171a7179beb4cee6a922d682864e600e9c

Give name for oojs-ui i18n files

Even though this shouldn't cause problems with array_merge etc.
everything else has a name, so why shouldn't this.

Change-Id: Id40d9036c2f3cf89343bcfd14d37a14d4c306d01

DatabaseMysqlBase: Remove broken check for Percona Server

Percona Server does not include its name in its version suffix.
It would be possible to check for precompiled versions of the
product using the version_comment variable, though not third-
party builds from its source code (which are merely indicated
as "Source distribution").

Given that even Percona itself does not distinguish between
Percona Server and MySQL in their version of the mysql command-
line client, I have chosen simply to remove the broken check.

I have also made the check for MariaDB consistent with that
in its version of mysql (by also checking for the old suffix
"-maria-").

Follows-up 0eab1ace67e7.

Change-Id: I62c3949b775a38f28f09568ebb28af2adb9be69b

Swap wfProfileIn/Out calls for ProfileSection

Change-Id: Iece1532241dda4297dd4ca2c93287c13462eb2e9

Merge "Avoid showing crazy staleness times at ActiveUsers"

Merge "Remove completely useless implementation of getInfo()"

MWException: Expand {{SITENAME}} in pagetitle with Message::text()

Bug: 58447
Change-Id: I5f4fcb6d6e044f708cd6adb1bc7312d592fc74dd

Localisation updates from https://translatewiki.net.

Change-Id: I18970accbce3b383e4ad89df2b5cd0d638861e26

Update OOjs UI to v0.1.0-pre (3b434d5388)

New changes:
076b85e Split autoAdd into catchall and group specific settings

Change-Id: Idb99c0af20f966ead553c7a7eded421ad8b44382

Removing Wikitravel from the default interwiki list

It has already been removed from the WMF list.

Note that this will have no effect on existing wikis, only new wikis.

Change-Id: I201bc28ab17fdae3b7c5d46f1c65d70e03380ca9

AutoLoader: Remove "Class ... not found" debug message

Per existing FIXME comment: "This is not very polite. Assume we
do not manage the class."

Reverts the remaining portion of r44296 / 2f584f68a95a while
clarifying the "Give up" comment.

Change-Id: I4471f9376cae6b3c3e8df99e960770863ebf37ca

Merge "Updating link protocols for WMF wikis in the interwiki map"

Merge "Adding Wikivoyage to the default interwiki map"

Merge "ResourceLoaderStartUpModule: Improve comment about modifiedTime hack"

Merge "ResourceLoaderStartUpModule: Remove no-op "delete isCompatible""

Merge "Fix possible error list of action=revisiondelete"

Exclude prop=uploadwarning from allimages and stashimageinfo

Was added with I4a0af8986f924cd127a73828e72da6998f28536c,
but looks only useful on prop=imageinfo

Change-Id: I59c5f11f83be7e59f317686ab7fa16ad6fda008b

Make Special:Whatlinkshere.php transcludable

This commit makes Special:Whatlinkshere transcludable by extending
SpecialWhatLinksHere with IncludableSpecialPage. It also adds a condition
in the showIndirectLinks function to avoid displaying the form when the
page is being transcluded.

Bug: 35486
Change-Id: I49d6be57a9a780f9f959cb4356c5ac026db81775

Remove $wgHttpOnlyBlacklist

This hack was added in r34083 / 6b16f4410897 to support IE for Mac.
That browser is no longer supported, and no additional user-agent
strings have been added in WMF configuration.

Change-Id: Iffba121a9964e2ad387fad8827ddfd8dabcbd12e

Updating link protocols for WMF wikis in the interwiki map

Change-Id: I18d281104c1c670c3b2dce6b2970930aba26bef0

Adding Wikivoyage to the default interwiki map

Note that this has already been added to the WMF interwiki map, so
it will have no effect on WMF wikis, only 3rd party wikis.

I'll update the link protocols in a follow-up commit.

Change-Id: I78fbd3ec32c02e2b080c3ee64c6bb36218930a84

Merge "Add link to PD help translatable pages from sidebar"

Merge "Get rid of Internet Explorer for Mac fixes in Monobook's main.css."

Fix possible error list of action=revisiondelete

Bug: 62853
Change-Id: I0efec9e8a6cf761def7673011bcc22a9f11111a9

Remove some unneeded local vars from EditPage.php

Looks like a left over from refactoring in r31475

Change-Id: I6ffd2e1db8a842bebc37ce67e6265714258e79f9

Merge "Fixed spacing"

Fixed spacing

- Added spaces after if/foreach/catch
- Added new line before end of file
- Added or removed spaces before/after parenthesis, comma
- Added spaces around string concat

Change-Id: I0590070f1b3542108e242730e8d9a3ba9831e94f

Localisation updates from https://translatewiki.net.

Change-Id: Idf379fd907b86b191bf863d4dded00e5d9c94f35

Merge "mediawiki.page.watch.ajax: Fail early if updateWatchLink is called wrong"

mediawiki.page.watch.ajax: Fail early if updateWatchLink is called wrong

If the exposed function is called from a gadget or user script but
no watch/unwatch link was found the function fails even if the
first parameter is a valid jQuery object as required by the
documentation. $link.attr(...) returns null and null.match(...)
fails.

A very simple example why this can happen is as follows:
mw.page.watch.updateWatchLink( $( '#ca-watch a' ), 'watch', 'loading' );
mw.page.watch.updateWatchLink( $( '#ca-watch a.loading' ), 'unwatch' );
(starts the spinning loading animation and tries to stop it
afterwards but may fail if something went wrong, e.g. the user
clicked the star).

The action parameter needs to be checked because it is used to
build an ID and a message key. Bad values turn the page in an
unrecoverable state (the watch link gets an empty action=, the
label turns into something <undefined> and no script can recover
that broken state since the ID turned into whatever). While such a
check is not necesarry in most cases it is here, because the
function is exposed.

Change-Id: I6ee9a7ee6b7c0fc7a5444674afd1ed6f8cacc858

Merge "Allow setting default thumb size in parser tests"

Allow setting default thumb size in parser tests

The mediawiki default thumb size is 180px.  The Parsoid default thumb
size is 220px, to match the default thumb size for most WMF wikipedias
(see https://bugzilla.wikimedia.org/show_bug.cgi?id=43336).

This discrepancy leads to inconsistent image-related test output.
Allow a test to set an explicit default thumb size with the
'thumbsize' option so that it is possible to write consistent tests.

Change-Id: Ib764d1f1660a50caaf8f0ff245822d1d1a1d264e

Merge "Update the documentation at the top of parserTests.txt"

Merge "Fix wording of protect-cascadeon"

Change URLs to mediawiki.org in comments to HTTPS

These are only documentation fixes
http://www.mediawiki.org --> https://www.mediawiki.org

Change-Id: I62ad42be1a3aac410cc53e98ce79389ceddd8988

Fix wording of protect-cascadeon

In some cases, users who are not capable of changing the protection level
will see the protect-cascadeon message. Update it to not imply that they
are able to change the protection level.

Change-Id: I102aea94f4b5ad29fa928e865d2f20ec7def4b30

Update the installer to use JSON i18n

Change-Id: Iedcacbd26385c331cd78b92f7ec55a4ef9c015ef

Merge "mediawiki.inspect: add method for grepping loaded modules"

ResourceLoaderStartUpModule: Improve comment about modifiedTime hack

* Re-ordering code to bring the modifiedTime hack and the
  loop it serves closer together.
* Separating the fact that it needs a value and the mtime of
  startup.js since that is just one of the three factors we use.
  This way it's clearer that the startup.js mtime is not just a
  bogus value, but not more or less important than wgCacheEpoch
  and modules mtime either.
* Remove duplicate '/* Methods */' comment, we already have this
  marker a few methods higher up.

Change-Id: Id3a07f02566c0f04b612b81f8353f70fa4ab3977

ResourceLoaderStartUpModule: Remove no-op "delete isCompatible"

Due to the way this function being defined as a function or var
declaration instead of a function expression assigned to a property
it can't be deleted.

JavaScript doesn't throw an error when deletion is not permitted
though, the operator returns false instead.

> delete isCompatible;
  false

We already removed 'delete startUp' from mediawiki.js in favour
of startUp = undefined; (r107402, r74325).

Change-Id: I7aa02e3f4deb3a4f00177b70978bfcb83c80988a

mediawiki.inspect: add method for grepping loaded modules

Example usage:

>>> mw.inspect.grep(':hover .settings-text')
["ext.uls.init"]
>>> mw.inspect.grep('options.expire')
["jquery.cookie", "mediawiki.user"]

Use cases:
* You're debugging a layout bug and want to trace a CSS rule to the module
  which inserts it.
* A JavaScript error contains a distinct and plausibly unique substring (such
  as a variable name), and you want to identify the module that is causing the
  error.

You can re-load the page with debug=1, but you still need to decipher load.php
URLs for non-file modules, and you might not be able to reproduce the issue in
debug mode. Alternately, you can grep a full clone of the production branch
that the wiki is running (assuming you have one handy), but that won't help you
if the module is a gadget.

Change-Id: Ie5f0e7d1c7022f8d399e895f157db8acefc8abee

Update OOjs UI to v0.1.0-pre (0f50cdc46c)

New changes:
6c014fd Revert "Use super to call parent methods"

Change-Id: Iccbb2a1b86af8dc10568855eecb7d31e67b86756

Merge "Update OOjs UI to v0.1.0-pre (53f2410336)"

Merge "Preferences.php: avoid "Undefined index" if key 'realname' don't exist in $formdata"