• Xing Liu's avatar
    Announcement notification: Handle case where Finch is not ready. · b88ff2d3
    Xing Liu authored
    Finch parameter is not guaranteed to be ready on first run. This may
    introduce unexpected behaviors, like the user see the notification on
    second time running Chrome.
    
    Now we always persist the first run timestamp and uses a Finch
    specified timestamp from the server to check if first run happens
    recently. If so then just skip the notification.
    
    Also address an issue that origin checks for
    NotificationPlatformBridgeMac is removed, now we let the empty GURL
    pass.
    
    Bug: 1047286,1046457
    Change-Id: I59723c7d35230345ca9d77ccc810a07646d34b0b
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2032346
    Commit-Queue: Xing Liu <xingliu@chromium.org>
    Reviewed-by: default avatarTommy Nyquist <nyquist@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#737193}
    b88ff2d3
announcement_notification_service.h 3.62 KB