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

Unified Diff: third_party/mojo/src/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc

Issue 1406073004: Change mojo_public_bindings_unittests to use chromium environment. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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: third_party/mojo/src/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc
diff --git a/third_party/mojo/src/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc b/third_party/mojo/src/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc
index 6c192854ee78ea94f84a4fc6ef75898cf1d3027e..508787a50d2f02af04b7ffd303ecfdd2c7922a7f 100644
--- a/third_party/mojo/src/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc
+++ b/third_party/mojo/src/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc
@@ -2,10 +2,10 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#include "base/message_loop/message_loop.h"
+#include "mojo/message_pump/message_pump_mojo.h"
#include "mojo/public/cpp/bindings/binding.h"
#include "mojo/public/cpp/bindings/strong_binding.h"
-#include "mojo/public/cpp/environment/environment.h"
-#include "mojo/public/cpp/utility/run_loop.h"
#include "mojo/public/interfaces/bindings/tests/math_calculator.mojom.h"
#include "mojo/public/interfaces/bindings/tests/sample_interfaces.mojom.h"
#include "mojo/public/interfaces/bindings/tests/sample_service.mojom.h"
@@ -119,7 +119,7 @@ class SelfDestructingMathCalculatorUI {
// Add some more and wait for re-entrant call to Output!
calculator_->Add(
1.0, MakeRunnable(&SelfDestructingMathCalculatorUI::Output, this));
- RunLoop::current()->RunUntilIdle();
+ base::MessageLoop::current()->RunUntilIdle();
} else {
delete this;
}
@@ -184,13 +184,13 @@ class IntegerAccessorImpl : public sample::IntegerAccessor {
class InterfacePtrTest : public testing::Test {
public:
+ InterfacePtrTest() : loop_(common::MessagePumpMojo::Create()) {}
~InterfacePtrTest() override { loop_.RunUntilIdle(); }
void PumpMessages() { loop_.RunUntilIdle(); }
private:
- Environment env_;
- RunLoop loop_;
+ base::MessageLoop loop_;
};
TEST_F(InterfacePtrTest, IsBound) {
@@ -470,8 +470,7 @@ class StrongMathCalculatorImpl : public math::Calculator {
};
TEST(StrongConnectorTest, Math) {
- Environment env;
- RunLoop loop;
+ base::MessageLoop loop(common::MessagePumpMojo::Create());
bool error_received = false;
bool destroyed = false;
@@ -539,8 +538,7 @@ class WeakMathCalculatorImpl : public math::Calculator {
};
TEST(WeakConnectorTest, Math) {
- Environment env;
- RunLoop loop;
+ base::MessageLoop loop(common::MessagePumpMojo::Create());
bool error_received = false;
bool destroyed = false;

Powered by Google App Engine
This is Rietveld 408576698