[00:04:40] <marktraceur>	 Bloody testing.
[00:05:08] <grrrit-wm>	 (03PS5) 10MarkTraceur: Add truncatable text field, use for some fields [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121309 
[01:04:48] <grrrit-wm>	 (03PS18) 10Aarcos: Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 
[01:05:27] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos)
[01:17:28] <grrrit-wm>	 (03PS19) 10Aarcos: Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 
[01:22:11] <grrrit-wm>	 (03CR) 10Aarcos: "I think I responded to all your comments Gergo, PTAL. As discussed verbally, I came up with a workaround for the problems that you see in:" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos)
[01:22:23] <grrrit-wm>	 (03CR) 10Aarcos: Download tab, basic download and preview (039 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos)
[02:24:50] <grrrit-wm>	 (03PS1) 10Gergő Tisza: Create StripeButtons class, convert reuse button to use it [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122631 
[02:25:22] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Create StripeButtons class, convert reuse button to use it [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122631 (owner: 10Gergő Tisza)
[06:23:02] <grrrit-wm>	 (03CR) 10Gilles: "I confirm that I left target=_blank as a safety net in case for some reason ?download stops working. Recent browsers correctly clear the t" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos)
[07:07:48] <grrrit-wm>	 (03CR) 10Gilles: [C: 032] Use mw-mmv CSS prefix consistently for all things [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122572 (owner: 10Gergő Tisza)
[07:08:19] <grrrit-wm>	 (03Merged) 10jenkins-bot: Use mw-mmv CSS prefix consistently for all things [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122572 (owner: 10Gergő Tisza)
[07:14:00] <grrrit-wm>	 (03CR) 10Gilles: [C: 031] Refactor dropdown size calculation [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122611 (owner: 10Gergő Tisza)
[07:25:56] <grrrit-wm>	 (03CR) 10Siebrand: [C: 031] "i18n/L10n reviewed." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos)
[07:32:25] <grrrit-wm>	 (03CR) 10Gilles: [C: 04-1] Add truncatable text field, use for some fields (037 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121309 (owner: 10MarkTraceur)
[07:38:18] <grrrit-wm>	 (03CR) 10Siebrand: [C: 04-1] "i18n/L10n reviewed. May contain changes author didn't mean to submit." (032 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122631 (owner: 10Gergő Tisza)
[07:53:36] <grrrit-wm>	 (03PS3) 10Gilles: Disable IE < 9 [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122422 
[08:09:34] <grrrit-wm>	 (03PS6) 10Gilles: Fixes various bugs in IE [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122398 
[08:09:59] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Fixes various bugs in IE [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122398 (owner: 10Gilles)
[08:12:01] <grrrit-wm>	 (03PS7) 10Gilles: Fixes various bugs in IE [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122398 
[08:12:09] <grrrit-wm>	 (03CR) 10Gilles: Fixes various bugs in IE (032 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122398 (owner: 10Gilles)
[08:12:24] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Fixes various bugs in IE [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122398 (owner: 10Gilles)
[08:37:39] <grrrit-wm>	 (03PS8) 10Gilles: Fixes various bugs in IE [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122398 
[08:47:03] <grrrit-wm>	 (03PS3) 10Gilles: Test to compare the performance of MMV and the Commons File: page [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/119917 
[09:59:03] <grrrit-wm>	 (03PS1) 10Gilles: Fix breakage on beta due to type hint being too strict [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122713 
[10:00:14] <grrrit-wm>	 (03CR) 10Gilles: [C: 032] "Self-merge because MMV is broken on beta without this change" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122713 (owner: 10Gilles)
[10:00:45] <grrrit-wm>	 (03Merged) 10jenkins-bot: Fix breakage on beta due to type hint being too strict [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122713 (owner: 10Gilles)
[10:31:32] <grrrit-wm>	 (03PS4) 10Gilles: Test to compare the performance of MMV and the Commons File: page [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/119917 
[10:31:46] <grrrit-wm>	 (03CR) 10Gilles: Test to compare the performance of MMV and the Commons File: page (0312 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/119917 (owner: 10Gilles)
[13:18:15] <grrrit-wm>	 (03PS2) 10Gilles: Select html embed by default for logged-out users [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121064 
[16:32:10] <grrrit-wm>	 (03CR) 10Aarcos: [C: 031] Fixes various bugs in IE [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122398 (owner: 10Gilles)
[16:33:32] <grrrit-wm>	 (03CR) 10MarkTraceur: [C: 04-1] "Needs a rebase." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122611 (owner: 10Gergő Tisza)
[16:53:50] <grrrit-wm>	 (03CR) 10Aarcos: Add truncatable text field, use for some fields (033 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/121309 (owner: 10MarkTraceur)
[17:24:56] <grrrit-wm>	 (03CR) 10MarkTraceur: "Other issues" (039 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122611 (owner: 10Gergő Tisza)
[17:29:07] <umang13>	 Hey! anybody has an idea how to test functions inside mediawiki? gives me error as File not a valid entry point into mediawiki
[17:30:15] <marktraceur>	 umang13: Do you mean "unit testing" or "running the function to see what it does"?
[17:31:16] <marktraceur>	 The former is somewhat a subset of the latter, I guess.
[17:31:30] <umang13>	 Actually I wanted to include the file language.php into my test code index2.php and access a function of the class in language.php
[17:31:54] <marktraceur>	 Uh...huh
[17:31:57] <marktraceur>	 umang13: Why?
[17:32:26] <umang13>	 there is a function truncate.php
[17:32:30] <umang13>	 sorry truncate ()
[17:32:31] <umang13>	 :p
[17:32:46] <umang13>	 i wanted to do some string munching
[17:33:17] <marktraceur>	 umang13: Is this code that will eventually go into a MediaWiki context, or is it a separate project entirely that just happens to use PHP?
[17:33:50] <marktraceur>	 In the former case, I would say stop procrastinating and put it into MediaWiki contexts *now*, in the latter case I would say you should copy the truncate code instead of trying to include the file
[17:35:20] <umang13>	 It is the former case
[17:35:29] <umang13>	 thanks
[17:37:15] * marktraceur  gets whiplash

[17:37:22] <marktraceur>	 When did we start using JSON i18n files
[17:37:56] <grrrit-wm>	 (03PS2) 10Gergő Tisza: Create StripeButtons class, convert reuse button to use it [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122631 
[17:38:00] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Create StripeButtons class, convert reuse button to use it [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122631 (owner: 10Gergő Tisza)
[17:39:10] <tgr>	 marktraceur: Siebrand has been converting extensions for a couple days now
[17:40:33] <tgr>	 umang13: in theory you can use phpsh to get a php shell with a preloaded mediawiki context if you are on a vagrant box
[17:40:37] <tgr>	 never tried though
[17:41:13] <marktraceur>	 Oh cool.
[17:41:32] <umang13>	 <+tgr> ok, will look into it
[17:44:00] <marktraceur>	 umang13: Protip, you can use the "tab" key to complete usernames
[17:44:06] <marktraceur>	 Try typing "mar" and then hitting tab.
[17:44:44] <marktraceur>	 I do the same thing with tgr, even though he has a short nick, just to make sure he's still in the room.
[17:45:15] <umang13>	 marktraceur: Oh nice! thanks :)
[17:46:01] <umang13>	 marktraceur: I actually did not get you completely. The file I am testing is not going into mediawiki. It is just testing a patch that i am going to submit for another file in extensions.
[17:46:41] <umang13>	 I need a function from language.php as i am planning to use a function defined there for my patch
[17:46:46] <marktraceur>	 umang13: OK, then you should add the test to that extension's unit tests.
[17:46:59] <marktraceur>	 Rather than hacking in your test in a third-party untracked script that we'll never see.
[17:47:03] <marktraceur>	 umang13: What extension?
[17:47:15] <umang13>	 TimedMediaHandler
[17:47:26] <marktraceur>	 Oh, good, they should have a phpunit directory already
[17:47:40] <marktraceur>	 umang13: Look at phpunit/tests
[17:47:41] <umang13>	 yeah, will look at that
[17:47:42] <marktraceur>	 Er.
[17:47:44] <marktraceur>	 tests/phpunit
[17:48:01] <marktraceur>	 Either add to one of those files or add a new file of the same format
[17:52:19] <umang13>	 marktraceur: after adding to this directory I just run it normally like any php script is there another way??
[17:53:10] <marktraceur>	 umang13: You should run the phpunit tests from the base mediawiki directory.
[17:53:14] * marktraceur  looks for docs

[17:53:24] <marktraceur>	 umang13: https://www.mediawiki.org/wiki/Phpunit
[18:15:59] <grrrit-wm>	 (03PS3) 10Gergő Tisza: Create StripeButtons class, convert reuse button to use it [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122631 
[18:18:32] <marktraceur>	 tgr, gi11es, I'm worried that we're doing utility classes wrong - shouldn't they be static?
[18:19:54] <tgr>	 not much of a difference IMO
[18:20:33] <tgr>	 actually i made HtmlUtils static at first, then converted it because of a misunderstood JSDuck error
[18:20:47] <tgr>	 and then i was too lazy to convert back
[18:20:57] <marktraceur>	 Hahaha
[18:21:11] <marktraceur>	 tgr: The difference would be memory allocation, I guess, which shouldn't impact anything too much
[18:21:15] <tgr>	 anyway, in PHP i would avoid statics like plague
[18:21:17] <marktraceur>	 But the ocd in me is complaining
[18:21:22] <marktraceur>	 Oh?
[18:21:27] <marktraceur>	 Use singletons instead, I guess?
[18:21:34] <tgr>	 even with simple utility classes it can be a major pain in unit testing
[18:21:41] <tgr>	 no, singletons are almost as bad
[18:22:05] <tgr>	 dependency injection wins big time if you want to write good tests
[18:22:12] <marktraceur>	 *nod*
[18:22:41] <tgr>	 in javascript it is not that much of a deal due to duck typing and the ability to replace functions dynamically
[18:24:29] <tgr>	 i don't think there is any significant difference in functionality, either way you have a prototype object with all the actual contents, if you use instances then you have a few empty objects on top of that, but that's like a couple dozen bytes of memory
[18:26:09] <tgr>	 IDE support for autocompletion etc might differ, but my IDE sucks both ways, not sure if others favor one or the other
[18:33:41] <marktraceur>	 Yeah, my vim will do autocomplete of whatever I tell it to
[18:33:44] <marktraceur>	 So it ain't matter
[18:35:01] <marktraceur>	 this.util.TruncatableTextField.ThumbnailSizeCalculator.UploadWizardUploadInterface.loadViewer
[18:35:18] <marktraceur>	 vim doesn't care
[18:36:46] <tgr>	 anyway, i can turn it back static if you feel strongly about this
[18:36:56] <marktraceur>	 Not really
[18:37:08] <tgr>	 a self-instantiated object is not much different from a static anyway
[18:37:12] <marktraceur>	 Just a little weirded out by having util instances of the same type in many places
[18:37:26] <marktraceur>	 Oh, maybe I'm misreading how this works
[18:37:32] <tgr>	 would be nice to haver a proper dependency injection container, but i doubt we will have the time for that
[18:38:51] <tgr>	 well, an instance is basically something like an object {__prototype__: mw.mmv.HtmlUtils}
[18:38:56] <tgr>	 so it is really small
[18:40:13] <marktraceur>	 Yeah, I see that now
[18:40:28] <marktraceur>	 I think I looked at it and thought HtmlUtils was HUP
[18:40:33] <marktraceur>	 So this is more better
[18:57:54] <marktraceur>	 	»   »   // Can't add the @property definition because jsduck parses files in alphabetical order 	40
[18:57:57] <marktraceur>	 			»   »   // and .embed comes before .utils 
[18:58:00] <marktraceur>	 Hahahahaha
[18:59:01] <tgr>	 that was another misunderstanding
[18:59:05] <marktraceur>	 Oh?
[18:59:18] <tgr>	 clarity of error messages is not a strong point of jsduck
[18:59:21] <marktraceur>	 Haha, no
[18:59:30] <marktraceur>	 I guess I will comment on it then
[18:59:42] <tgr>	 isn't it fixed already?
[18:59:56] <marktraceur>	 Uh...maybe? I'm reviewing PS
[18:59:59] <marktraceur>	 19*
[19:00:08] <tgr>	 iirc aarcos added the @property in the latest version
[19:00:24] <marktraceur>	 Hm
[19:00:25] <tgr>	 or maybe this happened at multiple places
[19:00:28] <marktraceur>	 Will re-review then
[19:00:29] <marktraceur>	 Hahaha
[19:00:54] <marktraceur>	 I almost don't want to fix it for the sheer joy it causes me to imagine the brokenness of jsduck
[19:26:39] <grrrit-wm>	 (03PS1) 10Gergő Tisza: Feedback button [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122877 
[19:40:12] <umang13>	 Does any one have knowledge about $wgContLang? what all functions can I call using it? where can I use it 
[20:00:00] <bawolff>	 gi11es: You may have to add me via my "bawolff@gmail.com" email address. Google hangout doesn't like '+' signs in emails
[20:00:49] <gi11es>	 alright
[20:00:51] <marktraceur>	 umang13: I guess it's a Language object
[20:01:13] <marktraceur>	 umang13: 
[20:01:16] <marktraceur>	 https://doc.wikimedia.org/mediawiki-core/master/php/html/classLanguage.html
[20:02:34] <bawolff>	 umang13: You can call it as $wgContLang->truncate
[20:02:40] <bawolff>	 where $wgContLang is a global variable
[20:03:44] <marktraceur>	 umang13: In general you should learn to look through the PHP docs, they're your most helpful reference in navigating mw core
[20:13:15] <umang13>	 bawolff: I tried calling it as $wgContLang->truncate, however it is showing an error saying undefined variable. Are there any prerequisties to use $wgContLang??
[20:13:24] <bawolff>	 global $wgContLang;
[20:13:27] <umang13>	 I tried declaring it as global
[20:13:31] <umang13>	 yeah exactly
[20:13:46] <umang13>	 it says non member accesing member function
[20:13:54] <umang13>	 *accessing
[20:21:20] <umang13>	 Do I need to include any file in my code to use $wgContLang?
[20:21:33] <marktraceur>	 It should be set up automatically in a MediaWiki context
[20:21:50] <marktraceur>	 umang13: Are you still calling this from some random PHP script that's not part of MW?
[20:22:33] <umang13>	 No, The script is a part of MW
[20:22:55] <marktraceur>	 umang13: When is the code you're writing called?
[20:23:08] <marktraceur>	 I assume here that it's a hook, special page, API module, or similar.
[20:23:50] <umang13>	 when I open the page using my browser, that is I removed the code that does not allow invalid entries into mediawiki and used my browser to run the script
[20:24:02] <marktraceur>	 ...no.
[20:24:06] <umang13>	 No, it is not a hook
[20:24:18] <marktraceur>	 umang13: Use a hook, or a special page, or something established.
[20:24:25] <marktraceur>	 Don't Hack Core(tm)
[20:24:53] <umang13>	 Ok, no that was just on my system, just commented out a bit, not advisable?
[20:25:08] <marktraceur>	 No.
[20:25:19] <marktraceur>	 You should be building whatever you're building in the context in which you want to build it
[20:25:26] <marktraceur>	 Not in some hacky context that clearly isn't working.
[20:25:45] <marktraceur>	 Especially since when you totally destroy the assumptions that MediaWiki operates under, we cannot help you at all.
[20:42:15] <bawolff>	 "Are you still calling this from some random PHP script that's not part of MW" <-- wat?
[21:02:15] <marktraceur>	 bawolff: umang13 has been using MediaWiki functions and classes from non-MW PHP scripts for no discernible reason
[21:26:04] <grrrit-wm>	 (03CR) 10MarkTraceur: [C: 04-1] "Bunch of comments, mostly minor, haven't tested" (0324 comments) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos)
[21:52:28] <grrrit-wm>	 (03PS1) 10Ori.livneh: Fix EventLogging schema registry [extensions/UploadWizard] (wmf/1.23wmf19) - 10https://gerrit.wikimedia.org/r/123107 
[21:53:04] <grrrit-wm>	 (03CR) 10Ori.livneh: [C: 032] Fix EventLogging schema registry [extensions/UploadWizard] (wmf/1.23wmf19) - 10https://gerrit.wikimedia.org/r/123107 (owner: 10Ori.livneh)
[21:53:15] <grrrit-wm>	 (03Merged) 10jenkins-bot: Fix EventLogging schema registry [extensions/UploadWizard] (wmf/1.23wmf19) - 10https://gerrit.wikimedia.org/r/123107 (owner: 10Ori.livneh)
[21:58:41] <grrrit-wm>	 (03PS1) 10Gergő Tisza: Make survey open in popup window instead of normal one [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/123109 
[21:58:47] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Make survey open in popup window instead of normal one [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/123109 (owner: 10Gergő Tisza)
[22:00:56] <grrrit-wm>	 (03PS2) 10Gergő Tisza: Feedback button [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122877 
[22:00:58] <grrrit-wm>	 (03PS2) 10Gergő Tisza: Make survey open in popup window instead of normal one [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/123109 
[22:01:00] <grrrit-wm>	 (03PS4) 10Gergő Tisza: Create StripeButtons class, convert reuse button to use it [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/122631 
[22:06:13] <bd808>	 bawolff: Do you have a minute for a code review? https://gerrit.wikimedia.org/r/#/c/121094 GWToolset patch for a bug you reported.
[22:06:22] <bawolff>	 right
[22:06:25] <bawolff>	 I need to do that
[22:07:15] <bd808>	 Dan is hoping to have that and a couple of other patches merged by Thursday for some demo they are doing
[22:07:44] <bawolff>	 I totally meant to look at the patch on the weekend, but didn't get to it
[22:07:47] * bawolff  goes looks at it now

[22:47:16] <tgr>	 https://en.wikipedia.org/w/api.php?action=query&format=jsonfm&meta=filerepoinfo
[22:47:22] <tgr>	 displayname: null
[22:47:30] <tgr>	 is that normal?
[22:47:38] * marktraceur  ...

[22:47:43] <marktraceur>	 No, I think not
[22:47:45] <bawolff>	 Umm, interesting
[22:47:48] <marktraceur>	 That seems t3h bad
[22:48:41] <bawolff>	 Looks like it is normal. Its also like that on my local install
[22:48:57] <marktraceur>	 "also broken locally" !== "normal"
[22:48:59] <bawolff>	 guess the local repo doesn't need to have a name, as there is only one ever
[22:49:06] <aaron_arcos>	 marktraceur: whenever u have a sec can we go over your comments on download, have some questions and it is faster f2f, tx !
[22:49:14] <marktraceur>	 Sure, was going to finish it up
[22:49:20] <marktraceur>	 I failed to get through all the files
[22:49:35] <tgr>	 bawolff: the name is always 'local' for local, i think
[22:49:47] <marktraceur>	 bawolff: I could have sworn we dumped in a human-name for it on local too
[22:49:50] <marktraceur>	 We really should
[22:49:52] <tgr>	 displayname is something that should be displayable to the user
[22:50:21] <tgr>	 iirc this used to work but i am not really sure about it
[22:50:32] <marktraceur>	 Yeah, I thought so too
[22:51:03] <marktraceur>	 You could probably argue it's the same as whatever's in siteinfo?
[22:51:14] <marktraceur>	 But that sounds like a non-argument
[22:51:24] <tgr>	 		if ( $this->isLocal() ) {
[22:51:24] <tgr>	 			return null;
[22:51:24] <tgr>	 		}
[22:51:41] <marktraceur>	 ...sigh
[22:51:51] <tgr>	 so yeah, apparently normal
[22:51:57] <marktraceur>	 Now I've sighed at it it's going to turn out that I wrote it
[22:51:58] <tgr>	 ...for some value of normal
[22:52:29] <tgr>	 Chad, per git blame
[22:52:44] <tgr>	 anyway, i'll fix
[22:52:49] <marktraceur>	 Hah
[22:52:54] <marktraceur>	 What a bastard
[22:52:57] <tgr>	 unless you know a good reason for it to be this way
[22:55:49] <marktraceur>	 I'm asking ^d in -dev
[22:56:04] <marktraceur>	 I don't think there's any good reason for anything in our API
[23:04:24] <grrrit-wm>	 (03PS20) 10Aarcos: Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 
[23:04:27] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos)
[23:18:50] <bawolff>	 Hmm, the code comment on GWToolset "may encode quotes, depending on the php.ini" sounds incredibly scary, but I guess that doesn't matter in this context
[23:40:53] <grrrit-wm>	 (03PS21) 10Aarcos: Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 
[23:42:12] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos)
[23:55:30] <grrrit-wm>	 (03PS22) 10Aarcos: Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 
[23:56:04] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738 (owner: 10Aarcos)
[23:58:05] <grrrit-wm>	 (03PS23) 10Aarcos: Download tab, basic download and preview [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/120738