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

Issue 2856037: [Mac] Center image in keyword-search and ev bubble in omnibox. (Closed)

Created:
10 years, 5 months ago by Scott Hess - ex-Googler
Modified:
9 years, 6 months ago
Reviewers:
John Grabowski
CC:
chromium-reviews, John Grabowski, pam+watch_chromium.org, ben+cc_chromium.org
Base URL:
git://codf21.jail/chromium.git
Visibility:
Public.

Description

[Mac] Center image in keyword-search and ev bubble in omnibox. Omnibox recently got taller on Mac, which screwed up the centering. Visually inspect and re-center. BUG=49101 TEST=https://www.thawte.com/, lock should be centered vertically. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52439

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M chrome/browser/cocoa/location_bar/bubble_decoration.mm View 1 chunk +1 line, -1 line 2 comments Download

Messages

Total messages: 4 (0 generated)
Scott Hess - ex-Googler
10 years, 5 months ago (2010-07-15 00:32:27 UTC) #1
Scott Hess - ex-Googler
Redirecting at John so that I can save Rohit for changes he's most relevant to ...
10 years, 5 months ago (2010-07-15 00:34:08 UTC) #2
John Grabowski
LGTM http://codereview.chromium.org/2856037/diff/1/2 File chrome/browser/cocoa/location_bar/bubble_decoration.mm (left): http://codereview.chromium.org/2856037/diff/1/2#oldcode16 chrome/browser/cocoa/location_bar/bubble_decoration.mm:16: const CGFloat kImageInset = 3.0; If inset is ...
10 years, 5 months ago (2010-07-15 00:39:38 UTC) #3
Scott Hess - ex-Googler
10 years, 5 months ago (2010-07-15 04:07:38 UTC) #4
http://codereview.chromium.org/2856037/diff/1/2
File chrome/browser/cocoa/location_bar/bubble_decoration.mm (left):

http://codereview.chromium.org/2856037/diff/1/2#oldcode16
chrome/browser/cocoa/location_bar/bubble_decoration.mm:16: const CGFloat
kImageInset = 3.0;
On 2010/07/15 00:39:38, John Grabowski wrote:
> If inset is now 0, perhaps you can remove this variable and all references to
> it?

OK.  I was going to say "What if things change again?", but then decided that if
that happens, maybe it could just be calculated dynamically and never have to
visit it again.  Though that's probably wishful thinking.

Powered by Google App Engine
This is Rietveld 408576698