Page MenuHomePhabricator
Create Task
Maniphest T280746

Reduce number of distinct module bundles registered by TimedMediaHandler
Closed, ResolvedPublic
Actions

Description

For context and rationale, see:

This extension has long been one of the largest extensions in terms of its share in the startup.js payload for all page views in production. From Grafana: Startup manifest size it currently registers 54 module bundles which transfer about 2KB on every page view during the critical path just to communicate the client about their existence.

I've thus far restisted the temptation to reduce this proactively knowing that a lot of this will be cut away once the transition from Kaltura to video.js completes (ref T100106).

However, I think there's a number of modules here even ignoring the legacy Kaltura parts that can potentially be reduced, and we may be able to do some very quick/easy consolidation or removal of unused modules besides that as well.

2021 progress:

  • https://gerrit.wikimedia.org/r/722899 : -10 modules by merging dependencies into mw.EmbedPlayer (removed mediawiki.absoluteUrl, mediawiki.client, fullScreenApi, jquery.debouncedresize, jquery.embedMenu, mw.EmbedTypes, mw.MediaElement, mw.MediaSource, mw.MediaPlayers, mw.MediaPlayer)
  • https://gerrit.wikimedia.org/r/722898 : -1 module by merging mw.PopUpMediaTransform.styles into mw.PopUpMediaTransform
  • https://gerrit.wikimedia.org/r/723195 : -2 modules by removing unused mw.EmbedPlayerVlc and mw.EmbedPlayerGeneric
  • https://gerrit.wikimedia.org/r/723199 : -2 modules by merging jquery.triggerQueueCallback and jquery.mwEmbedUtil into mw.MwEmbedSupport
  • https://gerrit.wikimedia.org/r/723201 : -1 module by merging mw.MwEmbedSupport.style into mw.MwEmbedSupport
  • https://gerrit.wikimedia.org/r/723635: -1 module by merging mw.ajaxProxy into mw.TextSource
  • https://gerrit.wikimedia.org/r/723638 : -2 modules by merging mw.EmbedPlayer.loader and mw.TimedText.loader into mw.MediaWikiPlayer.loader
  • https://gerrit.wikimedia.org/r/723639: -1 module by merging mw.TextSource into mw.TimedText
  • MABYE: mediawiki.UtilitiesTime is used by mw.EmbedPlayer and mw.TextSource, after the step above will be used by mw.EmbedPlayer and mw.TimedText, but mw.TimedText also depends on mw.EmbedPlayer, so mediawiki.UtilitiesTime can be merged into mw.EmbedPlayer ?
  • Consider merging some of the smaller mw.EmbedPlayer* modules. Sizes (library-lines): Kplayer-496, IEWebMPrompt-46 (plus 18 lines of styles), ImageOverlay-316, OgvJs-266 (has fairly small dependencies), VLCApp-101 (mediawiki.Uri dependency). Or maybe merge them all into a single module? It means some extra code would be loaded when not needed, but we could use package files to not run it when not needed, or just let it all run because it shouldn't conflict with others

Details

ProjectBranchLines +/-Subject
mediawiki/extensions/TimedMediaHandlermaster+0 -29Remove unused style module `ext.tmh.thumbnail.styles`
mediawiki/extensions/TimedMediaHandlermaster+6 -11Merge mw.TextSource module into mw.TimedText
mediawiki/extensions/TimedMediaHandlermaster+7 -11Merge dependencies into mw.MediaWikiPlayer.loader module
mediawiki/extensions/TimedMediaHandlermaster+10 -420Drop unused mw.EmbedPlayerVlc and mw.EmbedPlayerGeneric
mediawiki/extensions/TimedMediaHandlermaster+5 -6Merge mw.ajaxProxy into mw.TextSource
mediawiki/extensions/TimedMediaHandlermaster+5 -8Merge mw.MwEmbedSupport.style module into only user
mediawiki/extensions/TimedMediaHandlermaster+3 -9Merge two dependencies into `mw.MwEmbedSupport` module
mediawiki/extensions/TimedMediaHandlermaster+1 -4Merge mw.PopUpMediaTransform.styles module into only user
mediawiki/extensions/TimedMediaHandlermaster+12 -53Merge 10 dependencies into mw.EmbedPlayer module
mediawiki/extensions/TimedMediaHandlermaster+10 -1Grep comments for dynamically created RL module names loading
mediawiki/extensions/TimedMediaHandlermaster+3 -13Bundle `mw-info-button` with ext.tmh.player.inline
mediawiki/extensions/TimedMediaHandlermaster+7 -13Bundle videojs-resolution-switcher with ext.tmh.player.inline
mediawiki/extensions/TimedMediaHandlermaster+6 -12mw.MwEmbedSupport: Bundle Spinner and jquery.loadingSpinner
mediawiki/extensions/TimedMediaHandlermaster+5 -10ext.tmh.player.dialog: Move player inline styles into this bundle
mediawiki/extensions/TimedMediaHandlermaster+4 -5mwembed: Merge "mw.Api" into "mw.MediaWikiPlayerSupport" bundle
mediawiki/extensions/TimedMediaHandlermaster+0 -1 Kmwembed: Remove unused "iScroll" module
Show related patches Customize query in gerrit

