[uber page] Fix font-family for button and select elements on uber pages.

[uber page] Fix font-family for button and select elements on uber pages.

BUG=112360
TEST=Manual.
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=120136

[csilv, 2012-02-01 21:53:46]

+dbeam for review

[csilv, 2012-02-01 21:56:08]

Hold off on your review, I need to also fix select controls.

[csilv, 2012-02-01 22:01:21]

On 2012/02/01 21:56:08, csilv wrote:
> Hold off on your review, I need to also fix select controls.

Okay, ready for review.  thx!

[Dan Beam, 2012-02-01 23:07:29]

https://chromiumcodereview.appspot.com/9307039/diff/1003/chrome/browser/resou...
File chrome/browser/resources/options2/options_page.css (right):

https://chromiumcodereview.appspot.com/9307039/diff/1003/chrome/browser/resou...
chrome/browser/resources/options2/options_page.css:103: /* TODO(csilv): Move
some of this styling to a shared location for all webui. */
why not do this?

[csilv, 2012-02-01 23:18:29]

https://chromiumcodereview.appspot.com/9307039/diff/1003/chrome/browser/resou...
File chrome/browser/resources/options2/options_page.css (right):

https://chromiumcodereview.appspot.com/9307039/diff/1003/chrome/browser/resou...
chrome/browser/resources/options2/options_page.css:103: /* TODO(csilv): Move
some of this styling to a shared location for all webui. */
On 2012/02/01 23:07:29, Dan Beam wrote:
> why not do this?

I'd prefer to keep the scope of this CL small since it will be a merge to M18. 
Moving it is also related to the issue I just filed regarding focus rings.

[Dan Beam, 2012-02-02 00:15:55]

lgtm w/nit

https://chromiumcodereview.appspot.com/9307039/diff/1003/chrome/browser/resou...
File chrome/browser/resources/options2/options_page.css (right):

https://chromiumcodereview.appspot.com/9307039/diff/1003/chrome/browser/resou...
chrome/browser/resources/options2/options_page.css:103: /* TODO(csilv): Move
some of this styling to a shared location for all webui. */
On 2012/02/01 23:18:30, csilv wrote:
> On 2012/02/01 23:07:29, Dan Beam wrote:
> > why not do this?
> 
> I'd prefer to keep the scope of this CL small since it will be a merge to M18.

> Moving it is also related to the issue I just filed regarding focus rings.

Can you note the timeline in the TODO() then?

[csilv, 2012-02-02 00:57:49]

+jhawkins for OWNERS.

https://chromiumcodereview.appspot.com/9307039/diff/1003/chrome/browser/resou...
File chrome/browser/resources/options2/options_page.css (right):

https://chromiumcodereview.appspot.com/9307039/diff/1003/chrome/browser/resou...
chrome/browser/resources/options2/options_page.css:103: /* TODO(csilv): Move
some of this styling to a shared location for all webui. */
On 2012/02/02 00:15:55, Dan Beam wrote:
> On 2012/02/01 23:18:30, csilv wrote:
> > On 2012/02/01 23:07:29, Dan Beam wrote:
> > > why not do this?
> > 
> > I'd prefer to keep the scope of this CL small since it will be a merge to
M18.
> 
> > Moving it is also related to the issue I just filed regarding focus rings.
> 
> Can you note the timeline in the TODO() then?

Done.

[James Hawkins, 2012-02-02 01:00:40]

lgtm