@Krinkle @Catrope
Could one of you please review these patches:
~~Generalize ResourceLoader 'excludepage' functionality : <https://gerrit.wikimedia.org/r/#/c/340768>~~
~~ResourceLoader: Add wildcard modules: <https://gerrit.wikimedia.org/r/#/c/347442/>~~
They may need rebasing, but that should not affect review.
Many thanks!
>>! In T180817#3786242, @Anomie wrote:
> The new patches needing review are:
>
> * [[https://gerrit.wikimedia.org/r/#/c/393260/|393260: ParserOutput: Add stateless transforms to getText()]] (Krinkle proposed this idea a while back)* use post-cache transform instead of splitting the parser cache:
> ** [[https://gerrit.wikimedia.org/r/#/c/393261/|393261: Use9907/|399907 (core): Add 'unwrap' ParserOutput statelesspost-cache transforms]]
> ** [[https://gerrit.wikimedia.org/r/#/c/3932849909/|393284: ParserOutput: Add 'deduplicateStyles9909 (CiteThisPage): Use 'unwrap' post-cache transformation]] - The patch to do the deduplication in core instead of setWrapOutputClass( false )]]
> ** [[https://gerrit.wikimedia.org/r/#/c/399910/|399910 (MobileFrontend): Use 'unwrap' post-cache transform instead of setWrapOutputClass( false ), when available]]
** [[https://gerrit.wikimedia.org/r/#/c/399912/|399912 (TemplateStyles): Use 'unwrap' post-cache transform instead of setWrapOutputClass( false )]]
** [[https://gerrit.wikimedia.org/r/#/c/399914/|399914 (Wikibase): Use 'unwrap' post-cache transform instead of setWrapOutputClass( false )]]
** [[https://gerrit.wikimedia.org/r/#/c/399908/|399908 (core): Remove wrapclass from parser cache key]]
* style deduplication:
** [[https://gerrit.wikimedia.org/r/#/c/394369/|394369 (TemplateStyles): Cache processed stylesheets during the parse]]
** [[https://gerrit.wikimedia.org/r/#/c/393283/|393283 (MobileFrontend): ApiMobileView: Disable 'deduplicateStyles' ParserOutput::getText() transformation]]
** [[https://gerrit.wikimedia.org/r/#/c/393284/|393284 (core): ParserOutput: Add 'deduplicateStyles' post-cache transformation]] (has an unaddressed -1)
** [[https://gerrit.wikimedia.org/r/#/c/393285/|393285 (TemplateStyles): Deduplicate embedded style rules]]
* post-cache transforms followup (not strictly related, but tech debt reduction for something that was introduced in part due to TemplateStyles):
** [[https://gerrit.wikimedia.org/r/#/c/393262/|393262 (core): Warn if stateful ParserOutput transforms are used]]
** [[https://gerrit.wikimedia.org/r/#/c/393263/|393263 (core): Hard-deprecate ParserOutput stateful transform methods]] (probably in a separate MW release)
* accurate edit previews
** [[https://gerrit.wikimedia.org/r/#/c/340768/|340768 (core): Generalize ResourceLoader 'excludepage' functionality]]
** [[https://gerrit.wikimedia.org/r/#/c/340769/|340769 (TemplateSandbox): Use new OutputPage::addContentOverride to allow previewing edits to JS/CSS]]
* [[https://gerrit.wikimedia.org/r/#/c/352835/|352835 (core): Add mw-parser-output to indicators container]] (needs to be rewritten)
* [[https://gerrit.wikimedia.org/r/393285|393285: Deduplicate embedded style rules]] - TemplateStyles patch to use it#/c/347442/|347442 (core): ResourceLoader: Add wildcard modules]]
> * [[https://gerrit.wikimedia.org/r/#/c/393283/|393283: ApiMobileView: Disable 'deduplicateStyles' ParserOutput::getText() transformation]] - To avoid breaking MobileFrontend's action=mobileview if the above are merged