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

Issue 556003007: [Profiles] Improve the display name for default profiles. (Closed)

Created:
6 years, 3 months ago by noms (inactive)
Modified:
6 years, 3 months ago
CC:
chromium-reviews, rginda+watch_chromium.org, yoshiki+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

[Profiles] Improve the display name for default profiles. If a profile is signed in, but has a default name (because this is after enabling the new profiles UI but the Gaia profile name hasn't been downloaded) display the email address rather than the default profile name. It's one step better. BUG=412809 TEST=Start Chrome with an empty user data dir and --disable-new-avatar-menu. Create two profiles, and sign one of them in with a profile that has a clean sync (or reset the sync data after signin). Make sure that the last active profile is the *not* signed in one (so close all the browser windows for the signed in profile). Restart Chrome with --enable-new-avatar-menu. You should be in the local profile. Choose "Switch person" from the avatar menu. The user manager should display the other profile's email address in the pod. Once you launch that profile, the name should update to the gaia name. Committed: https://crrev.com/c929946b5b10c2cbaadfa49cd713da3c4e439ac5 Cr-Commit-Position: refs/heads/master@{#295166}

Patch Set 1 #

Total comments: 2

Patch Set 2 : fix test + typo #

Unified diffs Side-by-side diffs Delta from patch set Stats (+73 lines, -6 lines) Patch
M chrome/browser/profiles/profile_manager_unittest.cc View 1 1 chunk +60 lines, -0 lines 0 comments Download
M chrome/browser/profiles/profiles_state.cc View 1 chunk +8 lines, -6 lines 0 comments Download
M chrome/browser/ui/cocoa/profiles/profile_chooser_controller_unittest.mm View 1 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 10 (3 generated)
noms (inactive)
Moar better default profile names! :)
6 years, 3 months ago (2014-09-15 20:02:49 UTC) #2
rpetterson
lgtm https://codereview.chromium.org/556003007/diff/1/chrome/browser/profiles/profile_manager_unittest.cc File chrome/browser/profiles/profile_manager_unittest.cc (right): https://codereview.chromium.org/556003007/diff/1/chrome/browser/profiles/profile_manager_unittest.cc#newcode1233 chrome/browser/profiles/profile_manager_unittest.cc:1233: // Adding a Gaia name to a profile ...
6 years, 3 months ago (2014-09-16 19:05:02 UTC) #3
noms (inactive)
+ alexei for the cocoa unit test. https://codereview.chromium.org/556003007/diff/1/chrome/browser/profiles/profile_manager_unittest.cc File chrome/browser/profiles/profile_manager_unittest.cc (right): https://codereview.chromium.org/556003007/diff/1/chrome/browser/profiles/profile_manager_unittest.cc#newcode1233 chrome/browser/profiles/profile_manager_unittest.cc:1233: // Adding ...
6 years, 3 months ago (2014-09-16 21:08:06 UTC) #5
Alexei Svitkine (slow)
lgtm
6 years, 3 months ago (2014-09-16 21:08:57 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/556003007/20001
6 years, 3 months ago (2014-09-16 21:11:04 UTC) #8
commit-bot: I haz the power
Committed patchset #2 (id:20001) as d0667d67b272799eb93569051f51ed7f11961913
6 years, 3 months ago (2014-09-16 22:55:39 UTC) #9
commit-bot: I haz the power
6 years, 3 months ago (2014-09-16 22:56:13 UTC) #10
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/c929946b5b10c2cbaadfa49cd713da3c4e439ac5
Cr-Commit-Position: refs/heads/master@{#295166}

Powered by Google App Engine
This is Rietveld 408576698