• Tim Sergeant's avatar
    Reland "CrOS: Add Files app to default shelf pins in second position" · 527ae681
    Tim Sergeant authored
    This is a reland of 5d6ecbfb
    
    The failing Tast test (ui.ShelfLaunchedApps) has been updated to
    ensure it continues passing after this CL is landed.
    
    This is the second of a chain of 3 CLs:
    
    1. crrev.com/c/2538305: Allow Files app to be pinned by default in
       ui.ShelfLaunchedApps
    2. crrev.com/c/2538857: Add Files app to default shelf pins in second
       position
    3. crrev.com/c/2538307: Update ui.ShelfLaunchedApps to ensure Chrome and
       Files are always pinned
    
    Original change's description:
    > CrOS: Add Files app to default shelf pins in second position
    >
    > This adds the Files app to the spot after the Chrome icon, for new users
    > only. There's also some variations of the default pin layout controlled
    > by a Finch experiment, I've moved the Files app up in those layouts for
    > consistency.
    >
    > This change was requested for M88 and will be merged back.
    >
    > Bug: 1148519
    > Change-Id: Iae54561fec599b813ba97e82a5678fa834a02db2
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2537214
    > Reviewed-by: Dominick Ng <dominickn@chromium.org>
    > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
    > Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
    > Auto-Submit: Tim Sergeant <tsergeant@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#827272}
    
    Bug: 1148519
    Change-Id: I9186b43a651dea4c4de9e46f849d35da9cc6062c
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2538857Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
    Commit-Queue: Tim Sergeant <tsergeant@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#827981}
    527ae681
chrome_launcher_prefs.cc 27.3 KB