• Evan Stade's avatar
    Remove some instances of NOTIFICATION_SESSION_STARTED · 3e80dd3c
    Evan Stade authored
    This patch changes SessionManagerObserver::OnPrimaryUserSessionStarted
    to notify for non-primary user sessions as well. This makes it align
    with NOTIFICATION_SESSION_STARTED.
    
    Two notification observers are updated:
    - LocaleChangeGuard
    - first_run.cc (some dead code is also deleted)
    
    On[Primary]UserSessionStarted(), and NOTIFICATION_SESSION_STARTED,
    are different from UserSessionStateObserver::OnActiveUserChanged
    in that they are called when the UI is fully ready. This is appropriate
    for the two observers that are updated in this CL, both of which want
    to show UI right after login.
    
    Bug: 268984
    
    Change-Id: Iac5ba9541e99a74a29758db6e3c678e1dcbfa630
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1767494Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
    Commit-Queue: Evan Stade <estade@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#690074}
    3e80dd3c
locale_change_guard.cc 7.55 KB