Related Objects

Event Timeline

Krinkle created this task.Apr 21 2021, 3:05 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 21 2021, 3:05 AM
gerritbot added a comment.Apr 21 2021, 3:08 AM

Change 681509 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/extensions/TimedMediaHandler@master] mwembed: Remove unused "iScroll" module

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

gerritbot added a project: Patch-For-Review.Apr 21 2021, 3:08 AM
gerritbot added a comment.Apr 21 2021, 3:19 AM

Change 681510 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/extensions/TimedMediaHandler@master] mwembed: Merge "mw.Api" into "mw.MediaWikiPlayerSupport" bundle

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

gerritbot added a comment.Apr 21 2021, 3:19 AM

Change 681511 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/extensions/TimedMediaHandler@master] ext.tmh.player.dialog: Move player inline styles into this bundle

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

gerritbot added a comment.Apr 21 2021, 3:23 AM

Change 681512 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/extensions/TimedMediaHandler@master] mw.MwEmbedSupport: Bundle Spinner and jquery.loadingSpinner

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

Krinkle added a comment.EditedApr 21 2021, 3:29 AM

I've shaken out about 5 modules to start with (based on a quick script that looks for modules in extension.json with either 0 or 1 dependency to it; and then confirming with git-grep that it has no loading logic in PHP or JS, and also via Codesearch no references to it from anywhere.

These 5 would be enough to get our registry size back to at least where we were in January which would be nice. TMH didn't regress this quarter, but it's overdue to slim down a bit, and it was the easy target to free up some budget.

Krinkle added a subscriber: TheDJ.Apr 21 2021, 3:29 AM
gerritbot added a comment.Apr 21 2021, 3:54 AM

Change 681509 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] mwembed: Remove unused "iScroll" module

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

gerritbot added a comment.Apr 21 2021, 4:14 AM

Change 681510 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] mwembed: Merge "mw.Api" into "mw.MediaWikiPlayerSupport" bundle

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

gerritbot added a comment.Apr 21 2021, 4:14 AM

Change 681511 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] ext.tmh.player.dialog: Move player inline styles into this bundle

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

gerritbot added a comment.Apr 21 2021, 4:14 AM

Change 681512 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] mw.MwEmbedSupport: Bundle Spinner and jquery.loadingSpinner

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

ReleaseTaggerBot added a project: MW-1.37-notes (1.37.0-wmf.3; 2021-04-27).Apr 21 2021, 5:00 AM
Maintenance_bot removed a project: Patch-For-Review.Apr 21 2021, 5:10 AM
Krinkle moved this task from Limbo to Perf recommendation on the Performance-Team (Radar) board.Apr 29 2021, 8:19 PM
gerritbot added a comment.Apr 30 2021, 7:55 PM

Change 683985 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/extensions/TimedMediaHandler@master] Bundle videojs-resolution-switcher with ext.tmh.player.inline

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

gerritbot added a project: Patch-For-Review.Apr 30 2021, 7:55 PM
gerritbot added a comment.Apr 30 2021, 10:58 PM

Change 683985 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Bundle videojs-resolution-switcher with ext.tmh.player.inline

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

ReleaseTaggerBot edited projects, added MW-1.37-notes (1.37.0-wmf.4; 2021-05-04); removed MW-1.37-notes (1.37.0-wmf.3; 2021-04-27).Apr 30 2021, 11:00 PM
Maintenance_bot removed a project: Patch-For-Review.Apr 30 2021, 11:10 PM
gerritbot added a comment.Jun 11 2021, 6:50 PM

Change 699461 had a related patch set uploaded (by Krinkle; author: Krinkle):

[mediawiki/extensions/TimedMediaHandler@master] Bundle `mw-info-button` with ext.tmh.player.inline

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

gerritbot added a project: Patch-For-Review.Jun 11 2021, 6:50 PM
gerritbot added a comment.Jun 11 2021, 7:30 PM

Change 699461 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Bundle `mw-info-button` with ext.tmh.player.inline

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

ReleaseTaggerBot edited projects, added MW-1.37-notes (1.37.0-wmf.11; 2021-06-21); removed MW-1.37-notes (1.37.0-wmf.4; 2021-05-04).Jun 11 2021, 8:00 PM
Krinkle removed projects: MW-1.37-notes (1.37.0-wmf.11; 2021-06-21), Patch-For-Review.Jun 12 2021, 5:33 PM
gerritbot added a comment.Sep 22 2021, 4:18 PM

Change 722898 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Merge mw.PopUpMediaTransform.styles module into only user

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

gerritbot added a project: Patch-For-Review.Sep 22 2021, 4:18 PM
DannyS712 added a subscriber: DannyS712.Sep 22 2021, 4:20 PM
gerritbot added a comment.Sep 22 2021, 4:42 PM

