Commit 75b8b93a authored by Xing Liu's avatar Xing Liu Committed by Commit Bot

Fix announcement notification Android metrics.

We use auto generated Java enum based on a c++ enum. However the Java
API RecordHistogram.recordEnumeratedHistogram() need to pass a maximum
value +1 to the auto generated value.

TBR=dtrainor@chromium.org,isherman@chromium.org

Bug: 1054957
Change-Id: Ib6a8d9695fceab7bffaef4b765277a6a4f5d69f2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068754Reviewed-by: default avatarXing Liu <xingliu@chromium.org>
Commit-Queue: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#744113}
parent 5c735be6
...@@ -119,7 +119,7 @@ public class AnnouncementNotificationManager { ...@@ -119,7 +119,7 @@ public class AnnouncementNotificationManager {
private static void recordHistogram(@AnnouncementNotificationEvent int event) { private static void recordHistogram(@AnnouncementNotificationEvent int event) {
RecordHistogram.recordEnumeratedHistogram("Notifications.Announcement.Events", event, RecordHistogram.recordEnumeratedHistogram("Notifications.Announcement.Events", event,
org.chromium.chrome.browser.announcement.AnnouncementNotificationEvent.MAX_VALUE); AnnouncementNotificationEvent.MAX_VALUE + 1);
} }
@CalledByNative @CalledByNative
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment