Revert "Move NewPP limit report HTML comments to JS variables" and followups
authorBartosz Dziewoński <matma.rex@gmail.com>
Tue, 8 Nov 2016 21:03:21 +0000 (22:03 +0100)
committerBartosz Dziewoński <matma.rex@gmail.com>
Tue, 8 Nov 2016 21:35:15 +0000 (22:35 +0100)
commit0e15a6068a5a07fc109b5898ae51fdb8decafaf0
treefb1c0e91f467ae0e7978acc05ca66ca57b21800f
parent69ae945e8d39972a07bea89ddb64bc0189b43ac2
Revert "Move NewPP limit report HTML comments to JS variables" and followups

This change resulted in unreasonable feature loss (human-readable
limit report was gone). Three months and multiple followups later,
the functionality is still not completely restored. Given lack
of response from the original author, I think it is time to revert
and reconsider, especially since the 1.28 release is soon.

A machine-readable limit report would be a very useful feature,
but not at the cost of losing human-readable limit report.

This reverts the following commits:

* Move NewPP limit report HTML comments to JS variables
  b7c4c8717f964d1890d185ec3e6e9481fcb734e4
* Only pretty-print the parser report JS vars
  28adc4d7eef2d7d8e5696a4f9849538a769daa00
* Show wgPageParseReport on page previews too
  1255654ed5a89ed57491bda38f544ed87e3bc601
* Re-add human readable parser limit report
  0051f108b954b52b9981d5d85862ac1f292db80c
* Restore hooks.txt for ParserLimitReportFormat
  4663e7a7371fabb96ed9c909e5b93042c5f08438

Resolved minor merge conflicts in OutputPage (with 80e5b160)
and release notes.

Bug: T110763
Bug: T142210
Change-Id: Id88c8066fae3f369e8977b4b7488f67071bdeeb7
RELEASE-NOTES-1.28
includes/EditPage.php
includes/OutputPage.php
includes/parser/Parser.php
includes/parser/ParserOutput.php
includes/resourceloader/ResourceLoader.php