Include NewPP report as a JSON configuration object
Closed, ResolvedPublic
Actions

Description

Emit NewPP report data as "configuration" variables
Use the configuration variable and output the result in the Performance inspector.

This change should go out the same time as we push the inspector so it's easy to use the new format.

Details

Subject	Repo	Branch	Lines +/-
Include JS variable for NewPP report	mediawiki/core	master	+64 -5
Revert "Move NewPP limit report HTML comments to JS variables" and followups	mediawiki/core	REL1_28	+65 -92
Revert "Move NewPP limit report HTML comments to JS variables" and followups	mediawiki/core	master	+65 -92
Move NewPP limit report HTML comments to JS variables	mediawiki/core	master	+65 -75
Include the NewPP report in Performance inspector	mediawiki/extensions/PerformanceInspector	master	+343 -21

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Declined	• aripstra	T138621 Evaluate UX/Design for Performance Inspector
Declined	None	T130581 Performance inspector on mobile
Resolved	Peter	T117411 Launch on-wiki page performance inspector
Resolved	aaron	T110763 Include NewPP report as a JSON configuration object

Event Timeline

ori created this task.Aug 29 2015, 12:26 AM

ori raised the priority of this task from to Needs Triage.

ori updated the task description. (Show Details)

ori added projects: Performance-Team, MediaWiki-Parser.

ori subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 29 2015, 12:26 AM

Krinkle claimed this task.Aug 29 2015, 12:34 AM

Krinkle triaged this task as Medium priority.

Krinkle set Security to None.

Krinkle moved this task from Inbox, needs triage to To-do: Goals prioritized current Quarter on the Performance-Team board.

PleaseStand subscribed.Aug 30 2015, 1:45 AM

ori added a parent task: T117411: Launch on-wiki page performance inspector.Nov 2 2015, 7:21 PM

ori mentioned this in T117411: Launch on-wiki page performance inspector.Nov 8 2015, 6:50 PM

Krinkle removed Krinkle as the assignee of this task.Dec 9 2015, 8:54 PM

Peter claimed this task.Jun 13 2016, 8:59 AM

Peter moved this task from To-do: Goals prioritized current Quarter to Doing (old) on the Performance-Team board.

Change 294292 had a related patch set uploaded (by Phedenskog):
Include the NewPP report in Performance inspector

https://gerrit.wikimedia.org/r/294292

gerritbot added a project: Patch-For-Review.Jun 14 2016, 11:53 AM

Peter added a project: MediaWiki-extensions-PerformanceInspector.Jun 14 2016, 11:55 AM

Peter updated the task description. (Show Details)

This is trickier than I initially thought. The parser currently adds the report to the page HTML. Adding the report as a JavaScript config script is not a simple change, because JavaScript config scripts are not part of the parser output; they're part of the interface.

So, I think the right way to do this is to leave performance data in the parser output as data, and leave it up to the interface code to extract this data and expose it to users. This should be done in OutputPage.php, most likely.

ori reassigned this task from Peter to aaron.Jun 27 2016, 6:58 PM

ori added a subscriber: Peter.

Change 299693 had a related patch set uploaded (by Aaron Schulz):
Move NewPP limit report HTML comments to JS variables

https://gerrit.wikimedia.org/r/299693

Change 299693 merged by jenkins-bot:
Move NewPP limit report HTML comments to JS variables

https://gerrit.wikimedia.org/r/299693

ori closed this task as Resolved.Jul 26 2016, 6:38 PM

ReleaseTaggerBot added projects: MW-1.28-release-notes, MW-1.28-release (WMF-deploy-2016-08-02_(1.28.0-wmf.13)).Jul 26 2016, 7:00 PM

Change 294292 merged by jenkins-bot:
Include the NewPP report in Performance inspector

https://gerrit.wikimedia.org/r/294292

Zebulon84 mentioned this in T129322: How to invoke the performance inspector?.Sep 6 2016, 7:39 PM

Change 320461 had a related patch set uploaded (by Bartosz Dziewoński):
Revert "Move NewPP limit report HTML comments to JS variables" and followups

https://gerrit.wikimedia.org/r/320461

I am reverting this due to numerous regressions – see T142210 and the patches linked there for details. I think a machine-readable limit report would be a very useful feature, but not at the cost of losing the human-readable limit report, as it was implemented here.

(PerformanceInspector is not yet deployed to WMF production, as far as I know, so this shouldn't be problematic for day-to-day operations of the sites. But I'm doing this after a branch cut just in case.)

Change 320461 merged by Bartosz Dziewoński:
Revert "Move NewPP limit report HTML comments to JS variables" and followups

https://gerrit.wikimedia.org/r/320461

Change 320522 had a related patch set uploaded (by Bartosz Dziewoński):
Revert "Move NewPP limit report HTML comments to JS variables" and followups

https://gerrit.wikimedia.org/r/320522

gerritbot added a project: Patch-For-Review.Nov 8 2016, 10:10 PM

Change 320522 merged by Bartosz Dziewoński:
Revert "Move NewPP limit report HTML comments to JS variables" and followups

https://gerrit.wikimedia.org/r/320522

matmarex mentioned this in T142210: wgPageParseReport is empty on preview / section preview.Nov 8 2016, 10:26 PM

matmarex removed projects: Patch-For-Review, MW-1.28-release-notes.Nov 8 2016, 10:47 PM

Krinkle moved this task from Doing (old) to Inbox, needs triage on the Performance-Team board.Nov 9 2016, 6:49 PM

Krinkle reassigned this task from aaron to Peter.Nov 10 2016, 8:08 PM

Krinkle moved this task from Inbox, needs triage to To-do: Goals prioritized current Quarter on the Performance-Team board.

Krinkle added a subscriber: aaron.

aaron renamed this task from Convert NewPP report from an HTML comment to a JSON configuration object to Include NewPP report as a JSON configuration object.Nov 10 2016, 8:28 PM

aaron updated the task description. (Show Details)

Change 320861 had a related patch set uploaded (by Aaron Schulz):
Include JS variable for NewPP report

https://gerrit.wikimedia.org/r/320861

gerritbot added a project: Patch-For-Review.Nov 10 2016, 11:00 PM

Krinkle reassigned this task from Peter to aaron.Nov 21 2016, 6:31 PM

Krinkle removed a project: MW-1.28-release (WMF-deploy-2016-08-02_(1.28.0-wmf.13)).

Krinkle moved this task from To-do: Goals prioritized current Quarter to Doing (old) on the Performance-Team board.Dec 3 2016, 5:49 AM

aaron moved this task from Doing (old) to Blocked (old) on the Performance-Team board.Dec 22 2016, 9:20 PM

Change 320861 merged by jenkins-bot:
Include JS variable for NewPP report

https://gerrit.wikimedia.org/r/320861

ReleaseTaggerBot added projects: MW-1.29-release (WMF-deploy-2017-01-17_(1.29.0-wmf.8)), MW-1.29-release-notes.Jan 6 2017, 4:00 AM

aaron closed this task as Resolved.Feb 17 2017, 9:31 PM