• Kyle Horimoto's avatar
    [CrOS PhoneHub] Make notifications pop up when content is updated · 7fbc2c95
    Kyle Horimoto authored
    Before this CL, we'd show a pop-up notification for any new
    notifications (corresponding to new notification IDs), but we'd silently
    update the notification content whenever it was updated.
    
    However, many messaging apps use the same notification ID for a single
    chat thread and just replace the text content with new content when a
    new message is received. This meant that when users received a new
    message from the same recipient, the new notification would not pop up.
    
    This CL fixes the issue by always popping up updated notifications.
    
    Fixed: 1159063
    Bug: 1106937
    Change-Id: I21aa76368d8de63068a12ad23c65fd4ecc4d3a4d
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2606109Reviewed-by: default avatarRegan Hsu <hsuregan@chromium.org>
    Commit-Queue: Kyle Horimoto <khorimoto@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#839759}
    7fbc2c95
phone_hub_notification_controller.h 4.48 KB