Fix the the proto definition for app startup and fully drawn events.

ActivityMetricsLoggers build the proto with the compilation reason first
and then the compilation filter. Instead of changing the logging code,
we prefer to change the proto field order: this will facilitate the
speed up the resolution without the need to purge data or wait for
further release builds.

It is ok, because both fields are in fact int32 proto values so the ABI
does not change.

Test: m
Bug: 178108751

(cherry picked from commit cf676e8028c2611461dc3e009dd9fe771f44bdb2)

Merged-In: I36ec354d273dceace5245d7634c68ac63dc5a8cd
Change-Id: I03c701efe2181ae7796547267902f884007593eb
This commit is contained in:
Calin Juravle
2021-02-02 17:07:45 -08:00
parent 9d3b7aacd9
commit 6b232cc058

View File

@@ -3612,11 +3612,11 @@ message AppStartOccurred {
// Empty if not set.
optional string launch_token = 13;
// The compiler filter used when when the package was optimized.
optional int32 package_optimization_compilation_filter = 14;
// The reason why the package was optimized.
optional int32 package_optimization_compilation_reason = 15;
optional int32 package_optimization_compilation_reason = 14;
// The compiler filter used when when the package was optimized.
optional int32 package_optimization_compilation_filter = 15;
}
message AppStartCanceled {
@@ -3662,11 +3662,11 @@ message AppStartFullyDrawn {
// App startup time (until call to Activity#reportFullyDrawn()).
optional int64 app_startup_time_millis = 6;
// The compiler filter used when when the package was optimized.
optional int32 package_optimization_compilation_filter = 7;
// The reason why the package was optimized.
optional int32 package_optimization_compilation_reason = 8;
optional int32 package_optimization_compilation_reason = 7;
// The compiler filter used when when the package was optimized.
optional int32 package_optimization_compilation_filter = 8;
}
/**