4 commits

Author	SHA1	Message	Date
Stephen Niedzielski	ece4670710	Hygiene: use arrow for anonymous functions ... In some places, the arrow function seems more natural. This patch approximates the following with manual amendments: find \ -not \( \( -name node_modules -o -name .git -o -name vendor -o -name doc -o -name resources \) -prune \) \ -iname \*.js| xargs -rd\\n sed -ri 's%function\s*(\([^)]*\))%\1 =>%g' Files to focus on were identified with: rg -slg\!/resources/dist/ -g\!/i18n/ -g\!/doc/ 'this|self|arguments|bind|call|apply|new'| xargs -rd\\n git difftool -y Bug: T165036 Change-Id: Ic66b6000b8fc000f9bfde39749f9cfa69924a13c	2018-03-20 09:27:08 -05:00
jdlrobson	695a7b6b58	QA: Cleanup unnecessary test changes ... Various tweaks were made in Ibe212721807d3698dc45ef46b2dbde15ca9d2f70 to get the browser tests to pass. Many of them were unnecessary. Change-Id: I47ac37512da38a33f4b3b919cc412b79231b0324	2018-03-16 09:53:04 -07:00
jdlrobson	deaaf0961b	Remove popups from critical rendering path ... Instead load it via mw.loader.using We retain the module name ext.popups as this will be present in cached HTML, however now it will load the bulk of the code inside ext.popups.main Bug: T176211 Change-Id: Ibe212721807d3698dc45ef46b2dbde15ca9d2f70	2018-03-13 08:44:31 -07:00
jdlrobson	cf774be88b	Port Popups browser tests to selenium ... Bug: T177262 Change-Id: Ie77cf7bc79f454904983ed5cf65dc9fe2b1ececc	2017-11-01 13:31:36 -07:00