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

Issue 240093006: CoreAnimation: Fix text anti-aliasing in download shelf (Closed)

Created:
6 years, 8 months ago by ccameron
Modified:
6 years, 8 months ago
Reviewers:
Avi (use Gerrit)
CC:
chromium-reviews, asanka, tfarina, benjhayden+dwatch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@fix_omnibox_aa
Visibility:
Public.

Description

CoreAnimation: Fix text anti-aliasing in download shelf Make the download shelf buttons opaque, and have them use the download shelf's draw function to draw. Add a helper function to look up an ancestor view based on the view ID. This is particularly useful because the download shelf is five views above the download shelf button view. Update the bookmark button view function to refer to the toolbar view as the download shelf toolbar view (which does not have a view ID associated with it). BUG=363750 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=264363

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fix typo #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -4 lines) Patch
M chrome/browser/ui/cocoa/bookmarks/bookmark_button.mm View 1 chunk +4 lines, -4 lines 0 comments Download
M chrome/browser/ui/cocoa/download/download_item_button.mm View 2 chunks +14 lines, -0 lines 0 comments Download
M chrome/browser/ui/cocoa/download/download_show_all_button.mm View 2 chunks +14 lines, -0 lines 0 comments Download
M chrome/browser/ui/cocoa/view_id_util.h View 1 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/ui/cocoa/view_id_util.mm View 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
ccameron
Small fix, usual scheme. https://codereview.chromium.org/240093006/diff/1/chrome/browser/ui/cocoa/bookmarks/bookmark_button.mm File chrome/browser/ui/cocoa/bookmarks/bookmark_button.mm (right): https://codereview.chromium.org/240093006/diff/1/chrome/browser/ui/cocoa/bookmarks/bookmark_button.mm#newcode413 chrome/browser/ui/cocoa/bookmarks/bookmark_button.mm:413: - (void)drawRect:(NSRect)rect { I kept ...
6 years, 8 months ago (2014-04-16 19:17:55 UTC) #1
Avi (use Gerrit)
lgtm
6 years, 8 months ago (2014-04-16 19:57:56 UTC) #2
ccameron
Thanks!
6 years, 8 months ago (2014-04-16 20:11:19 UTC) #3
ccameron
The CQ bit was checked by ccameron@chromium.org
6 years, 8 months ago (2014-04-16 20:11:24 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ccameron@chromium.org/240093006/20001
6 years, 8 months ago (2014-04-16 20:12:05 UTC) #5
commit-bot: I haz the power
6 years, 8 months ago (2014-04-17 00:19:04 UTC) #6
Message was sent while issue was closed.
Change committed as 264363

Powered by Google App Engine
This is Rietveld 408576698