• Evan Stade's avatar
    Remove NotificationService usage from WakeOnWifiManager. · a26aa00f
    Evan Stade authored
    WakeOnWifiConnectObserver seems like a good candidate to be converted
    to a KeyedService, but its lifetime doesn't match up with Profile
    lifetime very well (particularly due to being destroyed early in some
    cases).
    
    It's not necessary to listen for profile destruction because normal
    profiles aren't destroyed until browser shutdown, by which time
    WakeOnWifiManager is gone. OTR profiles aren't observed since they
    are never added to the ProfileManager.
    
    Bug: 268984
    Change-Id: Ie669931e04661363b4080c329e708beeb00ba53c
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1867124
    Commit-Queue: Evan Stade <estade@chromium.org>
    Reviewed-by: default avatarSteven Bennetts <stevenjb@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#707528}
    a26aa00f
wake_on_wifi_manager.cc 8.33 KB