• Marc Treib's avatar
    Split histogram FCMInvalidations.FCMMessageStatus by sender_id · 4d290dd7
    Marc Treib authored
    This CL adds suffixes Sync, Drive, and Policy to the existing histogram
    FCMInvalidations.FCMMessageStatus. Having all the different senders in
    a single histogram made analysis tricky.
    
    Since the sender_id constants aren't accessible at the place where the
    histogram is recorded, they're duplicated. This should be okay since
    existing sender_ids are not expected to change, and new ones are added
    infrequently.
    
    While I'm here, also add myself as an owner of other FCMInvalidations
    histograms, since the original owner is OOO.
    
    Bug: 1023813
    Change-Id: I41d24f4d61c22641eb0e599176fcfff45f2740c8
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1972841
    Commit-Queue: Marc Treib <treib@chromium.org>
    Auto-Submit: Marc Treib <treib@chromium.org>
    Reviewed-by: default avatarAlex Ilin <alexilin@chromium.org>
    Reviewed-by: default avatarMikel Astiz <mastiz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#726419}
    4d290dd7
fcm_network_handler.cc 13.1 KB