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

Issue 8394004: [Multi Profile] On Personal Options page, users list should display avatar icons. (Closed)

Created:
9 years, 2 months ago by binji
Modified:
9 years, 2 months ago
CC:
chromium-reviews, arv (Not doing code reviews)
Visibility:
Public.

Description

[Multi Profile] On Personal Options page, users list should display avatar icons. BUG=101037 TEST=Go to chrome://settings/personal. Add a user. Icons should be displayed in the users list. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107179

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -2 lines) Patch
M chrome/browser/resources/options/personal_options.css View 1 chunk +11 lines, -0 lines 0 comments Download
M chrome/browser/resources/options/personal_options_profile_list.js View 1 chunk +7 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
binji
9 years, 2 months ago (2011-10-25 17:38:41 UTC) #1
James Hawkins
Pics or it didn't happen.
9 years, 2 months ago (2011-10-25 17:41:46 UTC) #2
James Hawkins
LGTM...lazy.
9 years, 2 months ago (2011-10-25 18:07:25 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/binji@chromium.org/8394004/1
9 years, 2 months ago (2011-10-25 18:19:19 UTC) #4
commit-bot: I haz the power
9 years, 2 months ago (2011-10-25 19:49:40 UTC) #5
Change committed as 107179

Powered by Google App Engine
This is Rietveld 408576698