Add support for high contrast mode to bubbles

Add support for high contrast mode to bubbles

BUG=105023
TEST=


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

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

[sail, 2011-12-08 02:43:06]

sky: ui/views, browser/ui/views OWNERS review

[msw, 2011-12-08 05:46:30]

LGTM with question/nit.

http://codereview.chromium.org/8887003/diff/1/chrome/browser/ui/views/avatar_...
File chrome/browser/ui/views/avatar_menu_bubble_view.cc (right):

http://codereview.chromium.org/8887003/diff/1/chrome/browser/ui/views/avatar_...
chrome/browser/ui/views/avatar_menu_bubble_view.cc:293:
name_label_->SetBackgroundColor(background_color);
Labels should find backgrounds up the view hierarchy, but that's out of scope.

http://codereview.chromium.org/8887003/diff/1/ui/views/bubble/bubble_delegate.cc
File ui/views/bubble/bubble_delegate.cc (right):

http://codereview.chromium.org/8887003/diff/1/ui/views/bubble/bubble_delegate...
ui/views/bubble/bubble_delegate.cc:75: #if defined(OS_WIN) && !defined(USE_AURA)
The impl seems to run on Aura, any reason you're blocking it here?

[commit-bot: I haz the power, 2011-12-08 05:52:01]

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

[commit-bot: I haz the power, 2011-12-08 05:52:04]

Presubmit check for 8887003-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Messages **
If this change requires manual test instructions to QA team, add
TEST=[instructions].

** Presubmit ERRORS **
Missing LGTM from an OWNER for: ui/views/bubble/bubble_delegate.cc

[sail, 2011-12-08 05:52:07]

http://codereview.chromium.org/8887003/diff/1/ui/views/bubble/bubble_delegate.cc
File ui/views/bubble/bubble_delegate.cc (right):

http://codereview.chromium.org/8887003/diff/1/ui/views/bubble/bubble_delegate...
ui/views/bubble/bubble_delegate.cc:75: #if defined(OS_WIN) && !defined(USE_AURA)
On 2011/12/08 05:46:30, msw wrote:
> The impl seems to run on Aura, any reason you're blocking it here?

This was copy paste from the original implementation.

[msw, 2011-12-08 06:35:28]

Fair enough (although I think it would probably just work on Aura).
LGTM. Please note the remaining work for Linux and Aura in the bug (or close
it).

[sky, 2011-12-08 17:08:44]

LGTM

[commit-bot: I haz the power, 2011-12-08 17:47:35]

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

[commit-bot: I haz the power, 2011-12-08 18:56:44]

Change committed as 113634