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

Issue 17544002: Adjust avatar menu and avatar label to the look on windows. (Closed)

Created:
7 years, 6 months ago by Adrian Kuegel
Modified:
7 years, 6 months ago
Reviewers:
Elliot Glaysher
CC:
chromium-reviews
Visibility:
Public.

Description

Adjust avatar menu and avatar label to the look on windows. This CL adjusts the look of the avatar menu for managed users to how it looks like on other platforms like windows. A screenshot can be found at the end of the bug description. BUG=241387 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=208073

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -53 lines) Patch
M chrome/browser/ui/gtk/avatar_menu_bubble_gtk.h View 2 chunks +0 lines, -5 lines 0 comments Download
M chrome/browser/ui/gtk/avatar_menu_bubble_gtk.cc View 3 chunks +20 lines, -46 lines 0 comments Download
M chrome/browser/ui/gtk/browser_titlebar.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
Adrian Kuegel
Elliot, can you please review this CL? I removed the GtkTextView and use a label ...
7 years, 6 months ago (2013-06-21 09:10:13 UTC) #1
Elliot Glaysher
lgtm
7 years, 6 months ago (2013-06-21 17:28:37 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/akuegel@chromium.org/17544002/1
7 years, 6 months ago (2013-06-21 21:06:41 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/akuegel@chromium.org/17544002/1
7 years, 6 months ago (2013-06-22 02:46:15 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/akuegel@chromium.org/17544002/1
7 years, 6 months ago (2013-06-22 03:08:59 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/akuegel@chromium.org/17544002/1
7 years, 6 months ago (2013-06-22 03:22:00 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/akuegel@chromium.org/17544002/1
7 years, 6 months ago (2013-06-22 03:36:35 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/akuegel@chromium.org/17544002/1
7 years, 6 months ago (2013-06-22 03:44:06 UTC) #8
commit-bot: I haz the power
7 years, 6 months ago (2013-06-22 17:10:20 UTC) #9
Message was sent while issue was closed.
Change committed as 208073

Powered by Google App Engine
This is Rietveld 408576698