From ba170fe1533900fa334792acebaca65a89b27371 Mon Sep 17 00:00:00 2001 From: Michael Comella Date: Thu, 28 Jan 2021 16:07:26 -0800 Subject: [PATCH] [fenix] Revert "For https://github.com/mozilla-mobile/fenix/issues/17447: add intentional delay to cold start up in debug and nightly." This reverts commit 3a5e6924da81500265c31e00d4499a50d1c4b003. This intentional regression was only intended to be temporary so now we're removing it. --- app/src/main/java/org/mozilla/fenix/FeatureFlags.kt | 7 ------- app/src/main/java/org/mozilla/fenix/FenixApplication.kt | 7 ------- 2 files changed, 14 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/FeatureFlags.kt b/app/src/main/java/org/mozilla/fenix/FeatureFlags.kt index 170520d853..2aedb9361c 100644 --- a/app/src/main/java/org/mozilla/fenix/FeatureFlags.kt +++ b/app/src/main/java/org/mozilla/fenix/FeatureFlags.kt @@ -34,13 +34,6 @@ object FeatureFlags { // users are still experiencing crashes. const val nimbusExperiments = false - /** - * Enables an intentional regression to validate perftest alerting. See - * https://github.com/mozilla-mobile/fenix/issues/17447 for details. This - * is expected to be removed within several days. - */ - val intentionalRegressionToValidatePerfTestAlerting = Config.channel.isNightlyOrDebug - /** * Enables the new MediaSession API. */ diff --git a/app/src/main/java/org/mozilla/fenix/FenixApplication.kt b/app/src/main/java/org/mozilla/fenix/FenixApplication.kt index 88d9506297..fdcbad404a 100644 --- a/app/src/main/java/org/mozilla/fenix/FenixApplication.kt +++ b/app/src/main/java/org/mozilla/fenix/FenixApplication.kt @@ -119,13 +119,6 @@ open class FenixApplication : LocaleAwareApplication(), Provider { @CallSuper open fun setupInMainProcessOnly() { - // See feature flags kdoc for details. - if (FeatureFlags.intentionalRegressionToValidatePerfTestAlerting) { - logger.info("Intentional thread sleep. See #17447") - @Suppress("MagicNumber") // it's fine for a quick patch. - Thread.sleep(100) - } - run { // Attention: Do not invoke any code from a-s in this scope. val megazordSetup = setupMegazord()