[09:35:08] <wikibugs>	 06Multimedia, 10GeoData, 10UploadWizard: Better support for Geocoding on the Upload Wizard - https://phabricator.wikimedia.org/T135599#2304248 (10Perhelion)
[09:37:53] <wikibugs>	 06Multimedia, 10GeoData, 10UploadWizard: Better support for Geocoding on the Upload Wizard - https://phabricator.wikimedia.org/T135599#2304264 (10Perhelion)
[09:45:11] <wikibugs>	 06Multimedia, 10GeoData, 10UploadWizard: Better support for Geocoding on the Upload Wizard - https://phabricator.wikimedia.org/T135599#2304282 (10Perhelion)
[12:33:28] <wikibugs>	 06Multimedia, 10MediaWiki-Parser: Exclude redundant srcset attribute values (e.g. same url) - https://phabricator.wikimedia.org/T135550#2304723 (10matmarex)
[15:35:07] <wikibugs>	 06Multimedia, 06Commons, 10MediaWiki-File-management, 10MediaWiki-Special-pages: Special:Missing files should be blank for wikis redirecting to commons - https://phabricator.wikimedia.org/T135640#2305384 (10Dvorapa)
[15:36:48] <MarkTraceur>	 matm<tab> prtksxna anything for Scrum of Scrums?
[15:45:58] <wikibugs>	 06Multimedia, 06Commons, 10MediaWiki-File-management, 10MediaWiki-Special-pages: Special:Missing files should be blank for wikis redirecting to commons - https://phabricator.wikimedia.org/T135640#2305472 (10matmarex) There is no "Special:Missing files". I assume you mean <https://en.wikipedia.org/wiki/Spec...
[15:47:44] <wikibugs>	 06Multimedia, 06Commons, 10MediaWiki-File-management, 10MediaWiki-Special-pages: Special:Missing files should be blank for wikis redirecting to commons - https://phabricator.wikimedia.org/T135640#2305473 (10matmarex)
[15:49:29] <grrrit-wm>	 (03PS1) 10TheDJ: Add a parser test for <audio> elements [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289436 
[15:53:47] <MarkTraceur>	 James_F: Anything for SoS?
[16:06:17] <prtksxna>	 o/
[16:26:14] <wikibugs>	 06Multimedia, 06Commons, 10MediaWiki-File-management, 10MediaWiki-Special-pages: Special:Missing files should be blank for wikis redirecting to commons - https://phabricator.wikimedia.org/T135640#2305669 (10Dvorapa) @matmarex Thank you, I translate its name from czech assuming my translation is the same as...
[17:21:37] <grrrit-wm>	 (03PS9) 10Brion VIBBER: Update ogv.js to 1.1.1 release [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/288345 (https://phabricator.wikimedia.org/T135098) 
[17:21:58] <grrrit-wm>	 (03CR) 10Brion VIBBER: "Ok this should be ready to merge, if no surprises!" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/288345 (https://phabricator.wikimedia.org/T135098) (owner: 10Brion VIBBER)
[17:50:27] <grrrit-wm>	 (03CR) 10Brion VIBBER: [C: 032] Fix the disablecontrols option for videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289308 (https://phabricator.wikimedia.org/T135516) (owner: 10TheDJ)
[17:50:46] <grrrit-wm>	 (03CR) 10Brion VIBBER: [C: 032] Allow transform callers to add a classname [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289318 (https://phabricator.wikimedia.org/T135501) (owner: 10TheDJ)
[17:55:27] <MarkTraceur>	 I didn't even go to SoS.
[17:55:39] <MarkTraceur>	 Markfail.
[18:03:56] <grrrit-wm>	 (03CR) 10Brion VIBBER: [C: 04-1] "No 'Audio.oga' file is registered during the parser test, so these render out as broken links:" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289436 (owner: 10TheDJ)
[18:54:11] <thedj>	 brion: did you test with the core dependency ?
[18:54:36] <brion>	 oops no
[18:54:50] <thedj>	 it's rediculious i know.. 
[18:54:55] <brion>	 but shouldn't a file be provided by the extension test case?
[18:54:58] <thedj>	 we really should do something about parser tests
[18:55:05] <thedj>	 brion: you can't
[18:55:15] <brion>	 haha
[18:55:17] <thedj>	 parsertests all run from core.
[18:55:36] <thedj>	 it's bonkers
[18:55:42] <brion>	 there should be hooks though
[18:55:56] <brion>	 for adding tables, doing other setup, adding the lists of parser test files
[18:56:37] <thedj>	 brion: the phpunit parsers testrunner lives in core
[18:56:42] <thedj>	 so it can only setup things in core
[18:56:53] <thedj>	 'from core'
[18:57:31] <brion>	 that's why core has hooks
[18:57:33] <thedj>	 the problem is not setting up the phpunit testcase, it's hooking into the specific one that sets up the parsertests
[18:57:54] <brion>	 for instance we add the transcode table to the list of tables parser test runner has to duplicate
[18:57:57] <thedj>	 anyway. i just went with what djvu and tiff handler did
[18:58:36] <brion>	 djvu and tiff are in core so they probably have stuff in core
[18:59:45] <grrrit-wm>	 (03CR) 10TheDJ: [C: 032] Update ogv.js to 1.1.1 release [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/288345 (https://phabricator.wikimedia.org/T135098) (owner: 10Brion VIBBER)
[19:03:06] <brion>	 ok so it looks like there's no fully suitable hook available for adding files to the parser test, might need to devise one
[19:03:26] <grrrit-wm>	 (03Merged) 10jenkins-bot: Fix the disablecontrols option for videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289308 (https://phabricator.wikimedia.org/T135516) (owner: 10TheDJ)
[19:03:29] <grrrit-wm>	 (03Merged) 10jenkins-bot: Allow transform callers to add a classname [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289318 (https://phabricator.wikimedia.org/T135501) (owner: 10TheDJ)
[19:05:11] <thedj>	 brion: there's a ticket about this somewhere... /me searches
[19:06:51] <grrrit-wm>	 (03Merged) 10jenkins-bot: Update ogv.js to 1.1.1 release [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/288345 (https://phabricator.wikimedia.org/T135098) (owner: 10Brion VIBBER)
[19:06:53] <brion>	 woops well i just filed https://phabricator.wikimedia.org/T135672 merge it if you find another :D
[19:08:24] <thedj>	 hmm, or maybe not
[19:08:35] <thedj>	 anyway, i'm guessing this comment pretty much says it all:
[19:08:35] <thedj>	 : tstarling.Apr 8 2016, 6:49 AM
[19:08:36] <thedj>	 I think it was a bad idea to make a complex ~1600 LOC integration test system be implemented as a single MediaWikiTestCase derived class. PHPUnit does not add much utility in this case. I think we should have a single implementation which provides an internal API which is used by both entry points.
[19:16:45] <grrrit-wm>	 (03CR) 10TheDJ: [C: 04-1] Apply ext.tmh.player.styles w/ videojs styles on mobile (031 comment) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/288643 (owner: 10Brion VIBBER)
[19:18:31] <brion>	 Heh yeah, tests in tests is funky
[19:19:05] <brion>	 Is using not in a root efficient selector? I always worry about fancy stuff like that
[19:19:52] <thedj>	 the browser matches the rule right to left, so it shouldn't make much difference
[19:20:35] <thedj>	 IE had a bug with * in the past, but that's a long time ago.
[19:21:12] <brion>	 Spiffy
[19:21:24] <brion>	 Yeah should work
[19:22:17] <thedj>	 i didn't test that, just thought of it while reading the code
[19:23:19] * thedj brews coffee, to get to work on mwembed
[19:26:25] <grrrit-wm>	 (03PS2) 10Brion VIBBER: Apply ext.tmh.player.styles w/ videojs styles on mobile [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/288643 
[19:26:53] <grrrit-wm>	 (03CR) 10Brion VIBBER: Apply ext.tmh.player.styles w/ videojs styles on mobile (031 comment) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/288643 (owner: 10Brion VIBBER)
[19:27:14] <brion>	 we have to destroy the mwembed village in order to save it
[19:29:11] <thedj>	 brion: what do you think of the infobutton patch ?
[19:29:47] <thedj>	 it's not passing jenkins yet, since i'm doubting between mediawiki codestyle and videojs codestyle, but other than that.
[19:37:17] <brion>	 I'll have to take a look
[20:30:35] <thedj>	 42 issues fixed in TMH since january
[20:31:04] <thedj>	 only 200 to go. + all the subtasks that will probably still be created :)
[20:50:05] <grrrit-wm>	 (03PS1) 10TheDJ: Remove poster attribute from <audio> elements [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289551 (https://phabricator.wikimedia.org/T135603) 
[21:23:24] <grrrit-wm>	 (03CR) 10TheDJ: [C: 032] Apply ext.tmh.player.styles w/ videojs styles on mobile [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/288643 (owner: 10Brion VIBBER)
[21:26:16] <wikibugs>	 06Multimedia, 06Commons, 06Community-Advocacy, 10Wikimedia-General-or-Unknown, 06Zero: Commons and testwiki used as video hoster by Wikipedia Zero - https://phabricator.wikimedia.org/T129845#2136145 (10Samuelbbc) Hello,  I'm a radio producer with the BBC and am hoping to speak to a member of this group a...
[21:29:28] <wikibugs>	 06Multimedia, 06Commons, 06Community-Advocacy, 10Wikimedia-General-or-Unknown, 06Zero: Commons and testwiki used as video hoster by Wikipedia Zero - https://phabricator.wikimedia.org/T129845#2307209 (10Matthewrbowker) >>! In T129845#2307192, @Samuelbbc wrote: > Hello, >  > I'm a radio producer with the B...
[21:32:29] <wikibugs>	 06Multimedia, 06Commons, 06Community-Advocacy, 10Wikimedia-General-or-Unknown, 06Zero: Commons and testwiki used as video hoster by Wikipedia Zero - https://phabricator.wikimedia.org/T129845#2307223 (10Samuelbbc) Thanks for getting back to me Matthew - I've put a request in to the Wikimedia foundation as...
[21:34:00] <grrrit-wm>	 (03Merged) 10jenkins-bot: Apply ext.tmh.player.styles w/ videojs styles on mobile [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/288643 (owner: 10Brion VIBBER)
[23:01:16] <grrrit-wm>	 (03PS1) 10TheDJ: Allow mwEmbed to pick up classes from media element [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/289573 (https://phabricator.wikimedia.org/T135537) 
[23:03:23] <MarkTraceur>	 prtksxna brion TrevorParscal anyone for a meeting?
[23:04:26] <thedj>	 bedtime for me. Have a nice rest of the day ppl.
[23:06:31] <MarkTraceur>	 I think we'll cut it out then