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

Issue 341733002: views: Fix painting text over old avatar menu button. (Closed)

Created:
6 years, 6 months ago by Elliot Glaysher
Modified:
6 years, 6 months ago
Reviewers:
msw, noms (inactive)
CC:
chromium-reviews, tfarina, noms (inactive), Marc Treib, Bernhard Bauer
Project:
chromium
Visibility:
Public.

Description

views: Fix painting text over old avatar menu button. We were setting text on the button, but that text was ignored previously by overriding OnPaint(). Now just get rid of it. BUG=385934

Patch Set 1 #

Total comments: 1

Patch Set 2 : Remove curly. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -4 lines) Patch
M chrome/browser/ui/views/frame/browser_non_client_frame_view.cc View 1 1 chunk +1 line, -4 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Elliot Glaysher
6 years, 6 months ago (2014-06-18 00:25:02 UTC) #1
msw
lgtm with a nit, presuming this doesn't break the new profile management button's text. CC'ing ...
6 years, 6 months ago (2014-06-18 00:29:38 UTC) #2
Elliot Glaysher
The CQ bit was checked by erg@chromium.org
6 years, 6 months ago (2014-06-18 00:34:16 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/erg@chromium.org/341733002/10002
6 years, 6 months ago (2014-06-18 00:37:57 UTC) #4
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 6 months ago (2014-06-18 14:54:51 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: win_chromium_rel on tryserver.chromium (http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/builds/28913)
6 years, 6 months ago (2014-06-18 14:54:52 UTC) #6
noms (inactive)
--new-profile-management should be fine; that function is only called for the old style profile avatar. ...
6 years, 6 months ago (2014-06-18 15:01:49 UTC) #7
Elliot Glaysher
6 years, 6 months ago (2014-06-18 18:14:06 UTC) #8
This patch is superseded by https://codereview.chromium.org/344643002/ . I have
rolled it into the reland of the patch which broke this in the first place.

Powered by Google App Engine
This is Rietveld 408576698