For #8803 - review: Add clarifying comments to onAppInit capture methods.
parent
2888d66ce0
commit
f3ed207bdb
|
@ -374,6 +374,8 @@ open class FenixApplication : LocaleAwareApplication() {
|
|||
protected fun recordOnInit() {
|
||||
// This gets called by more than one process. Ideally we'd only run this in the main process
|
||||
// but the code to check which process we're in crashes because the Context isn't valid yet.
|
||||
StartupTimeline.onApplicationInit()
|
||||
//
|
||||
// This method is not covered by our internal crash reporting: be very careful when modifying it.
|
||||
StartupTimeline.onApplicationInit() // DO NOT MOVE ANYTHING ABOVE HERE: the timing is critical.
|
||||
}
|
||||
}
|
||||
|
|
|
@ -46,6 +46,8 @@ object StartupTimeline {
|
|||
|
||||
fun onApplicationInit() {
|
||||
// This gets called from multiple processes: don't do anything expensive. See call site for details.
|
||||
//
|
||||
// This method also gets called multiple times if there are multiple Application implementations.
|
||||
frameworkStartMeasurement.onApplicationInit()
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue