From b1e41cf71861494f8da2580916808fcdd39c8aa7 Mon Sep 17 00:00:00 2001 From: "Ori.livneh" Date: Thu, 23 Jan 2014 00:25:02 +0000 Subject: [PATCH] Re-remove experiment code for moduleStorage The second iteration of the experiment is done, so this can once again be removed. This patch reverts commit Ifa97d36d3. Change-Id: If85b392d3e29de57316d32ba81c9b172bef758dd --- resources/mediawiki/mediawiki.js | 35 +------------------------------- 1 file changed, 1 insertion(+), 34 deletions(-) diff --git a/resources/mediawiki/mediawiki.js b/resources/mediawiki/mediawiki.js index 5623575294..f3bb414703 100644 --- a/resources/mediawiki/mediawiki.js +++ b/resources/mediawiki/mediawiki.js @@ -1815,38 +1815,6 @@ var mw = ( function ( $, undefined ) { // Cache hit stats stats: { hits: 0, misses: 0, expired: 0 }, - // Experiment data - experiment: ( function () { - var start = ( new Date() ).getTime(), id = 0, seed = 0; - - try { - id = JSON.parse( localStorage.getItem( 'moduleStorageExperiment2' ) ); - if ( typeof id !== 'number' ) { - id = Math.floor( Math.random() * Math.random() * 1e16 ); - localStorage.setItem( 'moduleStorageExperiment2', id ); - } - seed = id % 2000; - } catch ( e ) {} - - return { - // Unique identifier for this browser. This allows us to group all - // datapoints generated by a particular browser, which in turn allows us - // to see how the initial load compares to subsequent page loads. - id: id, - - // Group assignment may be 0 (not in experiment), 1 (control group), or 2 - // (experimental group). Browsers that don't implement all the prerequisite APIs - // (JSON and Web Storage) are ineligible. Eligible browsers have a 0.1% chance - // of being included in the experiment, in which case they are equally likely to - // be assigned to either the experimental or control group. - group: seed === 1 ? 1 : ( seed === 2 ? 2 : 0 ), - - // Assess module storage performance by measuring the time between this - // reference point and the window load event. - start: start - }; - }() ), - /** * Construct a JSON-serializable object representing the content of the store. * @return {Object} Module store contents. @@ -1907,8 +1875,7 @@ var mw = ( function ( $, undefined ) { return; } - if ( ( !mw.config.get( 'wgResourceLoaderStorageEnabled' ) && mw.loader.store.experiment.group !== 2 ) - || mw.config.get( 'debug' ) ) { + if ( !mw.config.get( 'wgResourceLoaderStorageEnabled' ) || mw.config.get( 'debug' ) ) { // Disabled by configuration, or because debug mode is set mw.loader.store.enabled = false; return; -- 2.20.1