Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(13)

Issue 2837283002: [Android O] Add SystemNotification.ChannelBlocked histogram (Closed)

Created:
3 years, 7 months ago by awdf
Modified:
3 years, 7 months ago
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3071
Project:
chromium
Visibility:
Public.

Description

[Android O] Add SystemNotification.ChannelBlocked histogram - Added a new histogram for notifications that were already being logged via NotificationUmaTracker. - Now, if notifications are enabled at the app level but blocked at the channel level, they will be counted in this histogram, instead of the SystemNotification.Shown histogram. - If notifications are blocked at the app level they will still be counted in SystemNotification.Blocked, regardless of channel status. TBR=dtrainor@chromium.org BUG=706813 Review-Url: https://codereview.chromium.org/2834633002 Cr-Commit-Position: refs/heads/master@{#466424} (cherry picked from commit d0ba3b6e3f693d2a11992a48dc28508bbfc89731) Review-Url: https://codereview.chromium.org/2837283002 . Cr-Commit-Position: refs/branch-heads/3071@{#197} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} Committed: https://chromium.googlesource.com/chromium/src/+/ab21f85201dc5947c009f97c6d0caf505f2be897

Patch Set 1 #

Messages

Total messages: 6 (5 generated)
awdf
3 years, 7 months ago (2017-04-25 09:50:34 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
ab21f85201dc5947c009f97c6d0caf505f2be897.

Powered by Google App Engine
This is Rietveld 408576698