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

Issue 8883030: Making profile avatars and names sync. (Closed)

Created:
9 years ago by jwd
Modified:
9 years ago
Reviewers:
csilv, sail, akalin
CC:
chromium-reviews, ncarter (slow), Raghu Simha, achuith+watch_chromium.org, rginda+watch_chromium.org, Paweł Hajdan Jr., tim (not reviewing)
Visibility:
Public.

Description

Making profile avatars and names sync. This is done by moving these two properties into profile preferences and making them syncable. The profile observes changes to these prefs and pushes them to the profile info cache. The initialization of these prefs and of the info cache is done in two steps. First, the profile prefs are setup with values generated by the profile info cache. This causes the profile to try to push the changed values to the info cache, but since the profile isn't in the cache, it fails BUG=87658 TEST=ProfileManagerTest.InitProfileUserPrefs, Have two machines with chrome, both with multiple profiles and each with one profile signed in to the same account. Change avatar/name on one, and it will change on the other. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114263

Patch Set 1 #

Patch Set 2 : Adding a Profile Manager unittest #

Unified diffs Side-by-side diffs Delta from patch set Stats (+225 lines, -45 lines) Patch
M chrome/browser/profiles/profile_impl.h View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/browser/profiles/profile_impl.cc View 4 chunks +34 lines, -0 lines 0 comments Download
M chrome/browser/profiles/profile_info_cache.h View 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/browser/profiles/profile_info_cache.cc View 1 chunk +7 lines, -2 lines 0 comments Download
M chrome/browser/profiles/profile_manager.h View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/profiles/profile_manager.cc View 4 chunks +59 lines, -11 lines 0 comments Download
M chrome/browser/profiles/profile_manager_unittest.cc View 1 11 chunks +90 lines, -27 lines 0 comments Download
M chrome/browser/sync/test/integration/sync_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/options/manage_profile_handler.cc View 4 chunks +16 lines, -4 lines 0 comments Download
M chrome/common/pref_names.h View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/common/pref_names.cc View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
jwd
This is a merge of http://codereview.chromium.org/8810001/ http://codereview.chromium.org/8771024/. 8771024 was committed and reverted due to causing ...
9 years ago (2011-12-09 19:38:59 UTC) #1
akalin
On 2011/12/09 19:38:59, Jesse Doherty wrote: > This is a merge of http://codereview.chromium.org/8810001/ > http://codereview.chromium.org/8771024/. ...
9 years ago (2011-12-09 23:57:28 UTC) #2
sail
lgtm
9 years ago (2011-12-12 23:09:08 UTC) #3
csilv
lgtm
9 years ago (2011-12-12 23:24:01 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jwd@chromium.org/8883030/2001
9 years ago (2011-12-13 19:14:49 UTC) #5
commit-bot: I haz the power
9 years ago (2011-12-13 20:39:58 UTC) #6
Change committed as 114263

Powered by Google App Engine
This is Rietveld 408576698