From 2ad11ee6527ca30819e4136b5d1906839808b3be Mon Sep 17 00:00:00 2001 From: Timo Tijhof Date: Sun, 24 Aug 2014 16:04:29 +0200 Subject: [PATCH] startup: Update documentation * Update links to mediawiki.org and jquery.com. * Remove irrelevant link to jquerymobile.com. * Fix documentation stating the isCompatible() function is deleted after use, because it isn't (and never was). The code pretending to delete it was removed in d0259d497d7e76d. Change-Id: I4521521210778d9eea3959afcd62647410958305 --- resources/src/startup.js | 11 +++++------ tests/qunit/suites/resources/startup.test.js | 5 +---- 2 files changed, 6 insertions(+), 10 deletions(-) diff --git a/resources/src/startup.js b/resources/src/startup.js index 01efef4aab..56bb3d3c10 100644 --- a/resources/src/startup.js +++ b/resources/src/startup.js @@ -7,15 +7,14 @@ var mediaWikiLoadStart = ( new Date() ).getTime(); /** - * Returns false when run in a black-listed browser + * Returns false for Grade C supported browsers. * - * This function will be deleted after it's used, so do not expand it to be - * generally useful beyond startup. + * This function should only be used by the Startup module, do not expand it to + * be generally useful beyond startup. * * See also: - * - https://www.mediawiki.org/wiki/Compatibility#Browser - * - http://jquerymobile.com/gbs/ - * - http://jquery.com/browser-support/ + * - https://www.mediawiki.org/wiki/Compatibility#Browsers + * - https://jquery.com/browser-support/ */ /*jshint unused: false */ diff --git a/tests/qunit/suites/resources/startup.test.js b/tests/qunit/suites/resources/startup.test.js index c3b9b3d70f..7dec626507 100644 --- a/tests/qunit/suites/resources/startup.test.js +++ b/tests/qunit/suites/resources/startup.test.js @@ -1,7 +1,6 @@ /*global isCompatible: true */ ( function ( $ ) { var testcases = { - // Supported: Compatible gradeA: [ // Chrome 'Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_7; en-US) AppleWebKit/534.16 (KHTML, like Gecko) Chrome/10.0.648.205 Safari/534.16', @@ -43,7 +42,6 @@ // Android 'Mozilla/5.0 (Linux; U; Android 2.1; en-us; Nexus One Build/ERD62) AppleWebKit/530.17 (KHTML, like Gecko) Version/4.0 Mobile Safari/530.17' ], - // Supported: Uncompatible, serve basic content gradeC: [ // Internet Explorer < 7 'Mozilla/2.0 (compatible; MSIE 3.03; Windows 3.1)', @@ -90,8 +88,7 @@ // Google Glass 'Mozilla/5.0 (Linux; U; Android 4.0.4; en-us; Glass 1 Build/IMM76L; XE11) AppleWebKit/534.30 (KHTML, like Gecko) Version/4.0 Mobile Safari/534.30' ], - // No explicit support for or against these browsers, they're - // given a shot at Grade A at their own risk. + // No explicit support for or against these browsers, they're given a shot at Grade A. gradeX: [ // Firefox 3.6 'Mozilla/5.0 (Windows; U; Windows NT 6.1; ru; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3', -- 2.20.1