diff options
| author | robliao@chromium.org <robliao@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-30 00:14:04 +0000 |
|---|---|---|
| committer | robliao@chromium.org <robliao@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-30 00:14:04 +0000 |
| commit | 7c44a3ae5a93f2fb2689a649e3220026a96aa95e (patch) | |
| tree | ba6253cf3a0783c304931d1556776765ea935495 /chrome/test/base/view_event_test_base.cc | |
| parent | aafcd5e77a8875d8cef4b00f24573e7876a6a5ca (diff) | |
| download | chromium_src-7c44a3ae5a93f2fb2689a649e3220026a96aa95e.zip chromium_src-7c44a3ae5a93f2fb2689a649e3220026a96aa95e.tar.gz chromium_src-7c44a3ae5a93f2fb2689a649e3220026a96aa95e.tar.bz2 | |
Revert of Move DefaultActivationClient to wm/core (https://codereview.chromium.org/250113002/)
Reason for revert:
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().
Original issue's description:
> Move DefaultActivationClient to wm/core
>
> Several tests are using AuraTestHelper and were relying on it to set up the DefaultActivationClient, so I had to have them instantiate the DAC directly after calling SetUp() on the helper. I also had to make the DAC delete itself after the root window it's bound to is destroyed.
>
> R=sky@chromium.org
> http://crbug.com/308710
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266616
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266975
TBR=sky@chromium.org,ben@chromium.org
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/261553002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267001 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test/base/view_event_test_base.cc')
| -rw-r--r-- | chrome/test/base/view_event_test_base.cc | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/chrome/test/base/view_event_test_base.cc b/chrome/test/base/view_event_test_base.cc index aae31d9..d2ace62 100644 --- a/chrome/test/base/view_event_test_base.cc +++ b/chrome/test/base/view_event_test_base.cc @@ -24,7 +24,6 @@ #include "ui/message_center/message_center.h" #include "ui/views/view.h" #include "ui/views/widget/widget.h" -#include "ui/wm/core/default_activation_client.h" #include "ui/wm/core/wm_state.h" #if defined(USE_ASH) @@ -141,7 +140,6 @@ void ViewEventTestBase::SetUp() { aura_test_helper_.reset( new aura::test::AuraTestHelper(base::MessageLoopForUI::current())); aura_test_helper_->SetUp(); - new wm::DefaultActivationClient(aura_test_helper_->root_window()); context = aura_test_helper_->root_window(); #endif // !USE_ASH && USE_AURA |