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

Issue 186803003: Use custom profile names (if available) for signed in profiles (Closed)

Created:
6 years, 9 months ago by noms (inactive)
Modified:
6 years, 9 months ago
Reviewers:
rpetterson
CC:
chromium-reviews, tim+watch_chromium.org, haitaol+watch_chromium.org, maniscalco+watch_chromium.org
Visibility:
Public.

Description

Use custom profile names (if available) for signed in profiles The problem was that if the user was set to use the Gaia profile info, then we ignored any custom profile names that were synced. We shouldn't do that :) BUG=343730, 348187, 349000 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255273

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+97 lines, -34 lines) Patch
M chrome/browser/profiles/gaia_info_update_service_unittest.cc View 2 chunks +9 lines, -2 lines 0 comments Download
M chrome/browser/profiles/profile_info_cache.h View 2 chunks +2 lines, -0 lines 0 comments Download
M chrome/browser/profiles/profile_info_cache.cc View 8 chunks +50 lines, -2 lines 0 comments Download
M chrome/browser/profiles/profile_info_cache_unittest.cc View 1 chunk +30 lines, -21 lines 0 comments Download
M chrome/browser/profiles/profile_info_interface.h View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/browser/profiles/profiles_state.cc View 2 chunks +1 line, -9 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
noms (inactive)
Hi Rachel, I've moved the "check if the profile is using a default name" code ...
6 years, 9 months ago (2014-03-05 14:52:24 UTC) #1
rpetterson
lgtm
6 years, 9 months ago (2014-03-05 22:10:44 UTC) #2
noms (inactive)
Thanks!
6 years, 9 months ago (2014-03-05 22:13:34 UTC) #3
noms (inactive)
The CQ bit was checked by noms@chromium.org
6 years, 9 months ago (2014-03-05 22:13:48 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/noms@chromium.org/186803003/60001
6 years, 9 months ago (2014-03-05 22:15:49 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/noms@chromium.org/186803003/60001
6 years, 9 months ago (2014-03-06 00:05:43 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/noms@chromium.org/186803003/60001
6 years, 9 months ago (2014-03-06 02:49:48 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/noms@chromium.org/186803003/60001
6 years, 9 months ago (2014-03-06 03:14:11 UTC) #8
commit-bot: I haz the power
6 years, 9 months ago (2014-03-06 06:00:39 UTC) #9
Message was sent while issue was closed.
Change committed as 255273

Powered by Google App Engine
This is Rietveld 408576698