Change 722899 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Merge 10 dependencies into mw.EmbedPlayer module

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

DannyS712 added a comment.EditedSep 22 2021, 4:54 PM

For later, also mergable:
mw.MwEmbedSupport:

  • jquery.triggerQueueCallback
  • jquery.mwEmbedUtil (also loaded in one place alongside mw.MwEmbedSupport)

mw.TextSource:

  • mw.ajaxProxy

mw.MediaWikiPlayer.loader

  • mw.EmbedPlayer.loader
  • mw.TimedText.loader
gerritbot added a comment.Sep 22 2021, 4:57 PM

Change 722900 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Grep comments for dynamically created RL module names loading

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

gerritbot added a comment.Sep 22 2021, 6:51 PM

Change 722900 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Grep comments for dynamically created RL module names loading

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

ReleaseTaggerBot added a project: MW-1.38-notes (1.38.0-wmf.2; 2021-09-28).Sep 22 2021, 7:00 PM
gerritbot added a comment.Sep 23 2021, 6:58 PM

Change 722899 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Merge 10 dependencies into mw.EmbedPlayer module

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

gerritbot added a comment.Sep 23 2021, 7:06 PM

Change 722898 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Merge mw.PopUpMediaTransform.styles module into only user

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

gerritbot added a comment.Sep 23 2021, 7:09 PM

Change 723195 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Drop unused mw.EmbedPlayerVlc and mw.EmbedPlayerGeneric

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

gerritbot added a comment.Sep 23 2021, 7:27 PM

Change 723199 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Merge two dependencies into `mw.MwEmbedSupport` module

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

DannyS712 updated the task description. (Show Details)Sep 23 2021, 7:30 PM
DannyS712 added a project: User-DannyS712.Sep 23 2021, 7:34 PM
DannyS712 moved this task from Unsorted to In progress on the User-DannyS712 board.
gerritbot added a comment.Sep 23 2021, 8:58 PM

Change 723201 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Merge mw.MwEmbedSupport.style module into only user

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

DannyS712 updated the task description. (Show Details)Sep 23 2021, 9:28 PM
gerritbot added a comment.Sep 24 2021, 9:22 PM

Change 723635 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Merge mw.ajaxProxy into mw.TextSource

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

gerritbot added a comment.Sep 24 2021, 9:30 PM

Change 723199 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Merge two dependencies into `mw.MwEmbedSupport` module

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

gerritbot added a comment.Sep 24 2021, 9:30 PM

Change 723201 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Merge mw.MwEmbedSupport.style module into only user

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

gerritbot added a comment.Sep 24 2021, 10:35 PM

Change 723635 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Merge mw.ajaxProxy into mw.TextSource

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

gerritbot added a comment.Sep 24 2021, 11:14 PM

Change 723195 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Drop unused mw.EmbedPlayerVlc and mw.EmbedPlayerGeneric

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

Jdforrester-WMF updated the task description. (Show Details)Sep 25 2021, 1:01 AM
gerritbot added a comment.Sep 25 2021, 1:50 AM

Change 723638 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Merge dependencies into mw.MediaWikiPlayer.loader module

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

gerritbot added a comment.Sep 25 2021, 1:55 AM

Change 723639 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Merge mw.TextSource module into mw.TimedText

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

DannyS712 updated the task description. (Show Details)Sep 25 2021, 1:57 AM
gerritbot added a comment.Sep 27 2021, 10:23 PM

Change 723638 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Merge dependencies into mw.MediaWikiPlayer.loader module

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

gerritbot added a comment.Sep 27 2021, 10:29 PM

Change 723639 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Merge mw.TextSource module into mw.TimedText

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

DannyS712 updated the task description. (Show Details)Sep 28 2021, 1:14 AM
DannyS712 removed a project: Patch-For-Review.Sep 28 2021, 2:40 AM
DannyS712 added a comment.Sep 28 2021, 2:50 AM

Now down to 27 modules, and switching the video player should help with a bunch more

brion moved this task from To sort to In progress on the TimedMediaHandler board.Jan 12 2022, 6:35 PM
gerritbot added a comment.Mon, May 30, 11:17 AM

Change 801191 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/TimedMediaHandler@master] Remove unused style module `ext.tmh.thumbnail.styles`

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

gerritbot added a project: Patch-For-Review.Mon, May 30, 11:17 AM
gerritbot added a comment.Mon, May 30, 12:18 PM

Change 801191 merged by jenkins-bot:

[mediawiki/extensions/TimedMediaHandler@master] Remove unused style module `ext.tmh.thumbnail.styles`

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

ReleaseTaggerBot added a project: MW-1.39-notes (1.39.0-wmf.14; 2022-05-30).Mon, May 30, 1:00 PM
Jdforrester-WMF closed this task as Resolved.Mon, May 30, 6:50 PM
Jdforrester-WMF updated the task description. (Show Details)
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL