From 1240a62ca0b7b782b69852c71b30a2a4e0ef8d81 Mon Sep 17 00:00:00 2001 From: Timo Tijhof Date: Thu, 15 Sep 2016 14:17:20 -0700 Subject: [PATCH] geoclient: Update localStorage comment Follows-up 4cd1def. localStorage is available by all supported browsers as of MediaWiki 1.25. As of MediaWiki 1.27, this is also enforced by an explicit feature test in startup.js. Change-Id: I12b577919579f498e294376f748715b868f21869 --- resources/js/ext.uls.geoclient.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/resources/js/ext.uls.geoclient.js b/resources/js/ext.uls.geoclient.js index 52937dee..2d07ceca 100644 --- a/resources/js/ext.uls.geoclient.js +++ b/resources/js/ext.uls.geoclient.js @@ -29,10 +29,9 @@ return; } - // Using cache for speed and to be nice for the service. Using cookies over - // localstorage because cookies support automatic expiring and are supported - // everywhere where as localstorage might not. This cookie is not currently - // read server side. + // Using cache for speed and to reduce load on the third-party service. + // Using cookies over localStorage because cookies support automatic expiring. + // This cookie is not currently read server-side. geo = mw.cookie.get( 'ULSGeo' ); if ( geo ) { try {