yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 1 | // Copyright 2015 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
Avi Drissman | 20b0cb0 | 2015-12-22 03:14:57 +0900 | [diff] [blame] | 5 | #include <stddef.h> |
| 6 | #include <stdint.h> |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 7 | #include <algorithm> |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 8 | #include <utility> |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 9 | |
| 10 | #include "base/bind.h" |
rockot | 7464c2f | 2016-06-20 02:08:36 +0900 | [diff] [blame] | 11 | #include "base/callback.h" |
rockot | cffd1b2 | 2016-08-25 08:37:11 +0900 | [diff] [blame] | 12 | #include "base/callback_helpers.h" |
fdoray | 64ef147 | 2016-12-24 04:51:48 +0900 | [diff] [blame] | 13 | #include "base/memory/ptr_util.h" |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 14 | #include "base/run_loop.h" |
fdoray | c278316 | 2016-07-05 06:56:42 +0900 | [diff] [blame] | 15 | #include "base/single_thread_task_runner.h" |
jcivelli | 3907bec | 2017-01-07 03:23:01 +0900 | [diff] [blame] | 16 | #include "base/synchronization/waitable_event.h" |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 17 | #include "base/task_scheduler/post_task.h" |
| 18 | #include "base/test/scoped_task_environment.h" |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 19 | #include "base/threading/sequenced_task_runner_handle.h" |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 20 | #include "base/threading/thread.h" |
gab | 1400a96 | 2016-05-12 03:11:16 +0900 | [diff] [blame] | 21 | #include "base/threading/thread_task_runner_handle.h" |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 22 | #include "mojo/public/cpp/bindings/associated_binding.h" |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 23 | #include "mojo/public/cpp/bindings/associated_interface_ptr.h" |
| 24 | #include "mojo/public/cpp/bindings/associated_interface_ptr_info.h" |
| 25 | #include "mojo/public/cpp/bindings/associated_interface_request.h" |
| 26 | #include "mojo/public/cpp/bindings/binding.h" |
| 27 | #include "mojo/public/cpp/bindings/lib/multiplex_router.h" |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 28 | #include "mojo/public/cpp/bindings/strong_binding.h" |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 29 | #include "mojo/public/cpp/bindings/thread_safe_interface_ptr.h" |
rockot | cffd1b2 | 2016-08-25 08:37:11 +0900 | [diff] [blame] | 30 | #include "mojo/public/interfaces/bindings/tests/ping_service.mojom.h" |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 31 | #include "mojo/public/interfaces/bindings/tests/test_associated_interfaces.mojom.h" |
| 32 | #include "testing/gtest/include/gtest/gtest.h" |
| 33 | |
| 34 | namespace mojo { |
| 35 | namespace test { |
| 36 | namespace { |
| 37 | |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 38 | using mojo::internal::MultiplexRouter; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 39 | |
| 40 | class IntegerSenderImpl : public IntegerSender { |
| 41 | public: |
| 42 | explicit IntegerSenderImpl(AssociatedInterfaceRequest<IntegerSender> request) |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 43 | : binding_(this, std::move(request)) {} |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 44 | |
| 45 | ~IntegerSenderImpl() override {} |
| 46 | |
| 47 | void set_notify_send_method_called( |
| 48 | const base::Callback<void(int32_t)>& callback) { |
| 49 | notify_send_method_called_ = callback; |
| 50 | } |
| 51 | |
| 52 | void Echo(int32_t value, const EchoCallback& callback) override { |
| 53 | callback.Run(value); |
| 54 | } |
| 55 | void Send(int32_t value) override { notify_send_method_called_.Run(value); } |
| 56 | |
| 57 | AssociatedBinding<IntegerSender>* binding() { return &binding_; } |
| 58 | |
rockot | 7464c2f | 2016-06-20 02:08:36 +0900 | [diff] [blame] | 59 | void set_connection_error_handler(const base::Closure& handler) { |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 60 | binding_.set_connection_error_handler(handler); |
| 61 | } |
| 62 | |
| 63 | private: |
| 64 | AssociatedBinding<IntegerSender> binding_; |
| 65 | base::Callback<void(int32_t)> notify_send_method_called_; |
| 66 | }; |
| 67 | |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 68 | class IntegerSenderConnectionImpl : public IntegerSenderConnection { |
| 69 | public: |
| 70 | explicit IntegerSenderConnectionImpl( |
| 71 | InterfaceRequest<IntegerSenderConnection> request) |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 72 | : binding_(this, std::move(request)) {} |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 73 | |
| 74 | ~IntegerSenderConnectionImpl() override {} |
| 75 | |
| 76 | void GetSender(AssociatedInterfaceRequest<IntegerSender> sender) override { |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 77 | IntegerSenderImpl* sender_impl = new IntegerSenderImpl(std::move(sender)); |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 78 | sender_impl->set_connection_error_handler( |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 79 | base::Bind(&DeleteSender, sender_impl)); |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 80 | } |
| 81 | |
| 82 | void AsyncGetSender(const AsyncGetSenderCallback& callback) override { |
yzshen | 9ce8109 | 2016-01-20 09:42:42 +0900 | [diff] [blame] | 83 | IntegerSenderAssociatedPtrInfo ptr_info; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 84 | auto request = MakeRequest(&ptr_info); |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 85 | GetSender(std::move(request)); |
| 86 | callback.Run(std::move(ptr_info)); |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 87 | } |
| 88 | |
yzshen | 09b1228 | 2015-12-11 10:37:50 +0900 | [diff] [blame] | 89 | Binding<IntegerSenderConnection>* binding() { return &binding_; } |
| 90 | |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 91 | private: |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 92 | static void DeleteSender(IntegerSenderImpl* sender) { delete sender; } |
| 93 | |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 94 | Binding<IntegerSenderConnection> binding_; |
| 95 | }; |
| 96 | |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 97 | class AssociatedInterfaceTest : public testing::Test { |
| 98 | public: |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 99 | AssociatedInterfaceTest() |
| 100 | : main_runner_(base::ThreadTaskRunnerHandle::Get()) {} |
fdoray | c278316 | 2016-07-05 06:56:42 +0900 | [diff] [blame] | 101 | ~AssociatedInterfaceTest() override { base::RunLoop().RunUntilIdle(); } |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 102 | |
fdoray | c278316 | 2016-07-05 06:56:42 +0900 | [diff] [blame] | 103 | void PumpMessages() { base::RunLoop().RunUntilIdle(); } |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 104 | |
| 105 | template <typename T> |
| 106 | AssociatedInterfacePtrInfo<T> EmulatePassingAssociatedPtrInfo( |
| 107 | AssociatedInterfacePtrInfo<T> ptr_info, |
yzshen | 8dc7bd7 | 2017-02-15 07:24:25 +0900 | [diff] [blame] | 108 | scoped_refptr<MultiplexRouter> source, |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 109 | scoped_refptr<MultiplexRouter> target) { |
rockot | a3695c5 | 2016-05-18 10:16:03 +0900 | [diff] [blame] | 110 | ScopedInterfaceEndpointHandle handle = ptr_info.PassHandle(); |
yzshen | 8dc7bd7 | 2017-02-15 07:24:25 +0900 | [diff] [blame] | 111 | CHECK(handle.pending_association()); |
| 112 | auto id = source->AssociateInterface(std::move(handle)); |
| 113 | return AssociatedInterfacePtrInfo<T>(target->CreateLocalEndpointHandle(id), |
| 114 | ptr_info.version()); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 115 | } |
| 116 | |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 117 | void CreateRouterPair(scoped_refptr<MultiplexRouter>* router0, |
| 118 | scoped_refptr<MultiplexRouter>* router1) { |
| 119 | MessagePipe pipe; |
yzshen | 3830edd | 2016-09-16 08:30:31 +0900 | [diff] [blame] | 120 | *router0 = new MultiplexRouter(std::move(pipe.handle0), |
| 121 | MultiplexRouter::MULTI_INTERFACE, true, |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 122 | main_runner_); |
yzshen | 3830edd | 2016-09-16 08:30:31 +0900 | [diff] [blame] | 123 | *router1 = new MultiplexRouter(std::move(pipe.handle1), |
| 124 | MultiplexRouter::MULTI_INTERFACE, false, |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 125 | main_runner_); |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 126 | } |
| 127 | |
| 128 | void CreateIntegerSenderWithExistingRouters( |
| 129 | scoped_refptr<MultiplexRouter> router0, |
| 130 | IntegerSenderAssociatedPtrInfo* ptr_info0, |
| 131 | scoped_refptr<MultiplexRouter> router1, |
| 132 | IntegerSenderAssociatedRequest* request1) { |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 133 | *request1 = MakeRequest(ptr_info0); |
yzshen | 8dc7bd7 | 2017-02-15 07:24:25 +0900 | [diff] [blame] | 134 | *ptr_info0 = EmulatePassingAssociatedPtrInfo(std::move(*ptr_info0), router1, |
| 135 | router0); |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 136 | } |
| 137 | |
| 138 | void CreateIntegerSender(IntegerSenderAssociatedPtrInfo* ptr_info, |
| 139 | IntegerSenderAssociatedRequest* request) { |
| 140 | scoped_refptr<MultiplexRouter> router0; |
| 141 | scoped_refptr<MultiplexRouter> router1; |
| 142 | CreateRouterPair(&router0, &router1); |
| 143 | CreateIntegerSenderWithExistingRouters(router1, ptr_info, router0, request); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 144 | } |
| 145 | |
| 146 | // Okay to call from any thread. |
| 147 | void QuitRunLoop(base::RunLoop* run_loop) { |
peary2 | e3be235 | 2017-06-29 11:15:28 +0900 | [diff] [blame] | 148 | if (main_runner_->RunsTasksInCurrentSequence()) { |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 149 | run_loop->Quit(); |
| 150 | } else { |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 151 | main_runner_->PostTask( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 152 | FROM_HERE, |
| 153 | base::Bind(&AssociatedInterfaceTest::QuitRunLoop, |
| 154 | base::Unretained(this), base::Unretained(run_loop))); |
| 155 | } |
| 156 | } |
| 157 | |
| 158 | private: |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 159 | base::test::ScopedTaskEnvironment task_environment; |
| 160 | scoped_refptr<base::SequencedTaskRunner> main_runner_; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 161 | }; |
| 162 | |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 163 | void DoSetFlagAndRunClosure(bool* flag, const base::Closure& closure) { |
| 164 | *flag = true; |
| 165 | closure.Run(); |
| 166 | } |
| 167 | |
| 168 | void DoExpectValueSetFlagAndRunClosure(int32_t expected_value, |
| 169 | bool* flag, |
| 170 | const base::Closure& closure, |
| 171 | int32_t value) { |
| 172 | EXPECT_EQ(expected_value, value); |
| 173 | DoSetFlagAndRunClosure(flag, closure); |
| 174 | } |
| 175 | |
| 176 | base::Closure SetFlagAndRunClosure(bool* flag, const base::Closure& closure) { |
| 177 | return base::Bind(&DoSetFlagAndRunClosure, flag, closure); |
| 178 | } |
| 179 | |
| 180 | base::Callback<void(int32_t)> ExpectValueSetFlagAndRunClosure( |
| 181 | int32_t expected_value, |
| 182 | bool* flag, |
| 183 | const base::Closure& closure) { |
| 184 | return base::Bind( |
| 185 | &DoExpectValueSetFlagAndRunClosure, expected_value, flag, closure); |
| 186 | } |
| 187 | |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 188 | void Fail() { |
| 189 | FAIL() << "Unexpected connection error"; |
| 190 | } |
| 191 | |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 192 | TEST_F(AssociatedInterfaceTest, InterfacesAtBothEnds) { |
| 193 | // Bind to the same pipe two associated interfaces, whose implementation lives |
| 194 | // at different ends. Test that the two don't interfere with each other. |
| 195 | |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 196 | scoped_refptr<MultiplexRouter> router0; |
| 197 | scoped_refptr<MultiplexRouter> router1; |
| 198 | CreateRouterPair(&router0, &router1); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 199 | |
| 200 | AssociatedInterfaceRequest<IntegerSender> request; |
yzshen | 9ce8109 | 2016-01-20 09:42:42 +0900 | [diff] [blame] | 201 | IntegerSenderAssociatedPtrInfo ptr_info; |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 202 | CreateIntegerSenderWithExistingRouters(router1, &ptr_info, router0, &request); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 203 | |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 204 | IntegerSenderImpl impl0(std::move(request)); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 205 | AssociatedInterfacePtr<IntegerSender> ptr0; |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 206 | ptr0.Bind(std::move(ptr_info)); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 207 | |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 208 | CreateIntegerSenderWithExistingRouters(router0, &ptr_info, router1, &request); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 209 | |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 210 | IntegerSenderImpl impl1(std::move(request)); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 211 | AssociatedInterfacePtr<IntegerSender> ptr1; |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 212 | ptr1.Bind(std::move(ptr_info)); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 213 | |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 214 | base::RunLoop run_loop, run_loop2; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 215 | bool ptr0_callback_run = false; |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 216 | ptr0->Echo(123, ExpectValueSetFlagAndRunClosure(123, &ptr0_callback_run, |
| 217 | run_loop.QuitClosure())); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 218 | |
| 219 | bool ptr1_callback_run = false; |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 220 | ptr1->Echo(456, ExpectValueSetFlagAndRunClosure(456, &ptr1_callback_run, |
| 221 | run_loop2.QuitClosure())); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 222 | |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 223 | run_loop.Run(); |
| 224 | run_loop2.Run(); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 225 | EXPECT_TRUE(ptr0_callback_run); |
| 226 | EXPECT_TRUE(ptr1_callback_run); |
| 227 | |
| 228 | bool ptr0_error_callback_run = false; |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 229 | base::RunLoop run_loop3; |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 230 | ptr0.set_connection_error_handler( |
| 231 | SetFlagAndRunClosure(&ptr0_error_callback_run, run_loop3.QuitClosure())); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 232 | |
| 233 | impl0.binding()->Close(); |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 234 | run_loop3.Run(); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 235 | EXPECT_TRUE(ptr0_error_callback_run); |
| 236 | |
| 237 | bool impl1_error_callback_run = false; |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 238 | base::RunLoop run_loop4; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 239 | impl1.binding()->set_connection_error_handler( |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 240 | SetFlagAndRunClosure(&impl1_error_callback_run, run_loop4.QuitClosure())); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 241 | |
| 242 | ptr1.reset(); |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 243 | run_loop4.Run(); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 244 | EXPECT_TRUE(impl1_error_callback_run); |
| 245 | } |
| 246 | |
| 247 | class TestSender { |
| 248 | public: |
| 249 | TestSender() |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 250 | : task_runner_(base::CreateSequencedTaskRunnerWithTraits({})), |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 251 | next_sender_(nullptr), |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 252 | max_value_to_send_(-1) {} |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 253 | |
| 254 | // The following three methods are called on the corresponding sender thread. |
yzshen | 9ce8109 | 2016-01-20 09:42:42 +0900 | [diff] [blame] | 255 | void SetUp(IntegerSenderAssociatedPtrInfo ptr_info, |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 256 | TestSender* next_sender, |
| 257 | int32_t max_value_to_send) { |
peary2 | e3be235 | 2017-06-29 11:15:28 +0900 | [diff] [blame] | 258 | CHECK(task_runner()->RunsTasksInCurrentSequence()); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 259 | |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 260 | ptr_.Bind(std::move(ptr_info)); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 261 | next_sender_ = next_sender ? next_sender : this; |
| 262 | max_value_to_send_ = max_value_to_send; |
| 263 | } |
| 264 | |
| 265 | void Send(int32_t value) { |
peary2 | e3be235 | 2017-06-29 11:15:28 +0900 | [diff] [blame] | 266 | CHECK(task_runner()->RunsTasksInCurrentSequence()); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 267 | |
| 268 | if (value > max_value_to_send_) |
| 269 | return; |
| 270 | |
| 271 | ptr_->Send(value); |
| 272 | |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 273 | next_sender_->task_runner()->PostTask( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 274 | FROM_HERE, |
| 275 | base::Bind(&TestSender::Send, base::Unretained(next_sender_), ++value)); |
| 276 | } |
| 277 | |
| 278 | void TearDown() { |
peary2 | e3be235 | 2017-06-29 11:15:28 +0900 | [diff] [blame] | 279 | CHECK(task_runner()->RunsTasksInCurrentSequence()); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 280 | |
| 281 | ptr_.reset(); |
| 282 | } |
| 283 | |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 284 | base::SequencedTaskRunner* task_runner() { return task_runner_.get(); } |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 285 | |
| 286 | private: |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 287 | scoped_refptr<base::SequencedTaskRunner> task_runner_; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 288 | TestSender* next_sender_; |
| 289 | int32_t max_value_to_send_; |
| 290 | |
| 291 | AssociatedInterfacePtr<IntegerSender> ptr_; |
| 292 | }; |
| 293 | |
| 294 | class TestReceiver { |
| 295 | public: |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 296 | TestReceiver() |
| 297 | : task_runner_(base::CreateSequencedTaskRunnerWithTraits({})), |
| 298 | expected_calls_(0) {} |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 299 | |
| 300 | void SetUp(AssociatedInterfaceRequest<IntegerSender> request0, |
| 301 | AssociatedInterfaceRequest<IntegerSender> request1, |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 302 | size_t expected_calls, |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 303 | const base::Closure& notify_finish) { |
peary2 | e3be235 | 2017-06-29 11:15:28 +0900 | [diff] [blame] | 304 | CHECK(task_runner()->RunsTasksInCurrentSequence()); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 305 | |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 306 | impl0_.reset(new IntegerSenderImpl(std::move(request0))); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 307 | impl0_->set_notify_send_method_called( |
| 308 | base::Bind(&TestReceiver::SendMethodCalled, base::Unretained(this))); |
dcheng | cb03523 | 2015-12-26 13:46:36 +0900 | [diff] [blame] | 309 | impl1_.reset(new IntegerSenderImpl(std::move(request1))); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 310 | impl1_->set_notify_send_method_called( |
| 311 | base::Bind(&TestReceiver::SendMethodCalled, base::Unretained(this))); |
| 312 | |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 313 | expected_calls_ = expected_calls; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 314 | notify_finish_ = notify_finish; |
| 315 | } |
| 316 | |
| 317 | void TearDown() { |
peary2 | e3be235 | 2017-06-29 11:15:28 +0900 | [diff] [blame] | 318 | CHECK(task_runner()->RunsTasksInCurrentSequence()); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 319 | |
| 320 | impl0_.reset(); |
| 321 | impl1_.reset(); |
| 322 | } |
| 323 | |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 324 | base::SequencedTaskRunner* task_runner() { return task_runner_.get(); } |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 325 | const std::vector<int32_t>& values() const { return values_; } |
| 326 | |
| 327 | private: |
| 328 | void SendMethodCalled(int32_t value) { |
| 329 | values_.push_back(value); |
| 330 | |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 331 | if (values_.size() >= expected_calls_) |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 332 | notify_finish_.Run(); |
| 333 | } |
| 334 | |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 335 | scoped_refptr<base::SequencedTaskRunner> task_runner_; |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 336 | size_t expected_calls_; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 337 | |
dcheng | 551f9e5 | 2016-04-22 09:57:38 +0900 | [diff] [blame] | 338 | std::unique_ptr<IntegerSenderImpl> impl0_; |
| 339 | std::unique_ptr<IntegerSenderImpl> impl1_; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 340 | |
| 341 | std::vector<int32_t> values_; |
| 342 | |
| 343 | base::Closure notify_finish_; |
| 344 | }; |
| 345 | |
yzshen | f9cc80a | 2015-12-09 06:48:32 +0900 | [diff] [blame] | 346 | class NotificationCounter { |
| 347 | public: |
| 348 | NotificationCounter(size_t total_count, const base::Closure& notify_finish) |
| 349 | : total_count_(total_count), |
| 350 | current_count_(0), |
| 351 | notify_finish_(notify_finish) {} |
dtrainor | a2e37bb | 2015-12-08 10:09:57 +0900 | [diff] [blame] | 352 | |
yzshen | f9cc80a | 2015-12-09 06:48:32 +0900 | [diff] [blame] | 353 | ~NotificationCounter() {} |
| 354 | |
| 355 | // Okay to call from any thread. |
| 356 | void OnGotNotification() { |
| 357 | bool finshed = false; |
| 358 | { |
| 359 | base::AutoLock locker(lock_); |
| 360 | CHECK_LT(current_count_, total_count_); |
| 361 | current_count_++; |
| 362 | finshed = current_count_ == total_count_; |
| 363 | } |
| 364 | |
| 365 | if (finshed) |
| 366 | notify_finish_.Run(); |
| 367 | } |
| 368 | |
| 369 | private: |
| 370 | base::Lock lock_; |
| 371 | const size_t total_count_; |
| 372 | size_t current_count_; |
| 373 | base::Closure notify_finish_; |
| 374 | }; |
| 375 | |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 376 | TEST_F(AssociatedInterfaceTest, MultiThreadAccess) { |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 377 | // Set up four associated interfaces on a message pipe. Use the inteface |
| 378 | // pointers on four threads in parallel; run the interface implementations on |
| 379 | // two threads. Test that multi-threaded access works. |
| 380 | |
| 381 | const int32_t kMaxValue = 1000; |
| 382 | MessagePipe pipe; |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 383 | scoped_refptr<MultiplexRouter> router0; |
| 384 | scoped_refptr<MultiplexRouter> router1; |
| 385 | CreateRouterPair(&router0, &router1); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 386 | |
| 387 | AssociatedInterfaceRequest<IntegerSender> requests[4]; |
yzshen | 9ce8109 | 2016-01-20 09:42:42 +0900 | [diff] [blame] | 388 | IntegerSenderAssociatedPtrInfo ptr_infos[4]; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 389 | for (size_t i = 0; i < 4; ++i) { |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 390 | CreateIntegerSenderWithExistingRouters(router1, &ptr_infos[i], router0, |
| 391 | &requests[i]); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 392 | } |
| 393 | |
| 394 | TestSender senders[4]; |
| 395 | for (size_t i = 0; i < 4; ++i) { |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 396 | senders[i].task_runner()->PostTask( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 397 | FROM_HERE, base::Bind(&TestSender::SetUp, base::Unretained(&senders[i]), |
| 398 | base::Passed(&ptr_infos[i]), nullptr, |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 399 | kMaxValue * (i + 1) / 4)); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 400 | } |
| 401 | |
| 402 | base::RunLoop run_loop; |
| 403 | TestReceiver receivers[2]; |
yzshen | f9cc80a | 2015-12-09 06:48:32 +0900 | [diff] [blame] | 404 | NotificationCounter counter( |
| 405 | 2, base::Bind(&AssociatedInterfaceTest::QuitRunLoop, |
| 406 | base::Unretained(this), base::Unretained(&run_loop))); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 407 | for (size_t i = 0; i < 2; ++i) { |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 408 | receivers[i].task_runner()->PostTask( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 409 | FROM_HERE, |
yzshen | f9cc80a | 2015-12-09 06:48:32 +0900 | [diff] [blame] | 410 | base::Bind(&TestReceiver::SetUp, base::Unretained(&receivers[i]), |
| 411 | base::Passed(&requests[2 * i]), |
| 412 | base::Passed(&requests[2 * i + 1]), |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 413 | static_cast<size_t>(kMaxValue / 2), |
yzshen | f9cc80a | 2015-12-09 06:48:32 +0900 | [diff] [blame] | 414 | base::Bind(&NotificationCounter::OnGotNotification, |
| 415 | base::Unretained(&counter)))); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 416 | } |
| 417 | |
| 418 | for (size_t i = 0; i < 4; ++i) { |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 419 | senders[i].task_runner()->PostTask( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 420 | FROM_HERE, base::Bind(&TestSender::Send, base::Unretained(&senders[i]), |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 421 | kMaxValue * i / 4 + 1)); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 422 | } |
| 423 | |
| 424 | run_loop.Run(); |
| 425 | |
| 426 | for (size_t i = 0; i < 4; ++i) { |
| 427 | base::RunLoop run_loop; |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 428 | senders[i].task_runner()->PostTaskAndReply( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 429 | FROM_HERE, |
| 430 | base::Bind(&TestSender::TearDown, base::Unretained(&senders[i])), |
| 431 | base::Bind(&AssociatedInterfaceTest::QuitRunLoop, |
| 432 | base::Unretained(this), base::Unretained(&run_loop))); |
| 433 | run_loop.Run(); |
| 434 | } |
| 435 | |
| 436 | for (size_t i = 0; i < 2; ++i) { |
| 437 | base::RunLoop run_loop; |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 438 | receivers[i].task_runner()->PostTaskAndReply( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 439 | FROM_HERE, |
| 440 | base::Bind(&TestReceiver::TearDown, base::Unretained(&receivers[i])), |
| 441 | base::Bind(&AssociatedInterfaceTest::QuitRunLoop, |
| 442 | base::Unretained(this), base::Unretained(&run_loop))); |
| 443 | run_loop.Run(); |
| 444 | } |
| 445 | |
| 446 | EXPECT_EQ(static_cast<size_t>(kMaxValue / 2), receivers[0].values().size()); |
| 447 | EXPECT_EQ(static_cast<size_t>(kMaxValue / 2), receivers[1].values().size()); |
| 448 | |
| 449 | std::vector<int32_t> all_values; |
| 450 | all_values.insert(all_values.end(), receivers[0].values().begin(), |
| 451 | receivers[0].values().end()); |
| 452 | all_values.insert(all_values.end(), receivers[1].values().begin(), |
| 453 | receivers[1].values().end()); |
| 454 | |
| 455 | std::sort(all_values.begin(), all_values.end()); |
| 456 | for (size_t i = 0; i < all_values.size(); ++i) |
| 457 | ASSERT_EQ(static_cast<int32_t>(i + 1), all_values[i]); |
| 458 | } |
| 459 | |
| 460 | TEST_F(AssociatedInterfaceTest, FIFO) { |
| 461 | // Set up four associated interfaces on a message pipe. Use the inteface |
| 462 | // pointers on four threads; run the interface implementations on two threads. |
| 463 | // Take turns to make calls using the four pointers. Test that FIFO-ness is |
| 464 | // preserved. |
| 465 | |
| 466 | const int32_t kMaxValue = 100; |
| 467 | MessagePipe pipe; |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 468 | scoped_refptr<MultiplexRouter> router0; |
| 469 | scoped_refptr<MultiplexRouter> router1; |
| 470 | CreateRouterPair(&router0, &router1); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 471 | |
| 472 | AssociatedInterfaceRequest<IntegerSender> requests[4]; |
yzshen | 9ce8109 | 2016-01-20 09:42:42 +0900 | [diff] [blame] | 473 | IntegerSenderAssociatedPtrInfo ptr_infos[4]; |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 474 | for (size_t i = 0; i < 4; ++i) { |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 475 | CreateIntegerSenderWithExistingRouters(router1, &ptr_infos[i], router0, |
| 476 | &requests[i]); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 477 | } |
| 478 | |
| 479 | TestSender senders[4]; |
| 480 | for (size_t i = 0; i < 4; ++i) { |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 481 | senders[i].task_runner()->PostTask( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 482 | FROM_HERE, |
| 483 | base::Bind(&TestSender::SetUp, base::Unretained(&senders[i]), |
| 484 | base::Passed(&ptr_infos[i]), |
| 485 | base::Unretained(&senders[(i + 1) % 4]), kMaxValue)); |
| 486 | } |
| 487 | |
| 488 | base::RunLoop run_loop; |
| 489 | TestReceiver receivers[2]; |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 490 | NotificationCounter counter( |
| 491 | 2, base::Bind(&AssociatedInterfaceTest::QuitRunLoop, |
| 492 | base::Unretained(this), base::Unretained(&run_loop))); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 493 | for (size_t i = 0; i < 2; ++i) { |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 494 | receivers[i].task_runner()->PostTask( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 495 | FROM_HERE, |
yzshen | 645103a | 2016-06-01 04:01:04 +0900 | [diff] [blame] | 496 | base::Bind(&TestReceiver::SetUp, base::Unretained(&receivers[i]), |
| 497 | base::Passed(&requests[2 * i]), |
| 498 | base::Passed(&requests[2 * i + 1]), |
| 499 | static_cast<size_t>(kMaxValue / 2), |
| 500 | base::Bind(&NotificationCounter::OnGotNotification, |
| 501 | base::Unretained(&counter)))); |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 502 | } |
| 503 | |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 504 | senders[0].task_runner()->PostTask( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 505 | FROM_HERE, |
| 506 | base::Bind(&TestSender::Send, base::Unretained(&senders[0]), 1)); |
| 507 | |
| 508 | run_loop.Run(); |
| 509 | |
| 510 | for (size_t i = 0; i < 4; ++i) { |
| 511 | base::RunLoop run_loop; |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 512 | senders[i].task_runner()->PostTaskAndReply( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 513 | FROM_HERE, |
| 514 | base::Bind(&TestSender::TearDown, base::Unretained(&senders[i])), |
| 515 | base::Bind(&AssociatedInterfaceTest::QuitRunLoop, |
| 516 | base::Unretained(this), base::Unretained(&run_loop))); |
| 517 | run_loop.Run(); |
| 518 | } |
| 519 | |
| 520 | for (size_t i = 0; i < 2; ++i) { |
| 521 | base::RunLoop run_loop; |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 522 | receivers[i].task_runner()->PostTaskAndReply( |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 523 | FROM_HERE, |
| 524 | base::Bind(&TestReceiver::TearDown, base::Unretained(&receivers[i])), |
| 525 | base::Bind(&AssociatedInterfaceTest::QuitRunLoop, |
| 526 | base::Unretained(this), base::Unretained(&run_loop))); |
| 527 | run_loop.Run(); |
| 528 | } |
| 529 | |
| 530 | EXPECT_EQ(static_cast<size_t>(kMaxValue / 2), receivers[0].values().size()); |
| 531 | EXPECT_EQ(static_cast<size_t>(kMaxValue / 2), receivers[1].values().size()); |
| 532 | |
| 533 | for (size_t i = 0; i < 2; ++i) { |
| 534 | for (size_t j = 1; j < receivers[i].values().size(); ++j) |
| 535 | EXPECT_LT(receivers[i].values()[j - 1], receivers[i].values()[j]); |
| 536 | } |
| 537 | } |
| 538 | |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 539 | void CaptureInt32(int32_t* storage, |
| 540 | const base::Closure& closure, |
| 541 | int32_t value) { |
| 542 | *storage = value; |
| 543 | closure.Run(); |
| 544 | } |
| 545 | |
| 546 | void CaptureSenderPtrInfo(IntegerSenderAssociatedPtr* storage, |
| 547 | const base::Closure& closure, |
| 548 | IntegerSenderAssociatedPtrInfo info) { |
| 549 | storage->Bind(std::move(info)); |
| 550 | closure.Run(); |
| 551 | } |
| 552 | |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 553 | TEST_F(AssociatedInterfaceTest, PassAssociatedInterfaces) { |
| 554 | IntegerSenderConnectionPtr connection_ptr; |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 555 | IntegerSenderConnectionImpl connection(MakeRequest(&connection_ptr)); |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 556 | |
yzshen | 8b050fa | 2015-12-01 07:44:14 +0900 | [diff] [blame] | 557 | IntegerSenderAssociatedPtr sender0; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 558 | connection_ptr->GetSender(MakeRequest(&sender0)); |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 559 | |
| 560 | int32_t echoed_value = 0; |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 561 | base::RunLoop run_loop; |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 562 | sender0->Echo(123, base::Bind(&CaptureInt32, &echoed_value, |
| 563 | run_loop.QuitClosure())); |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 564 | run_loop.Run(); |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 565 | EXPECT_EQ(123, echoed_value); |
| 566 | |
yzshen | 8b050fa | 2015-12-01 07:44:14 +0900 | [diff] [blame] | 567 | IntegerSenderAssociatedPtr sender1; |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 568 | base::RunLoop run_loop2; |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 569 | connection_ptr->AsyncGetSender( |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 570 | base::Bind(&CaptureSenderPtrInfo, &sender1, run_loop2.QuitClosure())); |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 571 | run_loop2.Run(); |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 572 | EXPECT_TRUE(sender1); |
| 573 | |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 574 | base::RunLoop run_loop3; |
rockot | 453673f | 2016-06-19 05:15:51 +0900 | [diff] [blame] | 575 | sender1->Echo(456, base::Bind(&CaptureInt32, &echoed_value, |
| 576 | run_loop3.QuitClosure())); |
jam | c0e79fc | 2016-01-05 09:06:02 +0900 | [diff] [blame] | 577 | run_loop3.Run(); |
yzshen | 50269ab | 2015-11-26 09:31:56 +0900 | [diff] [blame] | 578 | EXPECT_EQ(456, echoed_value); |
| 579 | } |
| 580 | |
yzshen | 09b1228 | 2015-12-11 10:37:50 +0900 | [diff] [blame] | 581 | TEST_F(AssociatedInterfaceTest, BindingWaitAndPauseWhenNoAssociatedInterfaces) { |
| 582 | IntegerSenderConnectionPtr connection_ptr; |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 583 | IntegerSenderConnectionImpl connection(MakeRequest(&connection_ptr)); |
yzshen | 09b1228 | 2015-12-11 10:37:50 +0900 | [diff] [blame] | 584 | |
| 585 | IntegerSenderAssociatedPtr sender0; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 586 | connection_ptr->GetSender(MakeRequest(&sender0)); |
yzshen | 09b1228 | 2015-12-11 10:37:50 +0900 | [diff] [blame] | 587 | |
| 588 | EXPECT_FALSE(connection.binding()->HasAssociatedInterfaces()); |
| 589 | // There are no associated interfaces running on the pipe yet. It is okay to |
| 590 | // pause. |
| 591 | connection.binding()->PauseIncomingMethodCallProcessing(); |
| 592 | connection.binding()->ResumeIncomingMethodCallProcessing(); |
| 593 | |
| 594 | // There are no associated interfaces running on the pipe yet. It is okay to |
| 595 | // wait. |
| 596 | EXPECT_TRUE(connection.binding()->WaitForIncomingMethodCall()); |
| 597 | |
| 598 | // The previous wait has dispatched the GetSender request message, therefore |
| 599 | // an associated interface has been set up on the pipe. It is not allowed to |
| 600 | // wait or pause. |
| 601 | EXPECT_TRUE(connection.binding()->HasAssociatedInterfaces()); |
| 602 | } |
| 603 | |
rockot | cffd1b2 | 2016-08-25 08:37:11 +0900 | [diff] [blame] | 604 | class PingServiceImpl : public PingService { |
| 605 | public: |
| 606 | explicit PingServiceImpl(PingServiceAssociatedRequest request) |
| 607 | : binding_(this, std::move(request)) {} |
| 608 | ~PingServiceImpl() override {} |
| 609 | |
| 610 | AssociatedBinding<PingService>& binding() { return binding_; } |
| 611 | |
| 612 | void set_ping_handler(const base::Closure& handler) { |
| 613 | ping_handler_ = handler; |
| 614 | } |
| 615 | |
| 616 | // PingService: |
| 617 | void Ping(const PingCallback& callback) override { |
| 618 | if (!ping_handler_.is_null()) |
| 619 | ping_handler_.Run(); |
| 620 | callback.Run(); |
| 621 | } |
| 622 | |
| 623 | private: |
| 624 | AssociatedBinding<PingService> binding_; |
| 625 | base::Closure ping_handler_; |
| 626 | }; |
| 627 | |
| 628 | class PingProviderImpl : public AssociatedPingProvider { |
| 629 | public: |
| 630 | explicit PingProviderImpl(AssociatedPingProviderRequest request) |
| 631 | : binding_(this, std::move(request)) {} |
| 632 | ~PingProviderImpl() override {} |
| 633 | |
| 634 | // AssociatedPingProvider: |
| 635 | void GetPing(PingServiceAssociatedRequest request) override { |
| 636 | ping_services_.emplace_back(new PingServiceImpl(std::move(request))); |
| 637 | |
| 638 | if (expected_bindings_count_ > 0 && |
| 639 | ping_services_.size() == expected_bindings_count_ && |
| 640 | !quit_waiting_.is_null()) { |
| 641 | expected_bindings_count_ = 0; |
| 642 | base::ResetAndReturn(&quit_waiting_).Run(); |
| 643 | } |
| 644 | } |
| 645 | |
| 646 | std::vector<std::unique_ptr<PingServiceImpl>>& ping_services() { |
| 647 | return ping_services_; |
| 648 | } |
| 649 | |
| 650 | void WaitForBindings(size_t count) { |
| 651 | DCHECK(quit_waiting_.is_null()); |
| 652 | |
| 653 | expected_bindings_count_ = count; |
| 654 | base::RunLoop loop; |
| 655 | quit_waiting_ = loop.QuitClosure(); |
| 656 | loop.Run(); |
| 657 | } |
| 658 | |
| 659 | private: |
| 660 | Binding<AssociatedPingProvider> binding_; |
| 661 | std::vector<std::unique_ptr<PingServiceImpl>> ping_services_; |
| 662 | size_t expected_bindings_count_ = 0; |
| 663 | base::Closure quit_waiting_; |
| 664 | }; |
| 665 | |
| 666 | class CallbackFilter : public MessageReceiver { |
| 667 | public: |
| 668 | explicit CallbackFilter(const base::Closure& callback) |
| 669 | : callback_(callback) {} |
| 670 | ~CallbackFilter() override {} |
| 671 | |
| 672 | static std::unique_ptr<CallbackFilter> Wrap(const base::Closure& callback) { |
| 673 | return base::MakeUnique<CallbackFilter>(callback); |
| 674 | } |
| 675 | |
| 676 | // MessageReceiver: |
| 677 | bool Accept(Message* message) override { |
| 678 | callback_.Run(); |
| 679 | return true; |
| 680 | } |
| 681 | |
| 682 | private: |
| 683 | const base::Closure callback_; |
| 684 | }; |
| 685 | |
| 686 | // Verifies that filters work as expected on associated bindings, i.e. that |
| 687 | // they're notified in order, before dispatch; and that each associated |
| 688 | // binding in a group operates with its own set of filters. |
| 689 | TEST_F(AssociatedInterfaceTest, BindingWithFilters) { |
| 690 | AssociatedPingProviderPtr provider; |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 691 | PingProviderImpl provider_impl(MakeRequest(&provider)); |
rockot | cffd1b2 | 2016-08-25 08:37:11 +0900 | [diff] [blame] | 692 | |
| 693 | PingServiceAssociatedPtr ping_a, ping_b; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 694 | provider->GetPing(MakeRequest(&ping_a)); |
| 695 | provider->GetPing(MakeRequest(&ping_b)); |
rockot | cffd1b2 | 2016-08-25 08:37:11 +0900 | [diff] [blame] | 696 | provider_impl.WaitForBindings(2); |
| 697 | |
| 698 | ASSERT_EQ(2u, provider_impl.ping_services().size()); |
| 699 | PingServiceImpl& ping_a_impl = *provider_impl.ping_services()[0]; |
| 700 | PingServiceImpl& ping_b_impl = *provider_impl.ping_services()[1]; |
| 701 | |
| 702 | int a_status, b_status; |
| 703 | auto handler_helper = [] (int* a_status, int* b_status, int expected_a_status, |
| 704 | int new_a_status, int expected_b_status, |
| 705 | int new_b_status) { |
| 706 | EXPECT_EQ(expected_a_status, *a_status); |
| 707 | EXPECT_EQ(expected_b_status, *b_status); |
| 708 | *a_status = new_a_status; |
| 709 | *b_status = new_b_status; |
| 710 | }; |
| 711 | auto create_handler = [&] (int expected_a_status, int new_a_status, |
| 712 | int expected_b_status, int new_b_status) { |
| 713 | return base::Bind(handler_helper, &a_status, &b_status, expected_a_status, |
| 714 | new_a_status, expected_b_status, new_b_status); |
| 715 | }; |
| 716 | |
| 717 | ping_a_impl.binding().AddFilter( |
| 718 | CallbackFilter::Wrap(create_handler(0, 1, 0, 0))); |
| 719 | ping_a_impl.binding().AddFilter( |
| 720 | CallbackFilter::Wrap(create_handler(1, 2, 0, 0))); |
| 721 | ping_a_impl.set_ping_handler(create_handler(2, 3, 0, 0)); |
| 722 | |
| 723 | ping_b_impl.binding().AddFilter( |
| 724 | CallbackFilter::Wrap(create_handler(3, 3, 0, 1))); |
| 725 | ping_b_impl.binding().AddFilter( |
| 726 | CallbackFilter::Wrap(create_handler(3, 3, 1, 2))); |
| 727 | ping_b_impl.set_ping_handler(create_handler(3, 3, 2, 3)); |
| 728 | |
| 729 | for (int i = 0; i < 10; ++i) { |
| 730 | a_status = 0; |
| 731 | b_status = 0; |
| 732 | |
| 733 | { |
| 734 | base::RunLoop loop; |
| 735 | ping_a->Ping(loop.QuitClosure()); |
| 736 | loop.Run(); |
| 737 | } |
| 738 | |
| 739 | EXPECT_EQ(3, a_status); |
| 740 | EXPECT_EQ(0, b_status); |
| 741 | |
| 742 | { |
| 743 | base::RunLoop loop; |
| 744 | ping_b->Ping(loop.QuitClosure()); |
| 745 | loop.Run(); |
| 746 | } |
| 747 | |
| 748 | EXPECT_EQ(3, a_status); |
| 749 | EXPECT_EQ(3, b_status); |
| 750 | } |
| 751 | } |
| 752 | |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 753 | TEST_F(AssociatedInterfaceTest, AssociatedPtrFlushForTesting) { |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 754 | AssociatedInterfaceRequest<IntegerSender> request; |
| 755 | IntegerSenderAssociatedPtrInfo ptr_info; |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 756 | CreateIntegerSender(&ptr_info, &request); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 757 | |
| 758 | IntegerSenderImpl impl0(std::move(request)); |
| 759 | AssociatedInterfacePtr<IntegerSender> ptr0; |
| 760 | ptr0.Bind(std::move(ptr_info)); |
| 761 | ptr0.set_connection_error_handler(base::Bind(&Fail)); |
| 762 | |
| 763 | bool ptr0_callback_run = false; |
| 764 | ptr0->Echo(123, ExpectValueSetFlagAndRunClosure( |
| 765 | 123, &ptr0_callback_run, base::Bind(&base::DoNothing))); |
| 766 | ptr0.FlushForTesting(); |
| 767 | EXPECT_TRUE(ptr0_callback_run); |
| 768 | } |
| 769 | |
| 770 | void SetBool(bool* value) { |
| 771 | *value = true; |
| 772 | } |
| 773 | |
| 774 | template <typename T> |
| 775 | void SetBoolWithUnusedParameter(bool* value, T unused) { |
| 776 | *value = true; |
| 777 | } |
| 778 | |
| 779 | TEST_F(AssociatedInterfaceTest, AssociatedPtrFlushForTestingWithClosedPeer) { |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 780 | AssociatedInterfaceRequest<IntegerSender> request; |
| 781 | IntegerSenderAssociatedPtrInfo ptr_info; |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 782 | CreateIntegerSender(&ptr_info, &request); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 783 | |
| 784 | AssociatedInterfacePtr<IntegerSender> ptr0; |
| 785 | ptr0.Bind(std::move(ptr_info)); |
| 786 | bool called = false; |
| 787 | ptr0.set_connection_error_handler(base::Bind(&SetBool, &called)); |
| 788 | request = nullptr; |
| 789 | |
| 790 | ptr0.FlushForTesting(); |
| 791 | EXPECT_TRUE(called); |
| 792 | ptr0.FlushForTesting(); |
| 793 | } |
| 794 | |
| 795 | TEST_F(AssociatedInterfaceTest, AssociatedBindingFlushForTesting) { |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 796 | AssociatedInterfaceRequest<IntegerSender> request; |
| 797 | IntegerSenderAssociatedPtrInfo ptr_info; |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 798 | CreateIntegerSender(&ptr_info, &request); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 799 | |
| 800 | IntegerSenderImpl impl0(std::move(request)); |
| 801 | impl0.set_connection_error_handler(base::Bind(&Fail)); |
| 802 | AssociatedInterfacePtr<IntegerSender> ptr0; |
| 803 | ptr0.Bind(std::move(ptr_info)); |
| 804 | |
| 805 | bool ptr0_callback_run = false; |
| 806 | ptr0->Echo(123, ExpectValueSetFlagAndRunClosure( |
| 807 | 123, &ptr0_callback_run, base::Bind(&base::DoNothing))); |
| 808 | // Because the flush is sent from the binding, it only guarantees that the |
| 809 | // request has been received, not the response. The second flush waits for the |
| 810 | // response to be received. |
| 811 | impl0.binding()->FlushForTesting(); |
| 812 | impl0.binding()->FlushForTesting(); |
| 813 | EXPECT_TRUE(ptr0_callback_run); |
| 814 | } |
| 815 | |
| 816 | TEST_F(AssociatedInterfaceTest, |
| 817 | AssociatedBindingFlushForTestingWithClosedPeer) { |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 818 | scoped_refptr<MultiplexRouter> router0; |
| 819 | scoped_refptr<MultiplexRouter> router1; |
| 820 | CreateRouterPair(&router0, &router1); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 821 | |
| 822 | AssociatedInterfaceRequest<IntegerSender> request; |
| 823 | { |
| 824 | IntegerSenderAssociatedPtrInfo ptr_info; |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 825 | CreateIntegerSenderWithExistingRouters(router1, &ptr_info, router0, |
| 826 | &request); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 827 | } |
| 828 | |
| 829 | IntegerSenderImpl impl(std::move(request)); |
| 830 | bool called = false; |
| 831 | impl.set_connection_error_handler(base::Bind(&SetBool, &called)); |
| 832 | impl.binding()->FlushForTesting(); |
| 833 | EXPECT_TRUE(called); |
| 834 | impl.binding()->FlushForTesting(); |
| 835 | } |
| 836 | |
| 837 | TEST_F(AssociatedInterfaceTest, BindingFlushForTesting) { |
| 838 | IntegerSenderConnectionPtr ptr; |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 839 | IntegerSenderConnectionImpl impl(MakeRequest(&ptr)); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 840 | bool called = false; |
| 841 | ptr->AsyncGetSender(base::Bind( |
| 842 | &SetBoolWithUnusedParameter<IntegerSenderAssociatedPtrInfo>, &called)); |
| 843 | EXPECT_FALSE(called); |
| 844 | impl.binding()->set_connection_error_handler(base::Bind(&Fail)); |
| 845 | // Because the flush is sent from the binding, it only guarantees that the |
| 846 | // request has been received, not the response. The second flush waits for the |
| 847 | // response to be received. |
| 848 | impl.binding()->FlushForTesting(); |
| 849 | impl.binding()->FlushForTesting(); |
| 850 | EXPECT_TRUE(called); |
| 851 | } |
| 852 | |
| 853 | TEST_F(AssociatedInterfaceTest, BindingFlushForTestingWithClosedPeer) { |
| 854 | IntegerSenderConnectionPtr ptr; |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 855 | IntegerSenderConnectionImpl impl(MakeRequest(&ptr)); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 856 | bool called = false; |
| 857 | impl.binding()->set_connection_error_handler(base::Bind(&SetBool, &called)); |
| 858 | ptr.reset(); |
| 859 | EXPECT_FALSE(called); |
| 860 | impl.binding()->FlushForTesting(); |
| 861 | EXPECT_TRUE(called); |
| 862 | impl.binding()->FlushForTesting(); |
| 863 | } |
| 864 | |
| 865 | TEST_F(AssociatedInterfaceTest, StrongBindingFlushForTesting) { |
| 866 | IntegerSenderConnectionPtr ptr; |
rockot | 23ad5fc | 2016-09-13 09:48:21 +0900 | [diff] [blame] | 867 | auto binding = |
| 868 | MakeStrongBinding(base::MakeUnique<IntegerSenderConnectionImpl>( |
| 869 | IntegerSenderConnectionRequest{}), |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 870 | MakeRequest(&ptr)); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 871 | bool called = false; |
| 872 | IntegerSenderAssociatedPtr sender_ptr; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 873 | ptr->GetSender(MakeRequest(&sender_ptr)); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 874 | sender_ptr->Echo(1, base::Bind(&SetBoolWithUnusedParameter<int>, &called)); |
| 875 | EXPECT_FALSE(called); |
| 876 | // Because the flush is sent from the binding, it only guarantees that the |
| 877 | // request has been received, not the response. The second flush waits for the |
| 878 | // response to be received. |
rockot | 23ad5fc | 2016-09-13 09:48:21 +0900 | [diff] [blame] | 879 | ASSERT_TRUE(binding); |
| 880 | binding->FlushForTesting(); |
| 881 | ASSERT_TRUE(binding); |
| 882 | binding->FlushForTesting(); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 883 | EXPECT_TRUE(called); |
| 884 | } |
| 885 | |
| 886 | TEST_F(AssociatedInterfaceTest, StrongBindingFlushForTestingWithClosedPeer) { |
| 887 | IntegerSenderConnectionPtr ptr; |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 888 | bool called = false; |
rockot | 23ad5fc | 2016-09-13 09:48:21 +0900 | [diff] [blame] | 889 | auto binding = |
| 890 | MakeStrongBinding(base::MakeUnique<IntegerSenderConnectionImpl>( |
| 891 | IntegerSenderConnectionRequest{}), |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 892 | MakeRequest(&ptr)); |
rockot | 23ad5fc | 2016-09-13 09:48:21 +0900 | [diff] [blame] | 893 | binding->set_connection_error_handler(base::Bind(&SetBool, &called)); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 894 | ptr.reset(); |
| 895 | EXPECT_FALSE(called); |
rockot | 23ad5fc | 2016-09-13 09:48:21 +0900 | [diff] [blame] | 896 | ASSERT_TRUE(binding); |
| 897 | binding->FlushForTesting(); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 898 | EXPECT_TRUE(called); |
rockot | 23ad5fc | 2016-09-13 09:48:21 +0900 | [diff] [blame] | 899 | ASSERT_FALSE(binding); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 900 | } |
| 901 | |
| 902 | TEST_F(AssociatedInterfaceTest, PtrFlushForTesting) { |
| 903 | IntegerSenderConnectionPtr ptr; |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 904 | IntegerSenderConnectionImpl impl(MakeRequest(&ptr)); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 905 | bool called = false; |
| 906 | ptr.set_connection_error_handler(base::Bind(&Fail)); |
| 907 | ptr->AsyncGetSender(base::Bind( |
| 908 | &SetBoolWithUnusedParameter<IntegerSenderAssociatedPtrInfo>, &called)); |
| 909 | EXPECT_FALSE(called); |
| 910 | ptr.FlushForTesting(); |
| 911 | EXPECT_TRUE(called); |
| 912 | } |
| 913 | |
| 914 | TEST_F(AssociatedInterfaceTest, PtrFlushForTestingWithClosedPeer) { |
| 915 | IntegerSenderConnectionPtr ptr; |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 916 | MakeRequest(&ptr); |
sammc | 7058890 | 2016-09-01 18:52:58 +0900 | [diff] [blame] | 917 | bool called = false; |
| 918 | ptr.set_connection_error_handler(base::Bind(&SetBool, &called)); |
| 919 | EXPECT_FALSE(called); |
| 920 | ptr.FlushForTesting(); |
| 921 | EXPECT_TRUE(called); |
| 922 | ptr.FlushForTesting(); |
| 923 | } |
| 924 | |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 925 | TEST_F(AssociatedInterfaceTest, AssociatedBindingConnectionErrorWithReason) { |
| 926 | AssociatedInterfaceRequest<IntegerSender> request; |
| 927 | IntegerSenderAssociatedPtrInfo ptr_info; |
| 928 | CreateIntegerSender(&ptr_info, &request); |
| 929 | |
| 930 | IntegerSenderImpl impl(std::move(request)); |
| 931 | AssociatedInterfacePtr<IntegerSender> ptr; |
| 932 | ptr.Bind(std::move(ptr_info)); |
| 933 | |
| 934 | base::RunLoop run_loop; |
| 935 | impl.binding()->set_connection_error_with_reason_handler(base::Bind( |
| 936 | [](const base::Closure& quit_closure, uint32_t custom_reason, |
| 937 | const std::string& description) { |
| 938 | EXPECT_EQ(123u, custom_reason); |
| 939 | EXPECT_EQ("farewell", description); |
| 940 | quit_closure.Run(); |
| 941 | }, |
| 942 | run_loop.QuitClosure())); |
| 943 | |
| 944 | ptr.ResetWithReason(123u, "farewell"); |
| 945 | |
| 946 | run_loop.Run(); |
| 947 | } |
| 948 | |
yzshen | 8dc7bd7 | 2017-02-15 07:24:25 +0900 | [diff] [blame] | 949 | TEST_F(AssociatedInterfaceTest, |
| 950 | PendingAssociatedBindingConnectionErrorWithReason) { |
| 951 | // Test that AssociatedBinding is notified with connection error when the |
| 952 | // interface hasn't associated with a message pipe and the peer is closed. |
| 953 | |
yzshen | 8dc7bd7 | 2017-02-15 07:24:25 +0900 | [diff] [blame] | 954 | IntegerSenderAssociatedPtr ptr; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 955 | IntegerSenderImpl impl(MakeRequest(&ptr)); |
yzshen | 8dc7bd7 | 2017-02-15 07:24:25 +0900 | [diff] [blame] | 956 | |
| 957 | base::RunLoop run_loop; |
| 958 | impl.binding()->set_connection_error_with_reason_handler(base::Bind( |
| 959 | [](const base::Closure& quit_closure, uint32_t custom_reason, |
| 960 | const std::string& description) { |
| 961 | EXPECT_EQ(123u, custom_reason); |
| 962 | EXPECT_EQ("farewell", description); |
| 963 | quit_closure.Run(); |
| 964 | }, |
| 965 | run_loop.QuitClosure())); |
| 966 | |
| 967 | ptr.ResetWithReason(123u, "farewell"); |
| 968 | |
| 969 | run_loop.Run(); |
| 970 | } |
| 971 | |
yzshen | 626de04 | 2016-09-14 02:03:53 +0900 | [diff] [blame] | 972 | TEST_F(AssociatedInterfaceTest, AssociatedPtrConnectionErrorWithReason) { |
| 973 | AssociatedInterfaceRequest<IntegerSender> request; |
| 974 | IntegerSenderAssociatedPtrInfo ptr_info; |
| 975 | CreateIntegerSender(&ptr_info, &request); |
| 976 | |
| 977 | IntegerSenderImpl impl(std::move(request)); |
| 978 | AssociatedInterfacePtr<IntegerSender> ptr; |
| 979 | ptr.Bind(std::move(ptr_info)); |
| 980 | |
| 981 | base::RunLoop run_loop; |
| 982 | ptr.set_connection_error_with_reason_handler(base::Bind( |
| 983 | [](const base::Closure& quit_closure, uint32_t custom_reason, |
| 984 | const std::string& description) { |
| 985 | EXPECT_EQ(456u, custom_reason); |
| 986 | EXPECT_EQ("farewell", description); |
| 987 | quit_closure.Run(); |
| 988 | }, |
| 989 | run_loop.QuitClosure())); |
| 990 | |
| 991 | impl.binding()->CloseWithReason(456u, "farewell"); |
| 992 | |
| 993 | run_loop.Run(); |
| 994 | } |
| 995 | |
yzshen | 8dc7bd7 | 2017-02-15 07:24:25 +0900 | [diff] [blame] | 996 | TEST_F(AssociatedInterfaceTest, PendingAssociatedPtrConnectionErrorWithReason) { |
| 997 | // Test that AssociatedInterfacePtr is notified with connection error when the |
| 998 | // interface hasn't associated with a message pipe and the peer is closed. |
| 999 | |
yzshen | 8dc7bd7 | 2017-02-15 07:24:25 +0900 | [diff] [blame] | 1000 | IntegerSenderAssociatedPtr ptr; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 1001 | auto request = MakeRequest(&ptr); |
yzshen | 8dc7bd7 | 2017-02-15 07:24:25 +0900 | [diff] [blame] | 1002 | |
| 1003 | base::RunLoop run_loop; |
| 1004 | ptr.set_connection_error_with_reason_handler(base::Bind( |
| 1005 | [](const base::Closure& quit_closure, uint32_t custom_reason, |
| 1006 | const std::string& description) { |
| 1007 | EXPECT_EQ(456u, custom_reason); |
| 1008 | EXPECT_EQ("farewell", description); |
| 1009 | quit_closure.Run(); |
| 1010 | }, |
| 1011 | run_loop.QuitClosure())); |
| 1012 | |
| 1013 | request.ResetWithReason(456u, "farewell"); |
| 1014 | |
| 1015 | run_loop.Run(); |
| 1016 | } |
| 1017 | |
yzshen | 7f47672 | 2016-09-14 06:50:07 +0900 | [diff] [blame] | 1018 | TEST_F(AssociatedInterfaceTest, AssociatedRequestResetWithReason) { |
| 1019 | AssociatedInterfaceRequest<IntegerSender> request; |
| 1020 | IntegerSenderAssociatedPtrInfo ptr_info; |
| 1021 | CreateIntegerSender(&ptr_info, &request); |
| 1022 | |
| 1023 | AssociatedInterfacePtr<IntegerSender> ptr; |
| 1024 | ptr.Bind(std::move(ptr_info)); |
| 1025 | |
| 1026 | base::RunLoop run_loop; |
| 1027 | ptr.set_connection_error_with_reason_handler(base::Bind( |
| 1028 | [](const base::Closure& quit_closure, uint32_t custom_reason, |
| 1029 | const std::string& description) { |
| 1030 | EXPECT_EQ(789u, custom_reason); |
| 1031 | EXPECT_EQ("long time no see", description); |
| 1032 | quit_closure.Run(); |
| 1033 | }, |
| 1034 | run_loop.QuitClosure())); |
| 1035 | |
| 1036 | request.ResetWithReason(789u, "long time no see"); |
| 1037 | |
| 1038 | run_loop.Run(); |
| 1039 | } |
| 1040 | |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 1041 | TEST_F(AssociatedInterfaceTest, ThreadSafeAssociatedInterfacePtr) { |
| 1042 | IntegerSenderConnectionPtr connection_ptr; |
blundell | 984c9e4 | 2016-12-20 20:23:32 +0900 | [diff] [blame] | 1043 | IntegerSenderConnectionImpl connection(MakeRequest(&connection_ptr)); |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 1044 | |
| 1045 | IntegerSenderAssociatedPtr sender; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 1046 | connection_ptr->GetSender(MakeRequest(&sender)); |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 1047 | |
| 1048 | scoped_refptr<ThreadSafeIntegerSenderAssociatedPtr> thread_safe_sender = |
| 1049 | ThreadSafeIntegerSenderAssociatedPtr::Create(std::move(sender)); |
| 1050 | |
| 1051 | { |
| 1052 | // Test the thread safe pointer can be used from the interface ptr thread. |
| 1053 | int32_t echoed_value = 0; |
| 1054 | base::RunLoop run_loop; |
| 1055 | (*thread_safe_sender) |
| 1056 | ->Echo(123, base::Bind(&CaptureInt32, &echoed_value, |
| 1057 | run_loop.QuitClosure())); |
| 1058 | run_loop.Run(); |
| 1059 | EXPECT_EQ(123, echoed_value); |
| 1060 | } |
| 1061 | |
| 1062 | // Test the thread safe pointer can be used from another thread. |
| 1063 | base::RunLoop run_loop; |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 1064 | |
| 1065 | auto run_method = base::Bind( |
| 1066 | [](const scoped_refptr<base::TaskRunner>& main_task_runner, |
| 1067 | const base::Closure& quit_closure, |
| 1068 | const scoped_refptr<ThreadSafeIntegerSenderAssociatedPtr>& |
| 1069 | thread_safe_sender) { |
| 1070 | auto done_callback = base::Bind( |
| 1071 | [](const scoped_refptr<base::TaskRunner>& main_task_runner, |
| 1072 | const base::Closure& quit_closure, |
Scott Graham | 052687f | 2017-08-09 00:19:01 +0900 | [diff] [blame^] | 1073 | scoped_refptr<base::SequencedTaskRunner> sender_sequence_runner, |
| 1074 | int32_t result) { |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 1075 | EXPECT_EQ(123, result); |
Scott Graham | 052687f | 2017-08-09 00:19:01 +0900 | [diff] [blame^] | 1076 | // Validate the callback is invoked on the calling sequence. |
| 1077 | EXPECT_TRUE(sender_sequence_runner->RunsTasksInCurrentSequence()); |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 1078 | // Notify the run_loop to quit. |
| 1079 | main_task_runner->PostTask(FROM_HERE, quit_closure); |
| 1080 | }); |
Scott Graham | 052687f | 2017-08-09 00:19:01 +0900 | [diff] [blame^] | 1081 | scoped_refptr<base::SequencedTaskRunner> current_sequence_runner = |
| 1082 | base::SequencedTaskRunnerHandle::Get(); |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 1083 | (*thread_safe_sender) |
Scott Graham | 052687f | 2017-08-09 00:19:01 +0900 | [diff] [blame^] | 1084 | ->Echo(123, base::Bind(done_callback, main_task_runner, |
| 1085 | quit_closure, current_sequence_runner)); |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 1086 | }, |
| 1087 | base::SequencedTaskRunnerHandle::Get(), run_loop.QuitClosure(), |
| 1088 | thread_safe_sender); |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 1089 | base::CreateSequencedTaskRunnerWithTraits({})->PostTask(FROM_HERE, |
| 1090 | run_method); |
jcivelli | 9027ad7 | 2016-11-18 12:22:41 +0900 | [diff] [blame] | 1091 | |
| 1092 | // Block until the method callback is called on the background thread. |
| 1093 | run_loop.Run(); |
| 1094 | } |
| 1095 | |
jcivelli | 4e08bca | 2016-11-30 01:15:05 +0900 | [diff] [blame] | 1096 | struct ForwarderTestContext { |
| 1097 | IntegerSenderConnectionPtr connection_ptr; |
| 1098 | std::unique_ptr<IntegerSenderConnectionImpl> interface_impl; |
rockot | 7900d34 | 2017-02-09 17:40:15 +0900 | [diff] [blame] | 1099 | IntegerSenderAssociatedRequest sender_request; |
jcivelli | 4e08bca | 2016-11-30 01:15:05 +0900 | [diff] [blame] | 1100 | }; |
| 1101 | |
rockot | 7900d34 | 2017-02-09 17:40:15 +0900 | [diff] [blame] | 1102 | TEST_F(AssociatedInterfaceTest, |
| 1103 | ThreadSafeAssociatedInterfacePtrWithTaskRunner) { |
Sam McNally | 7854eed | 2017-06-20 08:34:45 +0900 | [diff] [blame] | 1104 | const scoped_refptr<base::SequencedTaskRunner> other_thread_task_runner = |
| 1105 | base::CreateSequencedTaskRunnerWithTraits({}); |
jcivelli | 4e08bca | 2016-11-30 01:15:05 +0900 | [diff] [blame] | 1106 | |
rockot | 7900d34 | 2017-02-09 17:40:15 +0900 | [diff] [blame] | 1107 | ForwarderTestContext* context = new ForwarderTestContext(); |
| 1108 | IntegerSenderAssociatedPtrInfo sender_info; |
| 1109 | base::WaitableEvent sender_info_bound_event( |
jcivelli | 3907bec | 2017-01-07 03:23:01 +0900 | [diff] [blame] | 1110 | base::WaitableEvent::ResetPolicy::MANUAL, |
| 1111 | base::WaitableEvent::InitialState::NOT_SIGNALED); |
rockot | 7900d34 | 2017-02-09 17:40:15 +0900 | [diff] [blame] | 1112 | auto setup = [](base::WaitableEvent* sender_info_bound_event, |
| 1113 | IntegerSenderAssociatedPtrInfo* sender_info, |
| 1114 | ForwarderTestContext* context) { |
| 1115 | context->interface_impl = base::MakeUnique<IntegerSenderConnectionImpl>( |
| 1116 | MakeRequest(&context->connection_ptr)); |
jcivelli | 3907bec | 2017-01-07 03:23:01 +0900 | [diff] [blame] | 1117 | |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 1118 | auto sender_request = MakeRequest(sender_info); |
rockot | 7900d34 | 2017-02-09 17:40:15 +0900 | [diff] [blame] | 1119 | context->connection_ptr->GetSender(std::move(sender_request)); |
| 1120 | |
| 1121 | // Unblock the main thread as soon as |sender_info| is set. |
| 1122 | sender_info_bound_event->Signal(); |
| 1123 | }; |
| 1124 | other_thread_task_runner->PostTask( |
| 1125 | FROM_HERE, |
| 1126 | base::Bind(setup, &sender_info_bound_event, &sender_info, context)); |
| 1127 | sender_info_bound_event.Wait(); |
| 1128 | |
| 1129 | // Create a ThreadSafeAssociatedPtr that binds on the background thread and is |
| 1130 | // associated with |connection_ptr| there. |
jcivelli | 3907bec | 2017-01-07 03:23:01 +0900 | [diff] [blame] | 1131 | scoped_refptr<ThreadSafeIntegerSenderAssociatedPtr> thread_safe_ptr = |
rockot | 7900d34 | 2017-02-09 17:40:15 +0900 | [diff] [blame] | 1132 | ThreadSafeIntegerSenderAssociatedPtr::Create(std::move(sender_info), |
| 1133 | other_thread_task_runner); |
jcivelli | 3907bec | 2017-01-07 03:23:01 +0900 | [diff] [blame] | 1134 | |
rockot | 7900d34 | 2017-02-09 17:40:15 +0900 | [diff] [blame] | 1135 | // Issue a call on the thread-safe ptr immediately. Note that this may happen |
| 1136 | // before the interface is bound on the background thread, and that must be |
| 1137 | // OK. |
jcivelli | 4e08bca | 2016-11-30 01:15:05 +0900 | [diff] [blame] | 1138 | { |
jcivelli | 4e08bca | 2016-11-30 01:15:05 +0900 | [diff] [blame] | 1139 | auto echo_callback = |
| 1140 | base::Bind([](const base::Closure& quit_closure, int32_t result) { |
| 1141 | EXPECT_EQ(123, result); |
| 1142 | quit_closure.Run(); |
| 1143 | }); |
| 1144 | base::RunLoop run_loop; |
| 1145 | (*thread_safe_ptr) |
| 1146 | ->Echo(123, base::Bind(echo_callback, run_loop.QuitClosure())); |
jcivelli | 3907bec | 2017-01-07 03:23:01 +0900 | [diff] [blame] | 1147 | |
jcivelli | 4e08bca | 2016-11-30 01:15:05 +0900 | [diff] [blame] | 1148 | // Block until the method callback is called. |
| 1149 | run_loop.Run(); |
| 1150 | } |
| 1151 | |
| 1152 | other_thread_task_runner->DeleteSoon(FROM_HERE, context); |
| 1153 | |
| 1154 | // Reset the pointer now so the InterfacePtr associated resources can be |
| 1155 | // deleted before the background thread's message loop is invalidated. |
| 1156 | thread_safe_ptr = nullptr; |
| 1157 | } |
| 1158 | |
sammc | 92d4ac6 | 2017-02-04 10:33:38 +0900 | [diff] [blame] | 1159 | class DiscardingAssociatedPingProviderProvider |
| 1160 | : public AssociatedPingProviderProvider { |
| 1161 | public: |
| 1162 | void GetPingProvider( |
| 1163 | AssociatedPingProviderAssociatedRequest request) override {} |
| 1164 | }; |
| 1165 | |
| 1166 | TEST_F(AssociatedInterfaceTest, CloseWithoutBindingAssociatedRequest) { |
| 1167 | DiscardingAssociatedPingProviderProvider ping_provider_provider; |
| 1168 | mojo::Binding<AssociatedPingProviderProvider> binding( |
| 1169 | &ping_provider_provider); |
Ken Rockot | c87a0fd | 2017-06-08 02:35:19 +0900 | [diff] [blame] | 1170 | AssociatedPingProviderProviderPtr provider_provider; |
| 1171 | binding.Bind(mojo::MakeRequest(&provider_provider)); |
sammc | 92d4ac6 | 2017-02-04 10:33:38 +0900 | [diff] [blame] | 1172 | AssociatedPingProviderAssociatedPtr provider; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 1173 | provider_provider->GetPingProvider(mojo::MakeRequest(&provider)); |
sammc | 92d4ac6 | 2017-02-04 10:33:38 +0900 | [diff] [blame] | 1174 | PingServiceAssociatedPtr ping; |
yzshen | e968d28 | 2017-02-16 08:22:41 +0900 | [diff] [blame] | 1175 | provider->GetPing(mojo::MakeRequest(&ping)); |
sammc | 92d4ac6 | 2017-02-04 10:33:38 +0900 | [diff] [blame] | 1176 | base::RunLoop run_loop; |
| 1177 | ping.set_connection_error_handler(run_loop.QuitClosure()); |
| 1178 | run_loop.Run(); |
| 1179 | } |
| 1180 | |
yzshen | 91dd464 | 2017-03-25 17:27:17 +0900 | [diff] [blame] | 1181 | TEST_F(AssociatedInterfaceTest, GetIsolatedInterface) { |
| 1182 | IntegerSenderAssociatedPtr sender; |
| 1183 | GetIsolatedInterface(MakeRequest(&sender).PassHandle()); |
| 1184 | sender->Send(42); |
| 1185 | } |
| 1186 | |
yzshen | 37b7abd | 2015-11-25 01:43:17 +0900 | [diff] [blame] | 1187 | } // namespace |
| 1188 | } // namespace test |
| 1189 | } // namespace mojo |