Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(152)

Issue 2655553002: [Merge M57] Fix import legacy, pref based pins. (Closed)

Created:
3 years, 11 months ago by khmel
Modified:
3 years, 11 months ago
Reviewers:
stevenjb
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2987
Project:
chromium
Visibility:
Public.

Description

[Merge M57] Fix import legacy, pref based pins. This CL contains 2 fixes for importing legacy pref based pins. 1. Import may happen at the moment when profile is not synced yet and in last case it does not actually contain information from previous bulds. 2. Importing ignores apps that currently are not installed. This leads to situation that we loose pin information for such apps and in case apps are installed later they do not appear pinned. Solution is to import all existings apps in prefs. BUG=680821 TEST=Clearn app sync server data. Login on M52 and do some pinning. Next login on M55 and pin additional app. Next login to ToT and observe that pin is restored for existing apps and pins appear automatically once apps are installed later. TBR=stevenjb@chromium.org NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2646973003 Cr-Commit-Position: refs/heads/master@{#445398} (cherry picked from commit d8cc9d23b26dc31e050fdc1e2042f639d8a608fa) Review-Url: https://codereview.chromium.org/2655553002 Cr-Commit-Position: refs/branch-heads/2987@{#51} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} Committed: https://chromium.googlesource.com/chromium/src/+/932919878abdfe3d0e0da83da1125726425949c3

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+135 lines, -32 lines) Patch
M chrome/browser/ui/ash/chrome_launcher_prefs.cc View 10 chunks +35 lines, -20 lines 0 comments Download
M chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.h View 3 chunks +6 lines, -1 line 0 comments Download
M chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc View 5 chunks +10 lines, -0 lines 0 comments Download
M chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl_unittest.cc View 9 chunks +84 lines, -11 lines 0 comments Download

Messages

Total messages: 5 (3 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2655553002/1
3 years, 11 months ago (2017-01-24 01:35:28 UTC) #2
commit-bot: I haz the power
3 years, 11 months ago (2017-01-24 01:50:18 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/932919878abdfe3d0e0da83da112...

Powered by Google App Engine
This is Rietveld 408576698