Share Font Code for New Autofill.

Share Font Code for New Autofill.

Move the gfx::Font code into the base class to allow the platform specific subclasses to share it.

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

[csharp, 2012-09-25 14:31:24]

erg@ for gtk section
isherman@ for autofill section.

I have a follow up views cl that will use this newly shared code.

[tfarina, 2012-09-25 16:00:58]

http://codereview.chromium.org/10987018/diff/1/chrome/browser/autofill/autofi...
File chrome/browser/autofill/autofill_popup_view.h (right):

http://codereview.chromium.org/10987018/diff/1/chrome/browser/autofill/autofi...
chrome/browser/autofill/autofill_popup_view.h:74: const gfx::Font label_font()
const { return label_font_; }
return this by const-ref

[csharp, 2012-09-25 20:08:00]

http://codereview.chromium.org/10987018/diff/1/chrome/browser/autofill/autofi...
File chrome/browser/autofill/autofill_popup_view.h (right):

http://codereview.chromium.org/10987018/diff/1/chrome/browser/autofill/autofi...
chrome/browser/autofill/autofill_popup_view.h:74: const gfx::Font label_font()
const { return label_font_; }
On 2012/09/25 16:00:58, tfarina wrote:
> return this by const-ref

Done.

[Elliot Glaysher, 2012-09-25 20:54:31]

gtk lgtm

[Ilya Sherman, 2012-09-25 21:20:26]

lgtm

[commit-bot: I haz the power, 2012-09-25 21:20:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/csharp@chromium.org/10987018/5001

[commit-bot: I haz the power, 2012-09-25 23:17:58]

Change committed as 158684