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

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

Issue 1660143006: Mojo C++ bindings: Move MakeProxy() -> InterfacePtr::Create(). (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: fix failed 'gn' Created 4 years, 10 months 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: mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc
diff --git a/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc b/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc
index dc4a55a7eec15079f4c559862dde39f2a4bd4f2b..e0a21c76b0faac9282ea735c1ef88aacd4603824 100644
--- a/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc
+++ b/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc
@@ -2,6 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#include <utility>
+
#include "mojo/public/cpp/bindings/binding.h"
#include "mojo/public/cpp/bindings/strong_binding.h"
#include "mojo/public/cpp/environment/environment.h"
@@ -274,7 +276,8 @@ TEST_F(InterfacePtrTest, Resettable) {
// Save this so we can test it later.
Handle handle = pipe.handle0.get();
- a = MakeProxy(InterfaceHandle<math::Calculator>(pipe.handle0.Pass(), 0u));
+ a = math::CalculatorPtr::Create(
+ InterfaceHandle<math::Calculator>(std::move(pipe.handle0), 0u));
EXPECT_FALSE(!a);

Powered by Google App Engine
This is Rietveld 408576698