First cut at badge implementation. Right now it just uses static

First cut at badge implementation. Right now it just uses static
data. A subsequent change will hook it up to the extension APIs.

See http://dl.getdropbox.com/u/124107/badges.png for a picture
of what this looks like.

BUG=23268
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27761

[Erik does not do reviews, 2009-10-01 20:21:04]

LGTM

http://codereview.chromium.org/259004/diff/3001/4006
File chrome/browser/views/browser_actions_container.cc (right):

http://codereview.chromium.org/259004/diff/3001/4006#newcode286
Line 286: text_paint.setTextAlign(SkPaint::kLeft_Align);
Don't we want to be centered?  In the case where there's a minimum sized rect,
and a narrow string, I'd imagine that left aligned would look odd.

[Aaron Boodman, 2009-10-01 20:30:40]

On Thu, Oct 1, 2009 at 1:21 PM,  <erikkay@chromium.org> wrote:
> http://codereview.chromium.org/259004/diff/3001/4006
> File chrome/browser/views/browser_actions_container.cc (right):
>
> http://codereview.chromium.org/259004/diff/3001/4006#newcode286
> Line 286: text_paint.setTextAlign(SkPaint::kLeft_Align);
> Don't we want to be centered? =A0In the case where there's a minimum size=
d
> rect, and a narrow string, I'd imagine that left aligned would look odd.

The text box is centered separately, in the drawText() step.

- a

[Erik does not do reviews, 2009-10-01 20:36:11]

I see it now.  Thanks.