resourceloader: Sync order of startup test cases with actual code

[lhc/web/wiklou.git] / resources / src / startup / startup.js
diff --git a/resources/src/startup/startup.js b/resources/src/startup/startup.js

index bebf4dc..f2d964a 100644 (file)
--- a/resources/src/startup/startup.js
+++ b/resources/src/startup/startup.js
@@ -44,11 +44,11 @@
   *
   * Other browsers that pass the check are considered Grade X.
   *
- * @param {string} [str] User agent, defaults to navigator.userAgent
+ * @private
+ * @param {string} ua User agent string
   * @return {boolean} User agent is compatible with MediaWiki JS
   */
-function isCompatible( str ) {
-       var ua = str || navigator.userAgent;
+function isCompatible( ua ) {
         return !!(
                 // https://caniuse.com/#feat=es5
                 // https://caniuse.com/#feat=use-strict
@@ -69,14 +69,16 @@ function isCompatible( str ) {
                 // https://caniuse.com/#feat=addeventlistener
                 'addEventListener' in window &&
  
-               // Hardcoded exceptions for browsers that pass the requirement but we don't want to
-               // support in the modern run-time.
-               // Note: Please extend the regex instead of adding new ones
+               // Hardcoded exceptions for browsers that pass the requirement but we don't
+               // want to support in the modern run-time.
+               //
+               // Please extend the regex instead of adding new ones!
+               // And add a test case to startup.test.js
                 !ua.match( /MSIE 10|webOS\/1\.[0-4]|SymbianOS|Series60|NetFront|Opera Mini|S40OviBrowser|MeeGo|Android.+Glass|^Mozilla\/5\.0 .+ Gecko\/$|googleweblight|PLAYSTATION|PlayStation/ )
         );
  }
  
-if ( !isCompatible() ) {
+if ( !isCompatible( navigator.userAgent ) ) {
         // Handle Grade C
         // Undo speculative Grade A <html> class. See ResourceLoaderClientHtml::getDocumentAttributes().
         document.documentElement.className = document.documentElement.className