FYI, JoeWalsh added minification of the pagelib CSS in https://github.com/wikimedia/wikimedia-page-library/pull/244 and pagelib version 9.1.3.

PR 244 has been merged and published, and MCS deployed plus the JS and CSS of the pagelib URLs purged from Varnish, but doesn't show any noticeable improvement.

https://developers.google.com/speed/pagespeed/insights/?url=https%3A%2F%2Fen.m.wikipedia.org%2Fapi%2Frest_v1%2Fpage%2Fmobile-html%2FDog

Before change:

After change:

Not sure if we run into caching issues in pagespeed insights. I do see a .1s improvement when changing the URL to drop the .m domain to: https://en.wikipedia.org/api/rest_v1/page/mobile-html/Dog

We've met and decided that mobile-html should favor first paint performance of the first section, and delay any other processing of the rest of the page. Similar to the tradeoffs chosen by the Android app with mobile-sections.

@JoeWalsh will explore the approach of injecting settings that pagelib needs at the beginning of the page load.

We will need to review the transform and setup approach so that the base style changes like theme color and margins can be set up as early as possible, the trasnforms can be run only on the lead section, and then we can lazily do the rest after that work has been done, and with requestIdleCallback so that we block the main thread as little as possible.

Injecting the settings atDocumentStart and calling setup inline made no appreciable change to time to first paint on the iOS prototype.

Furthermore, removing the linked pagelib js script entirely and just making the callback in an inline script in the head tag also made no appreciable change to time to first paint.

Lastly, removing the pagelib js and not hiding the webview made no appreciable change to time to first paint.

Local loads are consistently between 0.5s and 1s on the prototype. Might be something else going on here that I'm missing, will re-check these numbers tomorrow. CC @Dbrant @Jhernandez @bearND

Change 533008 had a related patch set uploaded (by Joewalsh; owner: Joewalsh):
[mediawiki/services/mobileapps@master] Add mobile-html shell page endpoint

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

Change 533274 had a related patch set uploaded (by Joewalsh; owner: Joewalsh):
[mediawiki/services/mobileapps@master] Split PCS into pre/post load components

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

Change 534531 had a related patch set uploaded (by Joewalsh; owner: Joewalsh):
[mediawiki/services/mobileapps@master] Progressive content loading for mobile-html without splitting pagelib.

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

Change 533274 abandoned by Joewalsh:
Split PCS into pre/post load components

Reason:
replaced by 534531 - not necessary to split pagelib into separate components as it didn't have a huge impact on performance

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

Change 533008 abandoned by Joewalsh:
Add mobile-html shell page endpoint

Reason:
534531 is the better approach

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

Change 534531 merged by jenkins-bot:
[mediawiki/services/mobileapps@master] Speeds up time to first paint for mobile-html

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

[batch] remove patch for review tag from resolved tasks