Linux Aura: get tooltip background color from theme

Linux Aura: get tooltip background color from theme

the border color can still conflict with the theme (e.g., no visible border if the tooltip bg is dark), but fixing that is a lot harder because the border, window roundedness, and shadow are all one tiled set of images (see shadow.cc).

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

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

[Ben Goodger (Google), 2014-04-25 15:51:11]

lgtm

[Evan Stade, 2014-04-25 19:04:23]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2014-04-25 21:15:48]

The CQ bit was unchecked by estade@chromium.org

[Evan Stade, 2014-04-25 21:15:48]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-04-25 22:01:11]

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

[commit-bot: I haz the power, 2014-04-25 23:19:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-25 23:19:43]

Try jobs failed on following builders:
  tryserver.chromium on win_chromium_rel

[Evan Stade, 2014-04-28 22:42:14]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-04-28 22:42:53]

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

[commit-bot: I haz the power, 2014-04-28 23:41:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-28 23:41:31]

Try jobs failed on following builders:
  tryserver.chromium on win_chromium_rel

[Evan Stade, 2014-04-29 16:48:37]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-04-29 16:49:14]

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

[commit-bot: I haz the power, 2014-04-29 17:39:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-29 17:39:02]

Try jobs failed on following builders:
  win_chromium_rel on tryserver.chromium

[Evan Stade, 2014-04-29 19:23:39]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-04-29 19:27:15]

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

[commit-bot: I haz the power, 2014-04-29 22:15:36]

Change committed as 266974

[robliao, 2014-04-29 23:12:05]

A revert of this CL has been created in
https://codereview.chromium.org/263533003/ by robliao@chromium.org.

The reason for reverting is: views_unittest TooltipController test failures
views_unittests views_unittests 4 disabled 10 flaky failed 10 ( 9 secs )
stdio
Capture
CloseOnCaptureLost
DontShowEmptyTooltips
EnableOrDisableTooltips
HideOnExit
ReshowOnClickAfterEnterExit
TooltipHidesOnKeyPressAndStaysHiddenUntilChange
TooltipHidesOnTimeoutAndStaysHiddenUntilChange
TooltipsInMultipleViews
ViewTooltip


TooltipControllerTest.ViewTooltip (run #2):
[ RUN      ] TooltipControllerTest.ViewTooltip
Xlib:  extension "RANDR" missing on display ":9".
Received signal 11 SEGV_MAPERR 000000000000
#0 0x000000a88e4e base::debug::StackTrace::StackTrace()
#1 0x000000a89068 base::debug::(anonymous namespace)::StackDumpSignalHandler()
#2 0x7f9474270cb0 \u003Cunknown>
#3 0x000000736b83 views::NativeWidgetAura::Activate()
#4 0x000000737508 views::NativeWidgetAura::ShowWithWindowState()
#5 0x000000738acb views::Widget::Show()
#6 0x00000052cb09 views::corewm::test::(anonymous namespace)::CreateWidget()
#7 0x00000052d397 views::corewm::test::TooltipControllerTest::SetUp()
#8 0x00000065a16a testing::Test::Run().

[robliao, 2014-04-30 00:04:54]

The CQ bit was checked by robliao@chromium.org

[robliao, 2014-04-30 00:05:35]

On 2014/04/30 00:04:54, robliao wrote:
> The CQ bit was checked by mailto:robliao@chromium.org

Relanding as reverting this didn't fix the issue.

[commit-bot: I haz the power, 2014-04-30 00:07:57]

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

[commit-bot: I haz the power, 2014-04-30 00:10:25]

Change committed as 267000