Skip to content

Commit

Permalink
Revert "For mozilla-mobile#17447: add intentional delay to cold start…
Browse files Browse the repository at this point in the history
… up in debug and nightly."

This reverts commit 093470a. This
intentional regression was only intended to be temporary so now we're
removing it.
  • Loading branch information
mcomella authored and csadilek committed Feb 1, 2021
1 parent 10aa0be commit b6a503f
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 14 deletions.
7 changes: 0 additions & 7 deletions app/src/main/java/org/mozilla/fenix/FeatureFlags.kt
Original file line number Diff line number Diff line change
Expand Up @@ -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.
*/
Expand Down
7 changes: 0 additions & 7 deletions app/src/main/java/org/mozilla/fenix/FenixApplication.kt
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand Down

0 comments on commit b6a503f

Please sign in to comment.