From 7e065ec3d1bbf91720f3c7aa1d1edddaba40bc08 Mon Sep 17 00:00:00 2001 From: Thomas Rientjes <synzvato@protonmail.com> Date: Tue, 5 Mar 2019 01:00:38 +0100 Subject: [PATCH] Refactor existing codebase --- core/main.js | 6 ++---- core/state-manager.js | 5 +++++ 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/core/main.js b/core/main.js index cfe26c2..31797b1 100644 --- a/core/main.js +++ b/core/main.js @@ -42,9 +42,9 @@ main._initializeOptions = function () { } if (options.blockMissing === true) { - - // Improve coverage at the cost of website stability. stateManager.setExtensionEnvironment('staging'); + } else { + stateManager.setExtensionEnvironment('stable'); } if (options.disablePrefetch !== false) { @@ -96,8 +96,6 @@ main._showReleaseNotes = function (details) { */ chrome.runtime.onInstalled.addListener(main._showReleaseNotes); -stateManager.setExtensionEnvironment('stable'); - main._initializeOptions(); wrappers.setBadgeBackgroundColor({ diff --git a/core/state-manager.js b/core/state-manager.js index c483e87..40c6405 100644 --- a/core/state-manager.js +++ b/core/state-manager.js @@ -75,8 +75,13 @@ stateManager.setExtensionEnvironment = function (environment) { chrome.storage.local.get(Setting.ENFORCE_STAGING, function (items) { if (environment === 'staging' || items.enforceStaging === true) { + + // Improve coverage at the expense of website stability. files.active = Object.assign({}, files.stable, files.staging); + } else { + + // Find a balance between coverage and website stability. files.active = files.stable; } -- GitLab