• Tim Schumann's avatar
    Revert "[Nearby] Trigger onboarding when first enabling Nearby" · 61525561
    Tim Schumann authored
    This reverts commit a5d3ad0f.
    
    Reason for revert: Suspected to break NearbyOnboardingPageTest.All on linux-chromeos-dbg
    
    Original change's description:
    > [Nearby] Trigger onboarding when first enabling Nearby
    > 
    > This CL adds a pref to track if onboarding was run by this user before.
    > If it has not, when enabling for the first time, onboarding is run first
    > by navigating to the subpage and showing in a dialog. Once onboarding
    > has been done once, the user can freely toggle on/off without
    > triggering onboarding. However, onboarding will still run if the user
    > triggers the pod button and the feature is disabled. This matches the
    > Android behavior.
    > 
    > Additionally, this CL fixes a bug with the premature closing of the
    > shared receiveManager which needs to stay alive and connected after the
    > dialog closes otherwise getReceiveManager returns a remote with a closed
    > pipe.
    > 
    > Bug: b/154866951
    > Change-Id: I1e3cbd15f4995f019b3476c2f7a69f67d6eb1888
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424763
    > Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
    > Commit-Queue: James Vecore <vecore@google.com>
    > Cr-Commit-Position: refs/heads/master@{#809623}
    
    TBR=khorimoto@chromium.org,vecore@google.com
    
    Change-Id: I4bd779c4a17b7ef0560078cefd00287810dab796
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: b/154866951, 1131390
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2424160Reviewed-by: default avatarTim Schumann <tschumann@chromium.org>
    Commit-Queue: Tim Schumann <tschumann@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#809720}
    61525561
nearby_share_prefs.cc 5.16 KB