linux_aura: Ask the NativeTheme for various colors which were hardcoded.

linux_aura: Ask the NativeTheme for various colors which were hardcoded.

This makes text in the omnibox dropdown visible again on linux_aura.

BUG=130810
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192690

[sky, 2013-04-05 23:05:52]

LGTM

https://codereview.chromium.org/13722014/diff/1/chrome/browser/ui/views/omnib...
File chrome/browser/ui/views/omnibox/omnibox_result_view.cc (right):

https://codereview.chromium.org/13722014/diff/1/chrome/browser/ui/views/omnib...
chrome/browser/ui/views/omnibox/omnibox_result_view.cc:162:
colors[NORMAL][BACKGROUND] = theme->GetSystemColor(
Won't this clobber the values set above (such as 147) on windows?

[Elliot Glaysher, 2013-04-05 23:07:52]

https://codereview.chromium.org/13722014/diff/1/chrome/browser/ui/views/omnib...
File chrome/browser/ui/views/omnibox/omnibox_result_view.cc (right):

https://codereview.chromium.org/13722014/diff/1/chrome/browser/ui/views/omnib...
chrome/browser/ui/views/omnibox/omnibox_result_view.cc:162:
colors[NORMAL][BACKGROUND] = theme->GetSystemColor(
On 2013/04/05 23:05:52, sky wrote:
> Won't this clobber the values set above (such as 147) on windows?

That's in a different static array. colors != win_colors, right?

[sky, 2013-04-05 23:13:09]

Ya, sorry. I wrote that first and forgot to remove the comment. That's why I
LGTMd the last patch.

[commit-bot: I haz the power, 2013-04-05 23:18:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/erg@chromium.org/13722014/1

[commit-bot: I haz the power, 2013-04-06 01:46:41]

Change committed as 192690