blallo
/
fenix
1
0
Fork 0
Code Releases Activity

No issue: extract startMetricsIfEnabled function. 

This refactor, done entirely by IDE, is a no-op clean-up of this file.
master
Michael Comella 2020-07-21 13:33:46 -07:00 committed by Michael Comella
parent 11e2137fab
commit 169071ac61
1 changed files with 11 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
app/src/main/java/org/mozilla/fenix/FenixApplication.kt
View File

@ -139,14 +139,7 @@ open class FenixApplication : LocaleAwareApplication() {
prefetchForHomeFragment()
setupLeakCanary()
if (settings().isTelemetryEnabled) {
components.analytics.metrics.start(MetricServiceType.Data)
}
if (settings().isMarketingTelemetryEnabled) {
components.analytics.metrics.start(MetricServiceType.Marketing)
}
startMetricsIfEnabled()
setupPush()
visibilityLifecycleCallback = VisibilityLifecycleCallback(getSystemService())
@ -203,6 +196,16 @@ open class FenixApplication : LocaleAwareApplication() {
}
}
private fun startMetricsIfEnabled() {
if (settings().isTelemetryEnabled) {
components.analytics.metrics.start(MetricServiceType.Data)
}
if (settings().isMarketingTelemetryEnabled) {
components.analytics.metrics.start(MetricServiceType.Marketing)
}
}
// See https://github.com/mozilla-mobile/fenix/issues/7227 for context.
// To re-enable this, we need to do so in a way that won't interfere with any startup operations
// which acquire reserved+ sqlite lock. Currently, Fennec migrations need to write to storage