From 74ac91a932a119b21db2e7aae96169e1322d1b8a Mon Sep 17 00:00:00 2001 From: Howard Ro Date: Sun, 7 Apr 2019 17:34:47 -0700 Subject: [PATCH] More description on logBinaryPushStateChanged Provides more descriptive documentation for 'options' and 'state' parameters when calling StatsLog#logBinaryPushStateChanged Bug: 129768469 Fix: 129768469 Test: n/a Change-Id: I4b06cc2618258e8e24a26c4616658ce8f146e133 --- core/java/android/util/StatsLog.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/core/java/android/util/StatsLog.java b/core/java/android/util/StatsLog.java index 30d3d7d069b5d..dd22a26d61afa 100644 --- a/core/java/android/util/StatsLog.java +++ b/core/java/android/util/StatsLog.java @@ -136,7 +136,13 @@ public final class StatsLog extends StatsLogInternal { * @param trainName name of install train. * @param trainVersionCode version code of the train. * @param options optional flags about this install. - * @param state current install state. + * The last 3 bits indicate options: + * 0x01: FLAG_REQUIRE_STAGING + * 0x02: FLAG_ROLLBACK_ENABLED + * 0x04: FLAG_REQUIRE_LOW_LATENCY_MONITOR + * @param state current install state. Defined as State enums in + * BinaryPushStateChanged atom in + * frameworks/base/cmds/statsd/src/atoms.proto * @param experimentIds experiment ids. * @return True if the log request was sent to statsd. */