Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Unified Diff: ui/aura/test/aura_test_base.cc

Issue 8387043: [Aura] Support always-on-top top level window. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync, move AlwaysOnTonController from Shell to StackingController Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/aura/test/aura_test_base.cc
diff --git a/ui/aura/test/aura_test_base.cc b/ui/aura/test/aura_test_base.cc
index 8d7a7bcfb3abfc93dbc589731d278fcfb40455ad..4cd94b8730562ac9408a664e80eecce3c51c1504 100644
--- a/ui/aura/test/aura_test_base.cc
+++ b/ui/aura/test/aura_test_base.cc
@@ -38,8 +38,7 @@ AuraTestBase::~AuraTestBase() {
// Flush the message loop because we have pending release tasks
// and these tasks if un-executed would upset Valgrind.
- message_loop_.RunAllPendingWithDispatcher(
- Desktop::GetInstance()->GetDispatcher());
+ FlushMessageLoop();
// Ensure that we don't use the previously-allocated static Desktop object
// later -- on Linux, it holds a reference to our message loop's X connection.
@@ -61,5 +60,10 @@ void AuraTestBase::TearDown() {
testing::Test::TearDown();
}
+void AuraTestBase::FlushMessageLoop() {
+ message_loop_.RunAllPendingWithDispatcher(
+ Desktop::GetInstance()->GetDispatcher());
+}
+
} // namespace test
} // namespace aura

Powered by Google App Engine
This is Rietveld 408576698