• Ravi Chandra Sadineni's avatar
    cros: Remove notification reporter. · b63885fd
    Ravi Chandra Sadineni authored
    Notification reporter intends to treat notification as user activity for
    screen dim and idle timeouts. Thus it notifies powerd whenever a new
    notification (priority > DEFAULT) is drawn or the one that is already
    shown is updated.
    
    When a notification of system priority is drawn, chrome keeps updating
    the notification until is cleared by user which results in
    regular HandleWakeNotification calls to powerd. Thus this can result in
    screen staying on forever. Let us not do this.
    
    In the follow up CL, I intend to notify powerd directly where the
    notification is created. This way we do not use notification priority
    that is already overloaded to determine if the screen should stay on.
    
    BUG=chromium:964470
    TEST=Connect external monitor and make sure screen turns off.
    
    Change-Id: I616926994f66982ecb5886f17af85813ee34efe4
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1618047
    Commit-Queue: Ravi Chandra Sadineni <ravisadineni@chromium.org>
    Reviewed-by: default avatarDan Erat <derat@chromium.org>
    Auto-Submit: Ravi Chandra Sadineni <ravisadineni@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#661116}
    b63885fd
BUILD.gn 89.9 KB