blob: b1d24017b2658b4b2e3abebe3712ad3becbd4796 [file] [log] [blame]
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001// Copyright (c) 2012 The Chromium Authors. All rights reserved.
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09002// Use of this source code is governed by a BSD-style license that can be
3// found in the LICENSE file.
4//
5// Unit test for SyncChannel.
6
dmaclach@chromium.orgc1d3d422010-12-20 15:59:23 +09007#include "ipc/ipc_sync_channel.h"
8
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09009#include <string>
10#include <vector>
11
12#include "base/basictypes.h"
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +090013#include "base/bind.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090014#include "base/logging.h"
levin@chromium.org5c528682011-03-28 10:54:15 +090015#include "base/memory/scoped_ptr.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090016#include "base/message_loop.h"
jschuh@chromium.orga5cd0762012-04-05 11:38:34 +090017#include "base/process_util.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090018#include "base/string_util.h"
brettw@google.com7c5cc672011-01-01 11:17:08 +090019#include "base/threading/platform_thread.h"
brettw@chromium.org5b5f5e02011-01-01 10:01:06 +090020#include "base/threading/thread.h"
brettw@chromium.org5238c7d2011-01-02 15:05:39 +090021#include "base/synchronization/waitable_event.h"
brettw@chromium.orgf947ed02012-06-12 07:35:26 +090022#include "ipc/ipc_listener.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090023#include "ipc/ipc_message.h"
brettw@chromium.orgf947ed02012-06-12 07:35:26 +090024#include "ipc/ipc_sender.h"
jabdelmalek@google.comeb921652010-04-07 05:33:36 +090025#include "ipc/ipc_sync_message_filter.h"
jam@chromium.org86a8de12010-12-09 08:34:16 +090026#include "ipc/ipc_sync_message_unittest.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090027#include "testing/gtest/include/gtest/gtest.h"
28
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090029using base::WaitableEvent;
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +090030using namespace IPC;
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +090031
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090032namespace {
33
34// Base class for a "process" with listener and IPC threads.
brettw@chromium.orgf947ed02012-06-12 07:35:26 +090035class Worker : public Listener, public Sender {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090036 public:
37 // Will create a channel without a name.
38 Worker(Channel::Mode mode, const std::string& thread_name)
39 : done_(new WaitableEvent(false, false)),
40 channel_created_(new WaitableEvent(false, false)),
41 mode_(mode),
42 ipc_thread_((thread_name + "_ipc").c_str()),
43 listener_thread_((thread_name + "_listener").c_str()),
44 overrided_thread_(NULL),
viettrungluu@chromium.org42575b02013-01-08 15:09:07 +090045 shutdown_event_(true, false),
46 is_shutdown_(false) {
timurrrr@chromium.org03100a82009-10-27 20:28:58 +090047 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090048
49 // Will create a named channel and use this name for the threads' name.
50 Worker(const std::string& channel_name, Channel::Mode mode)
51 : done_(new WaitableEvent(false, false)),
52 channel_created_(new WaitableEvent(false, false)),
53 channel_name_(channel_name),
54 mode_(mode),
55 ipc_thread_((channel_name + "_ipc").c_str()),
56 listener_thread_((channel_name + "_listener").c_str()),
57 overrided_thread_(NULL),
viettrungluu@chromium.org42575b02013-01-08 15:09:07 +090058 shutdown_event_(true, false),
59 is_shutdown_(false) {
timurrrr@chromium.org03100a82009-10-27 20:28:58 +090060 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090061
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090062 virtual ~Worker() {
viettrungluu@chromium.org42575b02013-01-08 15:09:07 +090063 // Shutdown() must be called before destruction.
64 CHECK(is_shutdown_);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090065 }
66 void AddRef() { }
67 void Release() { }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090068 bool Send(Message* msg) { return channel_->Send(msg); }
69 bool SendWithTimeout(Message* msg, int timeout_ms) {
70 return channel_->SendWithTimeout(msg, timeout_ms);
71 }
72 void WaitForChannelCreation() { channel_created_->Wait(); }
73 void CloseChannel() {
74 DCHECK(MessageLoop::current() == ListenerThread()->message_loop());
75 channel_->Close();
76 }
77 void Start() {
78 StartThread(&listener_thread_, MessageLoop::TYPE_DEFAULT);
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +090079 ListenerThread()->message_loop()->PostTask(
80 FROM_HERE, base::Bind(&Worker::OnStart, this));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090081 }
viettrungluu@chromium.org42575b02013-01-08 15:09:07 +090082 void Shutdown() {
83 // The IPC thread needs to outlive SyncChannel. We can't do this in
84 // ~Worker(), since that'll reset the vtable pointer (to Worker's), which
85 // may result in a race conditions. See http://crbug.com/25841.
86 WaitableEvent listener_done(false, false), ipc_done(false, false);
87 ListenerThread()->message_loop()->PostTask(
88 FROM_HERE, base::Bind(&Worker::OnListenerThreadShutdown1, this,
89 &listener_done, &ipc_done));
90 listener_done.Wait();
91 ipc_done.Wait();
92 ipc_thread_.Stop();
93 listener_thread_.Stop();
94 is_shutdown_ = true;
95 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090096 void OverrideThread(base::Thread* overrided_thread) {
97 DCHECK(overrided_thread_ == NULL);
98 overrided_thread_ = overrided_thread;
99 }
100 bool SendAnswerToLife(bool pump, int timeout, bool succeed) {
101 int answer = 0;
102 SyncMessage* msg = new SyncChannelTestMsg_AnswerToLife(&answer);
103 if (pump)
104 msg->EnableMessagePumping();
105 bool result = SendWithTimeout(msg, timeout);
jam@chromium.orgebd07182009-12-01 11:34:18 +0900106 DCHECK_EQ(result, succeed);
107 DCHECK_EQ(answer, (succeed ? 42 : 0));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900108 return result;
109 }
110 bool SendDouble(bool pump, bool succeed) {
111 int answer = 0;
112 SyncMessage* msg = new SyncChannelTestMsg_Double(5, &answer);
113 if (pump)
114 msg->EnableMessagePumping();
115 bool result = Send(msg);
jam@chromium.orgebd07182009-12-01 11:34:18 +0900116 DCHECK_EQ(result, succeed);
117 DCHECK_EQ(answer, (succeed ? 10 : 0));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900118 return result;
119 }
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900120 const std::string& channel_name() { return channel_name_; }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900121 Channel::Mode mode() { return mode_; }
122 WaitableEvent* done_event() { return done_.get(); }
jabdelmalek@google.comeb921652010-04-07 05:33:36 +0900123 WaitableEvent* shutdown_event() { return &shutdown_event_; }
jam@chromium.orgebd07182009-12-01 11:34:18 +0900124 void ResetChannel() { channel_.reset(); }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900125 // Derived classes need to call this when they've completed their part of
126 // the test.
127 void Done() { done_->Signal(); }
jabdelmalek@google.comeb921652010-04-07 05:33:36 +0900128
129 protected:
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +0900130 SyncChannel* channel() { return channel_.get(); }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900131 // Functions for dervied classes to implement if they wish.
132 virtual void Run() { }
133 virtual void OnAnswer(int* answer) { NOTREACHED(); }
134 virtual void OnAnswerDelay(Message* reply_msg) {
135 // The message handler map below can only take one entry for
136 // SyncChannelTestMsg_AnswerToLife, so since some classes want
137 // the normal version while other want the delayed reply, we
138 // call the normal version if the derived class didn't override
139 // this function.
140 int answer;
141 OnAnswer(&answer);
142 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, answer);
143 Send(reply_msg);
144 }
145 virtual void OnDouble(int in, int* out) { NOTREACHED(); }
146 virtual void OnDoubleDelay(int in, Message* reply_msg) {
147 int result;
148 OnDouble(in, &result);
149 SyncChannelTestMsg_Double::WriteReplyParams(reply_msg, result);
150 Send(reply_msg);
151 }
152
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900153 virtual void OnNestedTestMsg(Message* reply_msg) {
154 NOTREACHED();
155 }
156
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900157 virtual SyncChannel* CreateChannel() {
158 return new SyncChannel(
159 channel_name_, mode_, this, ipc_thread_.message_loop_proxy(), true,
160 &shutdown_event_);
161 }
162
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900163 base::Thread* ListenerThread() {
164 return overrided_thread_ ? overrided_thread_ : &listener_thread_;
165 }
ananta@chromium.org999f2972010-09-03 06:45:50 +0900166
piman@google.com0cbefaa2011-04-08 12:38:21 +0900167 const base::Thread& ipc_thread() const { return ipc_thread_; }
168
ananta@chromium.org999f2972010-09-03 06:45:50 +0900169 private:
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900170 // Called on the listener thread to create the sync channel.
171 void OnStart() {
172 // Link ipc_thread_, listener_thread_ and channel_ altogether.
173 StartThread(&ipc_thread_, MessageLoop::TYPE_IO);
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900174 channel_.reset(CreateChannel());
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900175 channel_created_->Signal();
176 Run();
177 }
178
179 void OnListenerThreadShutdown1(WaitableEvent* listener_event,
180 WaitableEvent* ipc_event) {
181 // SyncChannel needs to be destructed on the thread that it was created on.
182 channel_.reset();
183
tfarina@chromium.org969f3202012-11-17 08:41:19 +0900184 MessageLoop::current()->RunUntilIdle();
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900185
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +0900186 ipc_thread_.message_loop()->PostTask(
187 FROM_HERE, base::Bind(&Worker::OnIPCThreadShutdown, this,
188 listener_event, ipc_event));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900189 }
190
191 void OnIPCThreadShutdown(WaitableEvent* listener_event,
192 WaitableEvent* ipc_event) {
tfarina@chromium.org969f3202012-11-17 08:41:19 +0900193 MessageLoop::current()->RunUntilIdle();
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900194 ipc_event->Signal();
195
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +0900196 listener_thread_.message_loop()->PostTask(
197 FROM_HERE, base::Bind(&Worker::OnListenerThreadShutdown2, this,
198 listener_event));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900199 }
200
201 void OnListenerThreadShutdown2(WaitableEvent* listener_event) {
tfarina@chromium.org969f3202012-11-17 08:41:19 +0900202 MessageLoop::current()->RunUntilIdle();
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900203 listener_event->Signal();
204 }
205
jam@chromium.org8a2c7842010-12-24 15:19:28 +0900206 bool OnMessageReceived(const Message& message) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900207 IPC_BEGIN_MESSAGE_MAP(Worker, message)
208 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelTestMsg_Double, OnDoubleDelay)
209 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelTestMsg_AnswerToLife,
210 OnAnswerDelay)
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900211 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelNestedTestMsg_String,
212 OnNestedTestMsg)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900213 IPC_END_MESSAGE_MAP()
jam@chromium.org8a2c7842010-12-24 15:19:28 +0900214 return true;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900215 }
216
217 void StartThread(base::Thread* thread, MessageLoop::Type type) {
218 base::Thread::Options options;
219 options.message_loop_type = type;
220 thread->StartWithOptions(options);
221 }
222
223 scoped_ptr<WaitableEvent> done_;
224 scoped_ptr<WaitableEvent> channel_created_;
225 std::string channel_name_;
226 Channel::Mode mode_;
227 scoped_ptr<SyncChannel> channel_;
228 base::Thread ipc_thread_;
229 base::Thread listener_thread_;
230 base::Thread* overrided_thread_;
231
232 base::WaitableEvent shutdown_event_;
233
viettrungluu@chromium.org42575b02013-01-08 15:09:07 +0900234 bool is_shutdown_;
235
tfarina@chromium.orgb73eaee2010-06-07 11:10:18 +0900236 DISALLOW_COPY_AND_ASSIGN(Worker);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900237};
238
239
240// Starts the test with the given workers. This function deletes the workers
241// when it's done.
242void RunTest(std::vector<Worker*> workers) {
243 // First we create the workers that are channel servers, or else the other
244 // workers' channel initialization might fail because the pipe isn't created..
245 for (size_t i = 0; i < workers.size(); ++i) {
dmaclach@chromium.orgf146c292011-02-04 05:35:09 +0900246 if (workers[i]->mode() & Channel::MODE_SERVER_FLAG) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900247 workers[i]->Start();
248 workers[i]->WaitForChannelCreation();
249 }
250 }
251
252 // now create the clients
253 for (size_t i = 0; i < workers.size(); ++i) {
dmaclach@chromium.orgf146c292011-02-04 05:35:09 +0900254 if (workers[i]->mode() & Channel::MODE_CLIENT_FLAG)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900255 workers[i]->Start();
256 }
257
258 // wait for all the workers to finish
259 for (size_t i = 0; i < workers.size(); ++i)
260 workers[i]->done_event()->Wait();
261
viettrungluu@chromium.org42575b02013-01-08 15:09:07 +0900262 for (size_t i = 0; i < workers.size(); ++i) {
263 workers[i]->Shutdown();
264 delete workers[i];
265 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900266}
267
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900268class IPCSyncChannelTest : public testing::Test {
269 private:
270 MessageLoop message_loop_;
271};
272
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900273//------------------------------------------------------------------------------
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900274
275class SimpleServer : public Worker {
276 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900277 explicit SimpleServer(bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900278 : Worker(Channel::MODE_SERVER, "simpler_server"),
279 pump_during_send_(pump_during_send) { }
280 void Run() {
281 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
282 Done();
283 }
284
285 bool pump_during_send_;
286};
287
288class SimpleClient : public Worker {
289 public:
290 SimpleClient() : Worker(Channel::MODE_CLIENT, "simple_client") { }
291
292 void OnAnswer(int* answer) {
293 *answer = 42;
294 Done();
295 }
296};
297
298void Simple(bool pump_during_send) {
299 std::vector<Worker*> workers;
300 workers.push_back(new SimpleServer(pump_during_send));
301 workers.push_back(new SimpleClient());
302 RunTest(workers);
303}
304
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900305// Tests basic synchronous call
306TEST_F(IPCSyncChannelTest, Simple) {
307 Simple(false);
308 Simple(true);
309}
310
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900311//------------------------------------------------------------------------------
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900312
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900313// Worker classes which override how the sync channel is created to use the
314// two-step initialization (calling the lightweight constructor and then
315// ChannelProxy::Init separately) process.
316class TwoStepServer : public Worker {
317 public:
318 explicit TwoStepServer(bool create_pipe_now)
319 : Worker(Channel::MODE_SERVER, "simpler_server"),
320 create_pipe_now_(create_pipe_now) { }
321
322 void Run() {
323 SendAnswerToLife(false, base::kNoTimeout, true);
324 Done();
325 }
326
327 virtual SyncChannel* CreateChannel() {
328 SyncChannel* channel = new SyncChannel(
329 this, ipc_thread().message_loop_proxy(), shutdown_event());
330 channel->Init(channel_name(), mode(), create_pipe_now_);
331 return channel;
332 }
333
334 bool create_pipe_now_;
335};
336
337class TwoStepClient : public Worker {
338 public:
339 TwoStepClient(bool create_pipe_now)
340 : Worker(Channel::MODE_CLIENT, "simple_client"),
341 create_pipe_now_(create_pipe_now) { }
342
343 void OnAnswer(int* answer) {
344 *answer = 42;
345 Done();
346 }
347
348 virtual SyncChannel* CreateChannel() {
349 SyncChannel* channel = new SyncChannel(
350 this, ipc_thread().message_loop_proxy(), shutdown_event());
351 channel->Init(channel_name(), mode(), create_pipe_now_);
352 return channel;
353 }
354
355 bool create_pipe_now_;
356};
357
358void TwoStep(bool create_server_pipe_now, bool create_client_pipe_now) {
359 std::vector<Worker*> workers;
360 workers.push_back(new TwoStepServer(create_server_pipe_now));
361 workers.push_back(new TwoStepClient(create_client_pipe_now));
362 RunTest(workers);
363}
364
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900365// Tests basic two-step initialization, where you call the lightweight
366// constructor then Init.
367TEST_F(IPCSyncChannelTest, TwoStepInitialization) {
368 TwoStep(false, false);
369 TwoStep(false, true);
370 TwoStep(true, false);
371 TwoStep(true, true);
372}
373
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900374//------------------------------------------------------------------------------
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900375
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900376class DelayClient : public Worker {
377 public:
378 DelayClient() : Worker(Channel::MODE_CLIENT, "delay_client") { }
379
380 void OnAnswerDelay(Message* reply_msg) {
381 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
382 Send(reply_msg);
383 Done();
384 }
385};
386
387void DelayReply(bool pump_during_send) {
388 std::vector<Worker*> workers;
389 workers.push_back(new SimpleServer(pump_during_send));
390 workers.push_back(new DelayClient());
391 RunTest(workers);
392}
393
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900394// Tests that asynchronous replies work
395TEST_F(IPCSyncChannelTest, DelayReply) {
396 DelayReply(false);
397 DelayReply(true);
398}
399
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900400//------------------------------------------------------------------------------
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900401
402class NoHangServer : public Worker {
403 public:
dilmah@chromium.org92b4c8e2011-07-27 02:25:25 +0900404 NoHangServer(WaitableEvent* got_first_reply, bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900405 : Worker(Channel::MODE_SERVER, "no_hang_server"),
406 got_first_reply_(got_first_reply),
407 pump_during_send_(pump_during_send) { }
408 void Run() {
409 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
410 got_first_reply_->Signal();
411
412 SendAnswerToLife(pump_during_send_, base::kNoTimeout, false);
413 Done();
414 }
415
416 WaitableEvent* got_first_reply_;
417 bool pump_during_send_;
418};
419
420class NoHangClient : public Worker {
421 public:
422 explicit NoHangClient(WaitableEvent* got_first_reply)
423 : Worker(Channel::MODE_CLIENT, "no_hang_client"),
424 got_first_reply_(got_first_reply) { }
425
426 virtual void OnAnswerDelay(Message* reply_msg) {
427 // Use the DELAY_REPLY macro so that we can force the reply to be sent
428 // before this function returns (when the channel will be reset).
429 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
430 Send(reply_msg);
431 got_first_reply_->Wait();
432 CloseChannel();
433 Done();
434 }
435
436 WaitableEvent* got_first_reply_;
437};
438
439void NoHang(bool pump_during_send) {
440 WaitableEvent got_first_reply(false, false);
441 std::vector<Worker*> workers;
442 workers.push_back(new NoHangServer(&got_first_reply, pump_during_send));
443 workers.push_back(new NoHangClient(&got_first_reply));
444 RunTest(workers);
445}
446
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900447// Tests that caller doesn't hang if receiver dies
448TEST_F(IPCSyncChannelTest, NoHang) {
449 NoHang(false);
450 NoHang(true);
451}
452
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900453//------------------------------------------------------------------------------
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900454
455class UnblockServer : public Worker {
456 public:
jam@chromium.orgebd07182009-12-01 11:34:18 +0900457 UnblockServer(bool pump_during_send, bool delete_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900458 : Worker(Channel::MODE_SERVER, "unblock_server"),
jam@chromium.orgebd07182009-12-01 11:34:18 +0900459 pump_during_send_(pump_during_send),
460 delete_during_send_(delete_during_send) { }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900461 void Run() {
jam@chromium.orgebd07182009-12-01 11:34:18 +0900462 if (delete_during_send_) {
463 // Use custom code since race conditions mean the answer may or may not be
464 // available.
465 int answer = 0;
466 SyncMessage* msg = new SyncChannelTestMsg_AnswerToLife(&answer);
467 if (pump_during_send_)
468 msg->EnableMessagePumping();
469 Send(msg);
470 } else {
471 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
472 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900473 Done();
474 }
475
jam@chromium.orgebd07182009-12-01 11:34:18 +0900476 void OnDoubleDelay(int in, Message* reply_msg) {
477 SyncChannelTestMsg_Double::WriteReplyParams(reply_msg, in * 2);
478 Send(reply_msg);
479 if (delete_during_send_)
480 ResetChannel();
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900481 }
482
483 bool pump_during_send_;
jam@chromium.orgebd07182009-12-01 11:34:18 +0900484 bool delete_during_send_;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900485};
486
487class UnblockClient : public Worker {
488 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900489 explicit UnblockClient(bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900490 : Worker(Channel::MODE_CLIENT, "unblock_client"),
491 pump_during_send_(pump_during_send) { }
492
493 void OnAnswer(int* answer) {
494 SendDouble(pump_during_send_, true);
495 *answer = 42;
496 Done();
497 }
498
499 bool pump_during_send_;
500};
501
jam@chromium.orgebd07182009-12-01 11:34:18 +0900502void Unblock(bool server_pump, bool client_pump, bool delete_during_send) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900503 std::vector<Worker*> workers;
jam@chromium.orgebd07182009-12-01 11:34:18 +0900504 workers.push_back(new UnblockServer(server_pump, delete_during_send));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900505 workers.push_back(new UnblockClient(client_pump));
506 RunTest(workers);
507}
508
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900509// Tests that the caller unblocks to answer a sync message from the receiver.
510TEST_F(IPCSyncChannelTest, Unblock) {
jam@chromium.orgebd07182009-12-01 11:34:18 +0900511 Unblock(false, false, false);
512 Unblock(false, true, false);
513 Unblock(true, false, false);
514 Unblock(true, true, false);
515}
516
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900517//------------------------------------------------------------------------------
jam@chromium.orgebd07182009-12-01 11:34:18 +0900518
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +0900519// Tests that the the SyncChannel object can be deleted during a Send.
jam@chromium.orgebd07182009-12-01 11:34:18 +0900520TEST_F(IPCSyncChannelTest, ChannelDeleteDuringSend) {
521 Unblock(false, false, true);
522 Unblock(false, true, true);
523 Unblock(true, false, true);
524 Unblock(true, true, true);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900525}
526
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900527//------------------------------------------------------------------------------
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900528
529class RecursiveServer : public Worker {
530 public:
dilmah@chromium.org92b4c8e2011-07-27 02:25:25 +0900531 RecursiveServer(bool expected_send_result, bool pump_first, bool pump_second)
532 : Worker(Channel::MODE_SERVER, "recursive_server"),
533 expected_send_result_(expected_send_result),
534 pump_first_(pump_first), pump_second_(pump_second) {}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900535 void Run() {
536 SendDouble(pump_first_, expected_send_result_);
537 Done();
538 }
539
540 void OnDouble(int in, int* out) {
541 *out = in * 2;
542 SendAnswerToLife(pump_second_, base::kNoTimeout, expected_send_result_);
543 }
544
545 bool expected_send_result_, pump_first_, pump_second_;
546};
547
548class RecursiveClient : public Worker {
549 public:
dilmah@chromium.org92b4c8e2011-07-27 02:25:25 +0900550 RecursiveClient(bool pump_during_send, bool close_channel)
551 : Worker(Channel::MODE_CLIENT, "recursive_client"),
552 pump_during_send_(pump_during_send), close_channel_(close_channel) {}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900553
554 void OnDoubleDelay(int in, Message* reply_msg) {
555 SendDouble(pump_during_send_, !close_channel_);
556 if (close_channel_) {
557 delete reply_msg;
558 } else {
559 SyncChannelTestMsg_Double::WriteReplyParams(reply_msg, in * 2);
560 Send(reply_msg);
561 }
562 Done();
563 }
564
565 void OnAnswerDelay(Message* reply_msg) {
566 if (close_channel_) {
567 delete reply_msg;
568 CloseChannel();
569 } else {
570 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
571 Send(reply_msg);
572 }
573 }
574
575 bool pump_during_send_, close_channel_;
576};
577
578void Recursive(
579 bool server_pump_first, bool server_pump_second, bool client_pump) {
580 std::vector<Worker*> workers;
581 workers.push_back(
582 new RecursiveServer(true, server_pump_first, server_pump_second));
583 workers.push_back(new RecursiveClient(client_pump, false));
584 RunTest(workers);
585}
586
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900587// Tests a server calling Send while another Send is pending.
588TEST_F(IPCSyncChannelTest, Recursive) {
589 Recursive(false, false, false);
590 Recursive(false, false, true);
591 Recursive(false, true, false);
592 Recursive(false, true, true);
593 Recursive(true, false, false);
594 Recursive(true, false, true);
595 Recursive(true, true, false);
596 Recursive(true, true, true);
597}
598
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900599//------------------------------------------------------------------------------
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900600
601void RecursiveNoHang(
602 bool server_pump_first, bool server_pump_second, bool client_pump) {
603 std::vector<Worker*> workers;
604 workers.push_back(
605 new RecursiveServer(false, server_pump_first, server_pump_second));
606 workers.push_back(new RecursiveClient(client_pump, true));
607 RunTest(workers);
608}
609
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900610// Tests that if a caller makes a sync call during an existing sync call and
611// the receiver dies, neither of the Send() calls hang.
612TEST_F(IPCSyncChannelTest, RecursiveNoHang) {
613 RecursiveNoHang(false, false, false);
614 RecursiveNoHang(false, false, true);
615 RecursiveNoHang(false, true, false);
616 RecursiveNoHang(false, true, true);
617 RecursiveNoHang(true, false, false);
618 RecursiveNoHang(true, false, true);
619 RecursiveNoHang(true, true, false);
620 RecursiveNoHang(true, true, true);
621}
622
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900623//------------------------------------------------------------------------------
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900624
625class MultipleServer1 : public Worker {
626 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900627 explicit MultipleServer1(bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900628 : Worker("test_channel1", Channel::MODE_SERVER),
629 pump_during_send_(pump_during_send) { }
630
631 void Run() {
632 SendDouble(pump_during_send_, true);
633 Done();
634 }
635
636 bool pump_during_send_;
637};
638
639class MultipleClient1 : public Worker {
640 public:
641 MultipleClient1(WaitableEvent* client1_msg_received,
642 WaitableEvent* client1_can_reply) :
643 Worker("test_channel1", Channel::MODE_CLIENT),
644 client1_msg_received_(client1_msg_received),
645 client1_can_reply_(client1_can_reply) { }
646
647 void OnDouble(int in, int* out) {
648 client1_msg_received_->Signal();
649 *out = in * 2;
650 client1_can_reply_->Wait();
651 Done();
652 }
653
654 private:
655 WaitableEvent *client1_msg_received_, *client1_can_reply_;
656};
657
658class MultipleServer2 : public Worker {
659 public:
660 MultipleServer2() : Worker("test_channel2", Channel::MODE_SERVER) { }
661
662 void OnAnswer(int* result) {
663 *result = 42;
664 Done();
665 }
666};
667
668class MultipleClient2 : public Worker {
669 public:
670 MultipleClient2(
671 WaitableEvent* client1_msg_received, WaitableEvent* client1_can_reply,
672 bool pump_during_send)
673 : Worker("test_channel2", Channel::MODE_CLIENT),
674 client1_msg_received_(client1_msg_received),
675 client1_can_reply_(client1_can_reply),
676 pump_during_send_(pump_during_send) { }
677
678 void Run() {
679 client1_msg_received_->Wait();
680 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
681 client1_can_reply_->Signal();
682 Done();
683 }
684
685 private:
686 WaitableEvent *client1_msg_received_, *client1_can_reply_;
687 bool pump_during_send_;
688};
689
690void Multiple(bool server_pump, bool client_pump) {
691 std::vector<Worker*> workers;
692
693 // A shared worker thread so that server1 and server2 run on one thread.
694 base::Thread worker_thread("Multiple");
695 ASSERT_TRUE(worker_thread.Start());
696
697 // Server1 sends a sync msg to client1, which blocks the reply until
698 // server2 (which runs on the same worker thread as server1) responds
699 // to a sync msg from client2.
700 WaitableEvent client1_msg_received(false, false);
701 WaitableEvent client1_can_reply(false, false);
702
703 Worker* worker;
704
705 worker = new MultipleServer2();
706 worker->OverrideThread(&worker_thread);
707 workers.push_back(worker);
708
709 worker = new MultipleClient2(
710 &client1_msg_received, &client1_can_reply, client_pump);
711 workers.push_back(worker);
712
713 worker = new MultipleServer1(server_pump);
714 worker->OverrideThread(&worker_thread);
715 workers.push_back(worker);
716
717 worker = new MultipleClient1(
718 &client1_msg_received, &client1_can_reply);
719 workers.push_back(worker);
720
721 RunTest(workers);
722}
723
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900724// Tests that multiple SyncObjects on the same listener thread can unblock each
725// other.
726TEST_F(IPCSyncChannelTest, Multiple) {
727 Multiple(false, false);
728 Multiple(false, true);
729 Multiple(true, false);
730 Multiple(true, true);
731}
732
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900733//------------------------------------------------------------------------------
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900734
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900735// This class provides server side functionality to test the case where
736// multiple sync channels are in use on the same thread on the client and
737// nested calls are issued.
738class QueuedReplyServer : public Worker {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900739 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900740 QueuedReplyServer(base::Thread* listener_thread,
741 const std::string& channel_name,
742 const std::string& reply_text)
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900743 : Worker(channel_name, Channel::MODE_SERVER),
744 reply_text_(reply_text) {
745 Worker::OverrideThread(listener_thread);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900746 }
747
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900748 virtual void OnNestedTestMsg(Message* reply_msg) {
pkasting@chromium.orgfcdd54b2010-10-20 08:50:00 +0900749 VLOG(1) << __FUNCTION__ << " Sending reply: " << reply_text_;
750 SyncChannelNestedTestMsg_String::WriteReplyParams(reply_msg, reply_text_);
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900751 Send(reply_msg);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900752 Done();
753 }
754
755 private:
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900756 std::string reply_text_;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900757};
758
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900759// The QueuedReplyClient class provides functionality to test the case where
760// multiple sync channels are in use on the same thread and they make nested
761// sync calls, i.e. while the first channel waits for a response it makes a
762// sync call on another channel.
763// The callstack should unwind correctly, i.e. the outermost call should
764// complete first, and so on.
765class QueuedReplyClient : public Worker {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900766 public:
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900767 QueuedReplyClient(base::Thread* listener_thread,
768 const std::string& channel_name,
769 const std::string& expected_text,
770 bool pump_during_send)
771 : Worker(channel_name, Channel::MODE_CLIENT),
thomasvl@google.com9a242072010-07-23 23:18:59 +0900772 pump_during_send_(pump_during_send),
773 expected_text_(expected_text) {
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900774 Worker::OverrideThread(listener_thread);
775 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900776
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900777 virtual void Run() {
778 std::string response;
779 SyncMessage* msg = new SyncChannelNestedTestMsg_String(&response);
780 if (pump_during_send_)
781 msg->EnableMessagePumping();
782 bool result = Send(msg);
783 DCHECK(result);
jam@chromium.orgebd07182009-12-01 11:34:18 +0900784 DCHECK_EQ(response, expected_text_);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900785
pkasting@chromium.orgfcdd54b2010-10-20 08:50:00 +0900786 VLOG(1) << __FUNCTION__ << " Received reply: " << response;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900787 Done();
788 }
789
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900790 private:
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900791 bool pump_during_send_;
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900792 std::string expected_text_;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900793};
794
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900795void QueuedReply(bool client_pump) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900796 std::vector<Worker*> workers;
797
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900798 // A shared worker thread for servers
799 base::Thread server_worker_thread("QueuedReply_ServerListener");
800 ASSERT_TRUE(server_worker_thread.Start());
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900801
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900802 base::Thread client_worker_thread("QueuedReply_ClientListener");
803 ASSERT_TRUE(client_worker_thread.Start());
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900804
805 Worker* worker;
806
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900807 worker = new QueuedReplyServer(&server_worker_thread,
808 "QueuedReply_Server1",
809 "Got first message");
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900810 workers.push_back(worker);
811
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900812 worker = new QueuedReplyServer(&server_worker_thread,
813 "QueuedReply_Server2",
814 "Got second message");
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900815 workers.push_back(worker);
816
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900817 worker = new QueuedReplyClient(&client_worker_thread,
818 "QueuedReply_Server1",
819 "Got first message",
820 client_pump);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900821 workers.push_back(worker);
822
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900823 worker = new QueuedReplyClient(&client_worker_thread,
824 "QueuedReply_Server2",
825 "Got second message",
826 client_pump);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900827 workers.push_back(worker);
828
829 RunTest(workers);
830}
831
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900832// While a blocking send is in progress, the listener thread might answer other
833// synchronous messages. This tests that if during the response to another
834// message the reply to the original messages comes, it is queued up correctly
835// and the original Send is unblocked later.
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900836// We also test that the send call stacks unwind correctly when the channel
837// pumps messages while waiting for a response.
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900838TEST_F(IPCSyncChannelTest, QueuedReply) {
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900839 QueuedReply(false);
840 QueuedReply(true);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900841}
842
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +0900843//------------------------------------------------------------------------------
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900844
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900845class ChattyClient : public Worker {
846 public:
847 ChattyClient() :
848 Worker(Channel::MODE_CLIENT, "chatty_client") { }
849
850 void OnAnswer(int* answer) {
851 // The PostMessage limit is 10k. Send 20% more than that.
852 const int kMessageLimit = 10000;
853 const int kMessagesToSend = kMessageLimit * 120 / 100;
854 for (int i = 0; i < kMessagesToSend; ++i) {
855 if (!SendDouble(false, true))
856 break;
857 }
858 *answer = 42;
859 Done();
860 }
861};
862
863void ChattyServer(bool pump_during_send) {
864 std::vector<Worker*> workers;
jam@chromium.orgebd07182009-12-01 11:34:18 +0900865 workers.push_back(new UnblockServer(pump_during_send, false));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900866 workers.push_back(new ChattyClient());
867 RunTest(workers);
868}
869
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900870// Tests http://b/1093251 - that sending lots of sync messages while
871// the receiver is waiting for a sync reply does not overflow the PostMessage
872// queue.
873TEST_F(IPCSyncChannelTest, ChattyServer) {
874 ChattyServer(false);
875 ChattyServer(true);
876}
877
878//------------------------------------------------------------------------------
879
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900880class TimeoutServer : public Worker {
881 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900882 TimeoutServer(int timeout_ms,
883 std::vector<bool> timeout_seq,
884 bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900885 : Worker(Channel::MODE_SERVER, "timeout_server"),
886 timeout_ms_(timeout_ms),
887 timeout_seq_(timeout_seq),
888 pump_during_send_(pump_during_send) {
889 }
890
891 void Run() {
892 for (std::vector<bool>::const_iterator iter = timeout_seq_.begin();
893 iter != timeout_seq_.end(); ++iter) {
894 SendAnswerToLife(pump_during_send_, timeout_ms_, !*iter);
895 }
896 Done();
897 }
898
899 private:
900 int timeout_ms_;
901 std::vector<bool> timeout_seq_;
902 bool pump_during_send_;
903};
904
905class UnresponsiveClient : public Worker {
906 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900907 explicit UnresponsiveClient(std::vector<bool> timeout_seq)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900908 : Worker(Channel::MODE_CLIENT, "unresponsive_client"),
909 timeout_seq_(timeout_seq) {
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900910 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900911
912 void OnAnswerDelay(Message* reply_msg) {
913 DCHECK(!timeout_seq_.empty());
914 if (!timeout_seq_[0]) {
915 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
916 Send(reply_msg);
917 } else {
918 // Don't reply.
919 delete reply_msg;
920 }
921 timeout_seq_.erase(timeout_seq_.begin());
922 if (timeout_seq_.empty())
923 Done();
924 }
925
926 private:
927 // Whether we should time-out or respond to the various messages we receive.
928 std::vector<bool> timeout_seq_;
929};
930
931void SendWithTimeoutOK(bool pump_during_send) {
932 std::vector<Worker*> workers;
933 std::vector<bool> timeout_seq;
934 timeout_seq.push_back(false);
935 timeout_seq.push_back(false);
936 timeout_seq.push_back(false);
937 workers.push_back(new TimeoutServer(5000, timeout_seq, pump_during_send));
938 workers.push_back(new SimpleClient());
939 RunTest(workers);
940}
941
942void SendWithTimeoutTimeout(bool pump_during_send) {
943 std::vector<Worker*> workers;
944 std::vector<bool> timeout_seq;
945 timeout_seq.push_back(true);
946 timeout_seq.push_back(false);
947 timeout_seq.push_back(false);
948 workers.push_back(new TimeoutServer(100, timeout_seq, pump_during_send));
949 workers.push_back(new UnresponsiveClient(timeout_seq));
950 RunTest(workers);
951}
952
953void SendWithTimeoutMixedOKAndTimeout(bool pump_during_send) {
954 std::vector<Worker*> workers;
955 std::vector<bool> timeout_seq;
956 timeout_seq.push_back(true);
957 timeout_seq.push_back(false);
958 timeout_seq.push_back(false);
959 timeout_seq.push_back(true);
960 timeout_seq.push_back(false);
961 workers.push_back(new TimeoutServer(100, timeout_seq, pump_during_send));
962 workers.push_back(new UnresponsiveClient(timeout_seq));
963 RunTest(workers);
964}
965
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900966// Tests that SendWithTimeout does not time-out if the response comes back fast
967// enough.
968TEST_F(IPCSyncChannelTest, SendWithTimeoutOK) {
969 SendWithTimeoutOK(false);
970 SendWithTimeoutOK(true);
971}
972
973// Tests that SendWithTimeout does time-out.
974TEST_F(IPCSyncChannelTest, SendWithTimeoutTimeout) {
975 SendWithTimeoutTimeout(false);
976 SendWithTimeoutTimeout(true);
977}
978
979// Sends some message that time-out and some that succeed.
phajdan.jr@chromium.orga8e3c092011-01-19 17:11:56 +0900980// Crashes flakily, http://crbug.com/70075.
981TEST_F(IPCSyncChannelTest, DISABLED_SendWithTimeoutMixedOKAndTimeout) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900982 SendWithTimeoutMixedOKAndTimeout(false);
983 SendWithTimeoutMixedOKAndTimeout(true);
984}
985
986//------------------------------------------------------------------------------
987
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +0900988void NestedCallback(Worker* server) {
989 // Sleep a bit so that we wake up after the reply has been received.
tedvessenes@gmail.com30dbbaa2012-01-13 09:11:01 +0900990 base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(250));
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +0900991 server->SendAnswerToLife(true, base::kNoTimeout, true);
992}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900993
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +0900994bool timeout_occurred = false;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900995
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +0900996void TimeoutCallback() {
997 timeout_occurred = true;
998}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900999
1000class DoneEventRaceServer : public Worker {
1001 public:
1002 DoneEventRaceServer()
1003 : Worker(Channel::MODE_SERVER, "done_event_race_server") { }
1004
1005 void Run() {
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001006 MessageLoop::current()->PostTask(FROM_HERE,
1007 base::Bind(&NestedCallback, this));
1008 MessageLoop::current()->PostDelayedTask(
tedvessenes@gmail.com56b33702012-03-07 13:41:40 +09001009 FROM_HERE,
1010 base::Bind(&TimeoutCallback),
1011 base::TimeDelta::FromSeconds(9));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001012 // Even though we have a timeout on the Send, it will succeed since for this
1013 // bug, the reply message comes back and is deserialized, however the done
1014 // event wasn't set. So we indirectly use the timeout task to notice if a
1015 // timeout occurred.
1016 SendAnswerToLife(true, 10000, true);
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001017 DCHECK(!timeout_occurred);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001018 Done();
1019 }
1020};
1021
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001022// Tests http://b/1474092 - that if after the done_event is set but before
1023// OnObjectSignaled is called another message is sent out, then after its
1024// reply comes back OnObjectSignaled will be called for the first message.
1025TEST_F(IPCSyncChannelTest, DoneEventRace) {
1026 std::vector<Worker*> workers;
1027 workers.push_back(new DoneEventRaceServer());
1028 workers.push_back(new SimpleClient());
1029 RunTest(workers);
1030}
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001031
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +09001032//------------------------------------------------------------------------------
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001033
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001034class TestSyncMessageFilter : public SyncMessageFilter {
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001035 public:
piman@chromium.org0345c812012-09-08 10:24:49 +09001036 TestSyncMessageFilter(base::WaitableEvent* shutdown_event,
1037 Worker* worker,
1038 scoped_refptr<base::MessageLoopProxy> message_loop)
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001039 : SyncMessageFilter(shutdown_event),
1040 worker_(worker),
piman@chromium.org0345c812012-09-08 10:24:49 +09001041 message_loop_(message_loop) {
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001042 }
1043
1044 virtual void OnFilterAdded(Channel* channel) {
1045 SyncMessageFilter::OnFilterAdded(channel);
piman@chromium.org0345c812012-09-08 10:24:49 +09001046 message_loop_->PostTask(
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001047 FROM_HERE,
1048 base::Bind(&TestSyncMessageFilter::SendMessageOnHelperThread, this));
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001049 }
1050
1051 void SendMessageOnHelperThread() {
1052 int answer = 0;
1053 bool result = Send(new SyncChannelTestMsg_AnswerToLife(&answer));
1054 DCHECK(result);
1055 DCHECK_EQ(answer, 42);
1056
1057 worker_->Done();
1058 }
1059
rsleevi@chromium.org997c1d42012-04-28 11:12:00 +09001060 private:
1061 virtual ~TestSyncMessageFilter() {}
1062
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001063 Worker* worker_;
piman@chromium.org0345c812012-09-08 10:24:49 +09001064 scoped_refptr<base::MessageLoopProxy> message_loop_;
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001065};
1066
1067class SyncMessageFilterServer : public Worker {
1068 public:
1069 SyncMessageFilterServer()
piman@chromium.org0345c812012-09-08 10:24:49 +09001070 : Worker(Channel::MODE_SERVER, "sync_message_filter_server"),
1071 thread_("helper_thread") {
1072 base::Thread::Options options;
1073 options.message_loop_type = MessageLoop::TYPE_DEFAULT;
1074 thread_.StartWithOptions(options);
1075 filter_ = new TestSyncMessageFilter(shutdown_event(), this,
1076 thread_.message_loop_proxy());
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001077 }
1078
1079 void Run() {
1080 channel()->AddFilter(filter_.get());
1081 }
1082
piman@chromium.org0345c812012-09-08 10:24:49 +09001083 base::Thread thread_;
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001084 scoped_refptr<TestSyncMessageFilter> filter_;
1085};
1086
ananta@chromium.org999f2972010-09-03 06:45:50 +09001087// This class provides functionality to test the case that a Send on the sync
1088// channel does not crash after the channel has been closed.
1089class ServerSendAfterClose : public Worker {
1090 public:
1091 ServerSendAfterClose()
1092 : Worker(Channel::MODE_SERVER, "simpler_server"),
1093 send_result_(true) {
1094 }
1095
1096 bool SendDummy() {
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001097 ListenerThread()->message_loop()->PostTask(
ajwong@chromium.orgc1e632d2012-01-06 11:37:17 +09001098 FROM_HERE, base::Bind(base::IgnoreResult(&ServerSendAfterClose::Send),
1099 this, new SyncChannelTestMsg_NoArgs));
ananta@chromium.org999f2972010-09-03 06:45:50 +09001100 return true;
1101 }
1102
1103 bool send_result() const {
1104 return send_result_;
1105 }
1106
1107 private:
1108 virtual void Run() {
1109 CloseChannel();
1110 Done();
1111 }
1112
1113 bool Send(Message* msg) {
1114 send_result_ = Worker::Send(msg);
1115 Done();
1116 return send_result_;
1117 }
1118
1119 bool send_result_;
1120};
1121
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001122// Tests basic synchronous call
1123TEST_F(IPCSyncChannelTest, SyncMessageFilter) {
1124 std::vector<Worker*> workers;
1125 workers.push_back(new SyncMessageFilterServer());
1126 workers.push_back(new SimpleClient());
1127 RunTest(workers);
1128}
ananta@chromium.org999f2972010-09-03 06:45:50 +09001129
1130// Test the case when the channel is closed and a Send is attempted after that.
1131TEST_F(IPCSyncChannelTest, SendAfterClose) {
1132 ServerSendAfterClose server;
1133 server.Start();
1134
1135 server.done_event()->Wait();
1136 server.done_event()->Reset();
1137
1138 server.SendDummy();
1139 server.done_event()->Wait();
1140
1141 EXPECT_FALSE(server.send_result());
viettrungluu@chromium.org42575b02013-01-08 15:09:07 +09001142
1143 server.Shutdown();
ananta@chromium.org999f2972010-09-03 06:45:50 +09001144}
1145
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +09001146//------------------------------------------------------------------------------
piman@google.com0cbefaa2011-04-08 12:38:21 +09001147
1148class RestrictedDispatchServer : public Worker {
1149 public:
piman@chromium.org5a00b882012-03-31 06:29:30 +09001150 RestrictedDispatchServer(WaitableEvent* sent_ping_event,
1151 WaitableEvent* wait_event)
piman@google.com0cbefaa2011-04-08 12:38:21 +09001152 : Worker("restricted_channel", Channel::MODE_SERVER),
piman@chromium.org5a00b882012-03-31 06:29:30 +09001153 sent_ping_event_(sent_ping_event),
1154 wait_event_(wait_event) { }
piman@google.com0cbefaa2011-04-08 12:38:21 +09001155
1156 void OnDoPing(int ping) {
1157 // Send an asynchronous message that unblocks the caller.
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001158 Message* msg = new SyncChannelTestMsg_Ping(ping);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001159 msg->set_unblock(true);
1160 Send(msg);
1161 // Signal the event after the message has been sent on the channel, on the
1162 // IPC thread.
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001163 ipc_thread().message_loop()->PostTask(
1164 FROM_HERE, base::Bind(&RestrictedDispatchServer::OnPingSent, this));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001165 }
1166
piman@chromium.org5a00b882012-03-31 06:29:30 +09001167 void OnPingTTL(int ping, int* out) {
1168 *out = ping;
1169 wait_event_->Wait();
1170 }
1171
piman@google.com0cbefaa2011-04-08 12:38:21 +09001172 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1173
1174 private:
1175 bool OnMessageReceived(const Message& message) {
1176 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchServer, message)
1177 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
piman@chromium.org5a00b882012-03-31 06:29:30 +09001178 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_PingTTL, OnPingTTL)
piman@google.com0cbefaa2011-04-08 12:38:21 +09001179 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, Done)
1180 IPC_END_MESSAGE_MAP()
1181 return true;
1182 }
1183
1184 void OnPingSent() {
1185 sent_ping_event_->Signal();
1186 }
1187
1188 void OnNoArgs() { }
1189 WaitableEvent* sent_ping_event_;
piman@chromium.org5a00b882012-03-31 06:29:30 +09001190 WaitableEvent* wait_event_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001191};
1192
1193class NonRestrictedDispatchServer : public Worker {
1194 public:
piman@chromium.org5a00b882012-03-31 06:29:30 +09001195 NonRestrictedDispatchServer(WaitableEvent* signal_event)
1196 : Worker("non_restricted_channel", Channel::MODE_SERVER),
1197 signal_event_(signal_event) {}
1198
1199 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1200
1201 void OnDoPingTTL(int ping) {
1202 int value = 0;
1203 Send(new SyncChannelTestMsg_PingTTL(ping, &value));
1204 signal_event_->Signal();
1205 }
piman@google.com0cbefaa2011-04-08 12:38:21 +09001206
1207 private:
1208 bool OnMessageReceived(const Message& message) {
1209 IPC_BEGIN_MESSAGE_MAP(NonRestrictedDispatchServer, message)
1210 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1211 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, Done)
1212 IPC_END_MESSAGE_MAP()
1213 return true;
1214 }
1215
1216 void OnNoArgs() { }
piman@chromium.org5a00b882012-03-31 06:29:30 +09001217 WaitableEvent* signal_event_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001218};
1219
1220class RestrictedDispatchClient : public Worker {
1221 public:
1222 RestrictedDispatchClient(WaitableEvent* sent_ping_event,
1223 RestrictedDispatchServer* server,
piman@chromium.org5a00b882012-03-31 06:29:30 +09001224 NonRestrictedDispatchServer* server2,
piman@google.com0cbefaa2011-04-08 12:38:21 +09001225 int* success)
1226 : Worker("restricted_channel", Channel::MODE_CLIENT),
1227 ping_(0),
1228 server_(server),
piman@chromium.org5a00b882012-03-31 06:29:30 +09001229 server2_(server2),
piman@google.com0cbefaa2011-04-08 12:38:21 +09001230 success_(success),
1231 sent_ping_event_(sent_ping_event) {}
1232
1233 void Run() {
1234 // Incoming messages from our channel should only be dispatched when we
1235 // send a message on that same channel.
piman@chromium.org5a00b882012-03-31 06:29:30 +09001236 channel()->SetRestrictDispatchChannelGroup(1);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001237
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001238 server_->ListenerThread()->message_loop()->PostTask(
1239 FROM_HERE, base::Bind(&RestrictedDispatchServer::OnDoPing, server_, 1));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001240 sent_ping_event_->Wait();
1241 Send(new SyncChannelTestMsg_NoArgs);
1242 if (ping_ == 1)
1243 ++*success_;
1244 else
1245 LOG(ERROR) << "Send failed to dispatch incoming message on same channel";
1246
piman@chromium.org5a00b882012-03-31 06:29:30 +09001247 non_restricted_channel_.reset(new SyncChannel(
piman@google.com0cbefaa2011-04-08 12:38:21 +09001248 "non_restricted_channel", Channel::MODE_CLIENT, this,
jam@chromium.org06d18442011-05-03 03:00:49 +09001249 ipc_thread().message_loop_proxy(), true, shutdown_event()));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001250
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001251 server_->ListenerThread()->message_loop()->PostTask(
1252 FROM_HERE, base::Bind(&RestrictedDispatchServer::OnDoPing, server_, 2));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001253 sent_ping_event_->Wait();
1254 // Check that the incoming message is *not* dispatched when sending on the
1255 // non restricted channel.
1256 // TODO(piman): there is a possibility of a false positive race condition
1257 // here, if the message that was posted on the server-side end of the pipe
1258 // is not visible yet on the client side, but I don't know how to solve this
1259 // without hooking into the internals of SyncChannel. I haven't seen it in
1260 // practice (i.e. not setting SetRestrictDispatchToSameChannel does cause
1261 // the following to fail).
piman@chromium.org5a00b882012-03-31 06:29:30 +09001262 non_restricted_channel_->Send(new SyncChannelTestMsg_NoArgs);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001263 if (ping_ == 1)
1264 ++*success_;
1265 else
1266 LOG(ERROR) << "Send dispatched message from restricted channel";
1267
1268 Send(new SyncChannelTestMsg_NoArgs);
1269 if (ping_ == 2)
1270 ++*success_;
1271 else
1272 LOG(ERROR) << "Send failed to dispatch incoming message on same channel";
1273
piman@chromium.org5a00b882012-03-31 06:29:30 +09001274 // Check that the incoming message on the non-restricted channel is
1275 // dispatched when sending on the restricted channel.
1276 server2_->ListenerThread()->message_loop()->PostTask(
1277 FROM_HERE,
1278 base::Bind(&NonRestrictedDispatchServer::OnDoPingTTL, server2_, 3));
1279 int value = 0;
1280 Send(new SyncChannelTestMsg_PingTTL(4, &value));
1281 if (ping_ == 3 && value == 4)
1282 ++*success_;
1283 else
1284 LOG(ERROR) << "Send failed to dispatch message from unrestricted channel";
1285
1286 non_restricted_channel_->Send(new SyncChannelTestMsg_Done);
1287 non_restricted_channel_.reset();
piman@google.com0cbefaa2011-04-08 12:38:21 +09001288 Send(new SyncChannelTestMsg_Done);
1289 Done();
1290 }
1291
1292 private:
1293 bool OnMessageReceived(const Message& message) {
1294 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchClient, message)
1295 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Ping, OnPing)
piman@chromium.org5a00b882012-03-31 06:29:30 +09001296 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelTestMsg_PingTTL, OnPingTTL)
piman@google.com0cbefaa2011-04-08 12:38:21 +09001297 IPC_END_MESSAGE_MAP()
1298 return true;
1299 }
1300
1301 void OnPing(int ping) {
1302 ping_ = ping;
1303 }
1304
piman@chromium.org5a00b882012-03-31 06:29:30 +09001305 void OnPingTTL(int ping, IPC::Message* reply) {
1306 ping_ = ping;
1307 // This message comes from the NonRestrictedDispatchServer, we have to send
1308 // the reply back manually.
1309 SyncChannelTestMsg_PingTTL::WriteReplyParams(reply, ping);
1310 non_restricted_channel_->Send(reply);
1311 }
1312
piman@google.com0cbefaa2011-04-08 12:38:21 +09001313 int ping_;
1314 RestrictedDispatchServer* server_;
piman@chromium.org5a00b882012-03-31 06:29:30 +09001315 NonRestrictedDispatchServer* server2_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001316 int* success_;
1317 WaitableEvent* sent_ping_event_;
piman@chromium.org5a00b882012-03-31 06:29:30 +09001318 scoped_ptr<SyncChannel> non_restricted_channel_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001319};
1320
piman@google.com0cbefaa2011-04-08 12:38:21 +09001321TEST_F(IPCSyncChannelTest, RestrictedDispatch) {
1322 WaitableEvent sent_ping_event(false, false);
piman@chromium.org5a00b882012-03-31 06:29:30 +09001323 WaitableEvent wait_event(false, false);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001324 RestrictedDispatchServer* server =
piman@chromium.org5a00b882012-03-31 06:29:30 +09001325 new RestrictedDispatchServer(&sent_ping_event, &wait_event);
1326 NonRestrictedDispatchServer* server2 =
1327 new NonRestrictedDispatchServer(&wait_event);
1328
piman@google.com0cbefaa2011-04-08 12:38:21 +09001329 int success = 0;
1330 std::vector<Worker*> workers;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001331 workers.push_back(server);
piman@chromium.org5a00b882012-03-31 06:29:30 +09001332 workers.push_back(server2);
1333 workers.push_back(new RestrictedDispatchClient(
1334 &sent_ping_event, server, server2, &success));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001335 RunTest(workers);
piman@chromium.org5a00b882012-03-31 06:29:30 +09001336 EXPECT_EQ(4, success);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001337}
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001338
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +09001339//------------------------------------------------------------------------------
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001340
1341// This test case inspired by crbug.com/108491
1342// We create two servers that use the same ListenerThread but have
1343// SetRestrictDispatchToSameChannel set to true.
1344// We create clients, then use some specific WaitableEvent wait/signalling to
1345// ensure that messages get dispatched in a way that causes a deadlock due to
1346// a nested dispatch and an eligible message in a higher-level dispatch's
1347// delayed_queue. Specifically, we start with client1 about so send an
1348// unblocking message to server1, while the shared listener thread for the
1349// servers server1 and server2 is about to send a non-unblocking message to
1350// client1. At the same time, client2 will be about to send an unblocking
1351// message to server2. Server1 will handle the client1->server1 message by
1352// telling server2 to send a non-unblocking message to client2.
1353// What should happen is that the send to server2 should find the pending,
1354// same-context client2->server2 message to dispatch, causing client2 to
1355// unblock then handle the server2->client2 message, so that the shared
1356// servers' listener thread can then respond to the client1->server1 message.
1357// Then client1 can handle the non-unblocking server1->client1 message.
1358// The old code would end up in a state where the server2->client2 message is
1359// sent, but the client2->server2 message (which is eligible for dispatch, and
1360// which is what client2 is waiting for) is stashed in a local delayed_queue
1361// that has server1's channel context, causing a deadlock.
1362// WaitableEvents in the events array are used to:
1363// event 0: indicate to client1 that server listener is in OnDoServerTask
1364// event 1: indicate to client1 that client2 listener is in OnDoClient2Task
1365// event 2: indicate to server1 that client2 listener is in OnDoClient2Task
1366// event 3: indicate to client2 that server listener is in OnDoServerTask
1367
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001368class RestrictedDispatchDeadlockServer : public Worker {
1369 public:
1370 RestrictedDispatchDeadlockServer(int server_num,
1371 WaitableEvent* server_ready_event,
1372 WaitableEvent** events,
1373 RestrictedDispatchDeadlockServer* peer)
1374 : Worker(server_num == 1 ? "channel1" : "channel2", Channel::MODE_SERVER),
1375 server_num_(server_num),
1376 server_ready_event_(server_ready_event),
1377 events_(events),
thakis@chromium.orgad1a2c52012-07-21 01:17:29 +09001378 peer_(peer) { }
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001379
1380 void OnDoServerTask() {
1381 events_[3]->Signal();
1382 events_[2]->Wait();
1383 events_[0]->Signal();
1384 SendMessageToClient();
1385 }
1386
1387 void Run() {
piman@chromium.org5a00b882012-03-31 06:29:30 +09001388 channel()->SetRestrictDispatchChannelGroup(1);
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001389 server_ready_event_->Signal();
1390 }
1391
1392 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1393
1394 private:
1395 bool OnMessageReceived(const Message& message) {
1396 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchDeadlockServer, message)
1397 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1398 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, Done)
1399 IPC_END_MESSAGE_MAP()
1400 return true;
1401 }
1402
1403 void OnNoArgs() {
1404 if (server_num_ == 1) {
1405 DCHECK(peer_ != NULL);
1406 peer_->SendMessageToClient();
1407 }
1408 }
1409
1410 void SendMessageToClient() {
1411 Message* msg = new SyncChannelTestMsg_NoArgs;
1412 msg->set_unblock(false);
1413 DCHECK(!msg->should_unblock());
1414 Send(msg);
1415 }
1416
1417 int server_num_;
1418 WaitableEvent* server_ready_event_;
1419 WaitableEvent** events_;
1420 RestrictedDispatchDeadlockServer* peer_;
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001421};
1422
1423class RestrictedDispatchDeadlockClient2 : public Worker {
1424 public:
1425 RestrictedDispatchDeadlockClient2(RestrictedDispatchDeadlockServer* server,
1426 WaitableEvent* server_ready_event,
1427 WaitableEvent** events)
1428 : Worker("channel2", Channel::MODE_CLIENT),
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001429 server_ready_event_(server_ready_event),
1430 events_(events),
1431 received_msg_(false),
1432 received_noarg_reply_(false),
1433 done_issued_(false) {}
1434
1435 void Run() {
1436 server_ready_event_->Wait();
1437 }
1438
1439 void OnDoClient2Task() {
1440 events_[3]->Wait();
1441 events_[1]->Signal();
1442 events_[2]->Signal();
1443 DCHECK(received_msg_ == false);
1444
1445 Message* message = new SyncChannelTestMsg_NoArgs;
1446 message->set_unblock(true);
1447 Send(message);
1448 received_noarg_reply_ = true;
1449 }
1450
1451 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1452 private:
1453 bool OnMessageReceived(const Message& message) {
1454 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchDeadlockClient2, message)
1455 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1456 IPC_END_MESSAGE_MAP()
1457 return true;
1458 }
1459
1460 void OnNoArgs() {
1461 received_msg_ = true;
1462 PossiblyDone();
1463 }
1464
1465 void PossiblyDone() {
1466 if (received_noarg_reply_ && received_msg_) {
1467 DCHECK(done_issued_ == false);
1468 done_issued_ = true;
1469 Send(new SyncChannelTestMsg_Done);
1470 Done();
1471 }
1472 }
1473
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001474 WaitableEvent* server_ready_event_;
1475 WaitableEvent** events_;
1476 bool received_msg_;
1477 bool received_noarg_reply_;
1478 bool done_issued_;
1479};
1480
1481class RestrictedDispatchDeadlockClient1 : public Worker {
1482 public:
1483 RestrictedDispatchDeadlockClient1(RestrictedDispatchDeadlockServer* server,
1484 RestrictedDispatchDeadlockClient2* peer,
1485 WaitableEvent* server_ready_event,
1486 WaitableEvent** events)
1487 : Worker("channel1", Channel::MODE_CLIENT),
1488 server_(server),
1489 peer_(peer),
1490 server_ready_event_(server_ready_event),
1491 events_(events),
1492 received_msg_(false),
1493 received_noarg_reply_(false),
1494 done_issued_(false) {}
1495
1496 void Run() {
1497 server_ready_event_->Wait();
1498 server_->ListenerThread()->message_loop()->PostTask(
1499 FROM_HERE,
1500 base::Bind(&RestrictedDispatchDeadlockServer::OnDoServerTask, server_));
1501 peer_->ListenerThread()->message_loop()->PostTask(
1502 FROM_HERE,
1503 base::Bind(&RestrictedDispatchDeadlockClient2::OnDoClient2Task, peer_));
1504 events_[0]->Wait();
1505 events_[1]->Wait();
1506 DCHECK(received_msg_ == false);
1507
1508 Message* message = new SyncChannelTestMsg_NoArgs;
1509 message->set_unblock(true);
1510 Send(message);
1511 received_noarg_reply_ = true;
1512 PossiblyDone();
1513 }
1514
1515 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1516 private:
1517 bool OnMessageReceived(const Message& message) {
1518 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchDeadlockClient1, message)
1519 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1520 IPC_END_MESSAGE_MAP()
1521 return true;
1522 }
1523
1524 void OnNoArgs() {
1525 received_msg_ = true;
1526 PossiblyDone();
1527 }
1528
1529 void PossiblyDone() {
1530 if (received_noarg_reply_ && received_msg_) {
1531 DCHECK(done_issued_ == false);
1532 done_issued_ = true;
1533 Send(new SyncChannelTestMsg_Done);
1534 Done();
1535 }
1536 }
1537
1538 RestrictedDispatchDeadlockServer* server_;
1539 RestrictedDispatchDeadlockClient2* peer_;
1540 WaitableEvent* server_ready_event_;
1541 WaitableEvent** events_;
1542 bool received_msg_;
1543 bool received_noarg_reply_;
1544 bool done_issued_;
1545};
1546
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001547TEST_F(IPCSyncChannelTest, RestrictedDispatchDeadlock) {
1548 std::vector<Worker*> workers;
1549
1550 // A shared worker thread so that server1 and server2 run on one thread.
1551 base::Thread worker_thread("RestrictedDispatchDeadlock");
1552 ASSERT_TRUE(worker_thread.Start());
1553
1554 WaitableEvent server1_ready(false, false);
1555 WaitableEvent server2_ready(false, false);
1556
1557 WaitableEvent event0(false, false);
1558 WaitableEvent event1(false, false);
1559 WaitableEvent event2(false, false);
1560 WaitableEvent event3(false, false);
1561 WaitableEvent* events[4] = {&event0, &event1, &event2, &event3};
1562
1563 RestrictedDispatchDeadlockServer* server1;
1564 RestrictedDispatchDeadlockServer* server2;
1565 RestrictedDispatchDeadlockClient1* client1;
1566 RestrictedDispatchDeadlockClient2* client2;
1567
1568 server2 = new RestrictedDispatchDeadlockServer(2, &server2_ready, events,
1569 NULL);
1570 server2->OverrideThread(&worker_thread);
1571 workers.push_back(server2);
1572
1573 client2 = new RestrictedDispatchDeadlockClient2(server2, &server2_ready,
1574 events);
1575 workers.push_back(client2);
1576
1577 server1 = new RestrictedDispatchDeadlockServer(1, &server1_ready, events,
1578 server2);
1579 server1->OverrideThread(&worker_thread);
1580 workers.push_back(server1);
1581
1582 client1 = new RestrictedDispatchDeadlockClient1(server1, client2,
1583 &server1_ready, events);
1584 workers.push_back(client1);
1585
1586 RunTest(workers);
1587}
1588
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +09001589//------------------------------------------------------------------------------
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001590
piman@chromium.org5a00b882012-03-31 06:29:30 +09001591// This test case inspired by crbug.com/120530
1592// We create 4 workers that pipe to each other W1->W2->W3->W4->W1 then we send a
1593// message that recurses through 3, 4 or 5 steps to make sure, say, W1 can
1594// re-enter when called from W4 while it's sending a message to W2.
1595// The first worker drives the whole test so it must be treated specially.
piman@chromium.org5a00b882012-03-31 06:29:30 +09001596
1597class RestrictedDispatchPipeWorker : public Worker {
1598 public:
1599 RestrictedDispatchPipeWorker(
1600 const std::string &channel1,
1601 WaitableEvent* event1,
1602 const std::string &channel2,
1603 WaitableEvent* event2,
1604 int group,
1605 int* success)
1606 : Worker(channel1, Channel::MODE_SERVER),
1607 event1_(event1),
1608 event2_(event2),
1609 other_channel_name_(channel2),
1610 group_(group),
1611 success_(success) {
1612 }
1613
1614 void OnPingTTL(int ping, int* ret) {
1615 *ret = 0;
1616 if (!ping)
1617 return;
1618 other_channel_->Send(new SyncChannelTestMsg_PingTTL(ping - 1, ret));
1619 ++*ret;
1620 }
1621
1622 void OnDone() {
1623 if (is_first())
1624 return;
1625 other_channel_->Send(new SyncChannelTestMsg_Done);
1626 other_channel_.reset();
1627 Done();
1628 }
1629
1630 void Run() {
1631 channel()->SetRestrictDispatchChannelGroup(group_);
1632 if (is_first())
1633 event1_->Signal();
1634 event2_->Wait();
1635 other_channel_.reset(new SyncChannel(
1636 other_channel_name_, Channel::MODE_CLIENT, this,
1637 ipc_thread().message_loop_proxy(), true, shutdown_event()));
1638 other_channel_->SetRestrictDispatchChannelGroup(group_);
1639 if (!is_first()) {
1640 event1_->Signal();
1641 return;
1642 }
1643 *success_ = 0;
1644 int value = 0;
1645 OnPingTTL(3, &value);
1646 *success_ += (value == 3);
1647 OnPingTTL(4, &value);
1648 *success_ += (value == 4);
1649 OnPingTTL(5, &value);
1650 *success_ += (value == 5);
1651 other_channel_->Send(new SyncChannelTestMsg_Done);
1652 other_channel_.reset();
1653 Done();
1654 }
1655
1656 bool is_first() { return !!success_; }
1657
1658 private:
1659 bool OnMessageReceived(const Message& message) {
1660 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchPipeWorker, message)
1661 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_PingTTL, OnPingTTL)
1662 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, OnDone)
1663 IPC_END_MESSAGE_MAP()
1664 return true;
1665 }
1666
1667 scoped_ptr<SyncChannel> other_channel_;
1668 WaitableEvent* event1_;
1669 WaitableEvent* event2_;
1670 std::string other_channel_name_;
1671 int group_;
1672 int* success_;
1673};
1674
piman@chromium.org5a00b882012-03-31 06:29:30 +09001675TEST_F(IPCSyncChannelTest, RestrictedDispatch4WayDeadlock) {
1676 int success = 0;
1677 std::vector<Worker*> workers;
1678 WaitableEvent event0(true, false);
1679 WaitableEvent event1(true, false);
1680 WaitableEvent event2(true, false);
1681 WaitableEvent event3(true, false);
1682 workers.push_back(new RestrictedDispatchPipeWorker(
1683 "channel0", &event0, "channel1", &event1, 1, &success));
1684 workers.push_back(new RestrictedDispatchPipeWorker(
1685 "channel1", &event1, "channel2", &event2, 2, NULL));
1686 workers.push_back(new RestrictedDispatchPipeWorker(
1687 "channel2", &event2, "channel3", &event3, 3, NULL));
1688 workers.push_back(new RestrictedDispatchPipeWorker(
1689 "channel3", &event3, "channel0", &event0, 4, NULL));
1690 RunTest(workers);
1691 EXPECT_EQ(3, success);
1692}
1693
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +09001694//------------------------------------------------------------------------------
piman@chromium.orgde4e75e2012-04-11 05:07:53 +09001695
piman@chromium.orgde4e75e2012-04-11 05:07:53 +09001696// This test case inspired by crbug.com/122443
1697// We want to make sure a reply message with the unblock flag set correctly
1698// behaves as a reply, not a regular message.
1699// We have 3 workers. Server1 will send a message to Server2 (which will block),
1700// during which it will dispatch a message comming from Client, at which point
1701// it will send another message to Server2. While sending that second message it
1702// will receive a reply from Server1 with the unblock flag.
1703
piman@chromium.orgde4e75e2012-04-11 05:07:53 +09001704class ReentrantReplyServer1 : public Worker {
1705 public:
1706 ReentrantReplyServer1(WaitableEvent* server_ready)
1707 : Worker("reentrant_reply1", Channel::MODE_SERVER),
1708 server_ready_(server_ready) { }
1709
1710 void Run() {
1711 server2_channel_.reset(new SyncChannel(
1712 "reentrant_reply2", Channel::MODE_CLIENT, this,
1713 ipc_thread().message_loop_proxy(), true, shutdown_event()));
1714 server_ready_->Signal();
1715 Message* msg = new SyncChannelTestMsg_Reentrant1();
1716 server2_channel_->Send(msg);
1717 server2_channel_.reset();
1718 Done();
1719 }
1720
1721 private:
1722 bool OnMessageReceived(const Message& message) {
1723 IPC_BEGIN_MESSAGE_MAP(ReentrantReplyServer1, message)
1724 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Reentrant2, OnReentrant2)
1725 IPC_REPLY_HANDLER(OnReply)
1726 IPC_END_MESSAGE_MAP()
1727 return true;
1728 }
1729
1730 void OnReentrant2() {
1731 Message* msg = new SyncChannelTestMsg_Reentrant3();
1732 server2_channel_->Send(msg);
1733 }
1734
1735 void OnReply(const Message& message) {
1736 // If we get here, the Send() will never receive the reply (thus would
1737 // hang), so abort instead.
1738 LOG(FATAL) << "Reply message was dispatched";
1739 }
1740
1741 WaitableEvent* server_ready_;
1742 scoped_ptr<SyncChannel> server2_channel_;
1743};
1744
1745class ReentrantReplyServer2 : public Worker {
1746 public:
1747 ReentrantReplyServer2()
1748 : Worker("reentrant_reply2", Channel::MODE_SERVER),
1749 reply_(NULL) { }
1750
1751 private:
1752 bool OnMessageReceived(const Message& message) {
1753 IPC_BEGIN_MESSAGE_MAP(ReentrantReplyServer2, message)
1754 IPC_MESSAGE_HANDLER_DELAY_REPLY(
1755 SyncChannelTestMsg_Reentrant1, OnReentrant1)
1756 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Reentrant3, OnReentrant3)
1757 IPC_END_MESSAGE_MAP()
1758 return true;
1759 }
1760
1761 void OnReentrant1(Message* reply) {
1762 DCHECK(!reply_);
1763 reply_ = reply;
1764 }
1765
1766 void OnReentrant3() {
1767 DCHECK(reply_);
1768 Message* reply = reply_;
1769 reply_ = NULL;
1770 reply->set_unblock(true);
1771 Send(reply);
1772 Done();
1773 }
1774
1775 Message* reply_;
1776};
1777
1778class ReentrantReplyClient : public Worker {
1779 public:
1780 ReentrantReplyClient(WaitableEvent* server_ready)
1781 : Worker("reentrant_reply1", Channel::MODE_CLIENT),
1782 server_ready_(server_ready) { }
1783
1784 void Run() {
1785 server_ready_->Wait();
1786 Send(new SyncChannelTestMsg_Reentrant2());
1787 Done();
1788 }
1789
1790 private:
1791 WaitableEvent* server_ready_;
1792};
1793
piman@chromium.orgde4e75e2012-04-11 05:07:53 +09001794TEST_F(IPCSyncChannelTest, ReentrantReply) {
1795 std::vector<Worker*> workers;
1796 WaitableEvent server_ready(false, false);
1797 workers.push_back(new ReentrantReplyServer2());
1798 workers.push_back(new ReentrantReplyServer1(&server_ready));
1799 workers.push_back(new ReentrantReplyClient(&server_ready));
1800 RunTest(workers);
1801}
1802
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +09001803//------------------------------------------------------------------------------
piman@chromium.org5a00b882012-03-31 06:29:30 +09001804
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001805// Generate a validated channel ID using Channel::GenerateVerifiedChannelID().
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001806
1807class VerifiedServer : public Worker {
1808 public:
1809 VerifiedServer(base::Thread* listener_thread,
1810 const std::string& channel_name,
1811 const std::string& reply_text)
1812 : Worker(channel_name, Channel::MODE_SERVER),
1813 reply_text_(reply_text) {
1814 Worker::OverrideThread(listener_thread);
1815 }
1816
1817 virtual void OnNestedTestMsg(Message* reply_msg) {
1818 VLOG(1) << __FUNCTION__ << " Sending reply: " << reply_text_;
1819 SyncChannelNestedTestMsg_String::WriteReplyParams(reply_msg, reply_text_);
1820 Send(reply_msg);
jschuh@chromium.orga5cd0762012-04-05 11:38:34 +09001821 ASSERT_EQ(channel()->peer_pid(), base::GetCurrentProcId());
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001822 Done();
1823 }
1824
1825 private:
1826 std::string reply_text_;
1827};
1828
1829class VerifiedClient : public Worker {
1830 public:
1831 VerifiedClient(base::Thread* listener_thread,
1832 const std::string& channel_name,
1833 const std::string& expected_text)
1834 : Worker(channel_name, Channel::MODE_CLIENT),
1835 expected_text_(expected_text) {
1836 Worker::OverrideThread(listener_thread);
1837 }
1838
1839 virtual void Run() {
1840 std::string response;
1841 SyncMessage* msg = new SyncChannelNestedTestMsg_String(&response);
1842 bool result = Send(msg);
1843 DCHECK(result);
1844 DCHECK_EQ(response, expected_text_);
thakis@chromium.orgcdf5b3b2012-06-28 01:07:34 +09001845 // expected_text_ is only used in the above DCHECK. This line suppresses the
1846 // "unused private field" warning in release builds.
1847 (void)expected_text_;
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001848
1849 VLOG(1) << __FUNCTION__ << " Received reply: " << response;
jschuh@chromium.orga5cd0762012-04-05 11:38:34 +09001850 ASSERT_EQ(channel()->peer_pid(), base::GetCurrentProcId());
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001851 Done();
1852 }
1853
1854 private:
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001855 std::string expected_text_;
1856};
1857
1858void Verified() {
1859 std::vector<Worker*> workers;
1860
1861 // A shared worker thread for servers
1862 base::Thread server_worker_thread("Verified_ServerListener");
1863 ASSERT_TRUE(server_worker_thread.Start());
1864
1865 base::Thread client_worker_thread("Verified_ClientListener");
1866 ASSERT_TRUE(client_worker_thread.Start());
1867
1868 std::string channel_id = Channel::GenerateVerifiedChannelID("Verified");
1869 Worker* worker;
1870
1871 worker = new VerifiedServer(&server_worker_thread,
1872 channel_id,
1873 "Got first message");
1874 workers.push_back(worker);
1875
1876 worker = new VerifiedClient(&client_worker_thread,
1877 channel_id,
1878 "Got first message");
1879 workers.push_back(worker);
1880
1881 RunTest(workers);
1882
1883#if defined(OS_WIN)
1884#endif
1885}
1886
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001887// Windows needs to send an out-of-band secret to verify the client end of the
1888// channel. Test that we still connect correctly in that case.
1889TEST_F(IPCSyncChannelTest, Verified) {
1890 Verified();
1891}
1892
viettrungluu@chromium.org7ca19132013-01-12 05:56:22 +09001893} // namespace