blob: 0424b2fb17d667668d826c8b6ddf993334aa3ec0 [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"
dilmah@chromium.orgdc4b9702011-07-20 07:13:24 +090018#include "base/stl_util.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090019#include "base/string_util.h"
timurrrr@chromium.orgf39c3ff2010-05-14 17:24:42 +090020#include "base/third_party/dynamic_annotations/dynamic_annotations.h"
brettw@google.com7c5cc672011-01-01 11:17:08 +090021#include "base/threading/platform_thread.h"
brettw@chromium.org5b5f5e02011-01-01 10:01:06 +090022#include "base/threading/thread.h"
brettw@chromium.org5238c7d2011-01-02 15:05:39 +090023#include "base/synchronization/waitable_event.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090024#include "ipc/ipc_message.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;
30
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +090031namespace IPC {
32
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090033namespace {
34
35// Base class for a "process" with listener and IPC threads.
36class Worker : public Channel::Listener, public Message::Sender {
37 public:
38 // Will create a channel without a name.
39 Worker(Channel::Mode mode, const std::string& thread_name)
40 : done_(new WaitableEvent(false, false)),
41 channel_created_(new WaitableEvent(false, false)),
42 mode_(mode),
43 ipc_thread_((thread_name + "_ipc").c_str()),
44 listener_thread_((thread_name + "_listener").c_str()),
45 overrided_thread_(NULL),
timurrrr@chromium.org03100a82009-10-27 20:28:58 +090046 shutdown_event_(true, false) {
47 // The data race on vfptr is real but is very hard
48 // to suppress using standard Valgrind mechanism (suppressions).
49 // We have to use ANNOTATE_BENIGN_RACE to hide the reports and
50 // make ThreadSanitizer bots green.
51 ANNOTATE_BENIGN_RACE(this, "Race on vfptr, http://crbug.com/25841");
52 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090053
54 // Will create a named channel and use this name for the threads' name.
55 Worker(const std::string& channel_name, Channel::Mode mode)
56 : done_(new WaitableEvent(false, false)),
57 channel_created_(new WaitableEvent(false, false)),
58 channel_name_(channel_name),
59 mode_(mode),
60 ipc_thread_((channel_name + "_ipc").c_str()),
61 listener_thread_((channel_name + "_listener").c_str()),
62 overrided_thread_(NULL),
timurrrr@chromium.org03100a82009-10-27 20:28:58 +090063 shutdown_event_(true, false) {
64 // The data race on vfptr is real but is very hard
65 // to suppress using standard Valgrind mechanism (suppressions).
66 // We have to use ANNOTATE_BENIGN_RACE to hide the reports and
67 // make ThreadSanitizer bots green.
68 ANNOTATE_BENIGN_RACE(this, "Race on vfptr, http://crbug.com/25841");
69 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090070
71 // The IPC thread needs to outlive SyncChannel, so force the correct order of
72 // destruction.
73 virtual ~Worker() {
74 WaitableEvent listener_done(false, false), ipc_done(false, false);
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +090075 ListenerThread()->message_loop()->PostTask(
76 FROM_HERE, base::Bind(&Worker::OnListenerThreadShutdown1, this,
77 &listener_done, &ipc_done));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090078 listener_done.Wait();
79 ipc_done.Wait();
80 ipc_thread_.Stop();
81 listener_thread_.Stop();
82 }
83 void AddRef() { }
84 void Release() { }
darin@chromium.org5cb996e2009-09-30 13:29:20 +090085 static bool ImplementsThreadSafeReferenceCounting() { return true; }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090086 bool Send(Message* msg) { return channel_->Send(msg); }
87 bool SendWithTimeout(Message* msg, int timeout_ms) {
88 return channel_->SendWithTimeout(msg, timeout_ms);
89 }
90 void WaitForChannelCreation() { channel_created_->Wait(); }
91 void CloseChannel() {
92 DCHECK(MessageLoop::current() == ListenerThread()->message_loop());
93 channel_->Close();
94 }
95 void Start() {
96 StartThread(&listener_thread_, MessageLoop::TYPE_DEFAULT);
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +090097 ListenerThread()->message_loop()->PostTask(
98 FROM_HERE, base::Bind(&Worker::OnStart, this));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090099 }
100 void OverrideThread(base::Thread* overrided_thread) {
101 DCHECK(overrided_thread_ == NULL);
102 overrided_thread_ = overrided_thread;
103 }
104 bool SendAnswerToLife(bool pump, int timeout, bool succeed) {
105 int answer = 0;
106 SyncMessage* msg = new SyncChannelTestMsg_AnswerToLife(&answer);
107 if (pump)
108 msg->EnableMessagePumping();
109 bool result = SendWithTimeout(msg, timeout);
jam@chromium.orgebd07182009-12-01 11:34:18 +0900110 DCHECK_EQ(result, succeed);
111 DCHECK_EQ(answer, (succeed ? 42 : 0));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900112 return result;
113 }
114 bool SendDouble(bool pump, bool succeed) {
115 int answer = 0;
116 SyncMessage* msg = new SyncChannelTestMsg_Double(5, &answer);
117 if (pump)
118 msg->EnableMessagePumping();
119 bool result = Send(msg);
jam@chromium.orgebd07182009-12-01 11:34:18 +0900120 DCHECK_EQ(result, succeed);
121 DCHECK_EQ(answer, (succeed ? 10 : 0));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900122 return result;
123 }
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900124 const std::string& channel_name() { return channel_name_; }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900125 Channel::Mode mode() { return mode_; }
126 WaitableEvent* done_event() { return done_.get(); }
jabdelmalek@google.comeb921652010-04-07 05:33:36 +0900127 WaitableEvent* shutdown_event() { return &shutdown_event_; }
jam@chromium.orgebd07182009-12-01 11:34:18 +0900128 void ResetChannel() { channel_.reset(); }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900129 // Derived classes need to call this when they've completed their part of
130 // the test.
131 void Done() { done_->Signal(); }
jabdelmalek@google.comeb921652010-04-07 05:33:36 +0900132
133 protected:
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +0900134 SyncChannel* channel() { return channel_.get(); }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900135 // Functions for dervied classes to implement if they wish.
136 virtual void Run() { }
137 virtual void OnAnswer(int* answer) { NOTREACHED(); }
138 virtual void OnAnswerDelay(Message* reply_msg) {
139 // The message handler map below can only take one entry for
140 // SyncChannelTestMsg_AnswerToLife, so since some classes want
141 // the normal version while other want the delayed reply, we
142 // call the normal version if the derived class didn't override
143 // this function.
144 int answer;
145 OnAnswer(&answer);
146 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, answer);
147 Send(reply_msg);
148 }
149 virtual void OnDouble(int in, int* out) { NOTREACHED(); }
150 virtual void OnDoubleDelay(int in, Message* reply_msg) {
151 int result;
152 OnDouble(in, &result);
153 SyncChannelTestMsg_Double::WriteReplyParams(reply_msg, result);
154 Send(reply_msg);
155 }
156
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900157 virtual void OnNestedTestMsg(Message* reply_msg) {
158 NOTREACHED();
159 }
160
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900161 virtual SyncChannel* CreateChannel() {
162 return new SyncChannel(
163 channel_name_, mode_, this, ipc_thread_.message_loop_proxy(), true,
164 &shutdown_event_);
165 }
166
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900167 base::Thread* ListenerThread() {
168 return overrided_thread_ ? overrided_thread_ : &listener_thread_;
169 }
ananta@chromium.org999f2972010-09-03 06:45:50 +0900170
piman@google.com0cbefaa2011-04-08 12:38:21 +0900171 const base::Thread& ipc_thread() const { return ipc_thread_; }
172
ananta@chromium.org999f2972010-09-03 06:45:50 +0900173 private:
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900174 // Called on the listener thread to create the sync channel.
175 void OnStart() {
176 // Link ipc_thread_, listener_thread_ and channel_ altogether.
177 StartThread(&ipc_thread_, MessageLoop::TYPE_IO);
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900178 channel_.reset(CreateChannel());
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900179 channel_created_->Signal();
180 Run();
181 }
182
183 void OnListenerThreadShutdown1(WaitableEvent* listener_event,
184 WaitableEvent* ipc_event) {
185 // SyncChannel needs to be destructed on the thread that it was created on.
186 channel_.reset();
187
188 MessageLoop::current()->RunAllPending();
189
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +0900190 ipc_thread_.message_loop()->PostTask(
191 FROM_HERE, base::Bind(&Worker::OnIPCThreadShutdown, this,
192 listener_event, ipc_event));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900193 }
194
195 void OnIPCThreadShutdown(WaitableEvent* listener_event,
196 WaitableEvent* ipc_event) {
197 MessageLoop::current()->RunAllPending();
198 ipc_event->Signal();
199
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +0900200 listener_thread_.message_loop()->PostTask(
201 FROM_HERE, base::Bind(&Worker::OnListenerThreadShutdown2, this,
202 listener_event));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900203 }
204
205 void OnListenerThreadShutdown2(WaitableEvent* listener_event) {
206 MessageLoop::current()->RunAllPending();
207 listener_event->Signal();
208 }
209
jam@chromium.org8a2c7842010-12-24 15:19:28 +0900210 bool OnMessageReceived(const Message& message) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900211 IPC_BEGIN_MESSAGE_MAP(Worker, message)
212 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelTestMsg_Double, OnDoubleDelay)
213 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelTestMsg_AnswerToLife,
214 OnAnswerDelay)
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900215 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelNestedTestMsg_String,
216 OnNestedTestMsg)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900217 IPC_END_MESSAGE_MAP()
jam@chromium.org8a2c7842010-12-24 15:19:28 +0900218 return true;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900219 }
220
221 void StartThread(base::Thread* thread, MessageLoop::Type type) {
222 base::Thread::Options options;
223 options.message_loop_type = type;
224 thread->StartWithOptions(options);
225 }
226
227 scoped_ptr<WaitableEvent> done_;
228 scoped_ptr<WaitableEvent> channel_created_;
229 std::string channel_name_;
230 Channel::Mode mode_;
231 scoped_ptr<SyncChannel> channel_;
232 base::Thread ipc_thread_;
233 base::Thread listener_thread_;
234 base::Thread* overrided_thread_;
235
236 base::WaitableEvent shutdown_event_;
237
tfarina@chromium.orgb73eaee2010-06-07 11:10:18 +0900238 DISALLOW_COPY_AND_ASSIGN(Worker);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900239};
240
241
242// Starts the test with the given workers. This function deletes the workers
243// when it's done.
244void RunTest(std::vector<Worker*> workers) {
245 // First we create the workers that are channel servers, or else the other
246 // workers' channel initialization might fail because the pipe isn't created..
247 for (size_t i = 0; i < workers.size(); ++i) {
dmaclach@chromium.orgf146c292011-02-04 05:35:09 +0900248 if (workers[i]->mode() & Channel::MODE_SERVER_FLAG) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900249 workers[i]->Start();
250 workers[i]->WaitForChannelCreation();
251 }
252 }
253
254 // now create the clients
255 for (size_t i = 0; i < workers.size(); ++i) {
dmaclach@chromium.orgf146c292011-02-04 05:35:09 +0900256 if (workers[i]->mode() & Channel::MODE_CLIENT_FLAG)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900257 workers[i]->Start();
258 }
259
260 // wait for all the workers to finish
261 for (size_t i = 0; i < workers.size(); ++i)
262 workers[i]->done_event()->Wait();
263
264 STLDeleteContainerPointers(workers.begin(), workers.end());
265}
266
267} // namespace
268
269class IPCSyncChannelTest : public testing::Test {
270 private:
271 MessageLoop message_loop_;
272};
273
274//-----------------------------------------------------------------------------
275
276namespace {
277
278class SimpleServer : public Worker {
279 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900280 explicit SimpleServer(bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900281 : Worker(Channel::MODE_SERVER, "simpler_server"),
282 pump_during_send_(pump_during_send) { }
283 void Run() {
284 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
285 Done();
286 }
287
288 bool pump_during_send_;
289};
290
291class SimpleClient : public Worker {
292 public:
293 SimpleClient() : Worker(Channel::MODE_CLIENT, "simple_client") { }
294
295 void OnAnswer(int* answer) {
296 *answer = 42;
297 Done();
298 }
299};
300
301void Simple(bool pump_during_send) {
302 std::vector<Worker*> workers;
303 workers.push_back(new SimpleServer(pump_during_send));
304 workers.push_back(new SimpleClient());
305 RunTest(workers);
306}
307
308} // namespace
309
310// Tests basic synchronous call
311TEST_F(IPCSyncChannelTest, Simple) {
312 Simple(false);
313 Simple(true);
314}
315
316//-----------------------------------------------------------------------------
317
318namespace {
319
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900320// Worker classes which override how the sync channel is created to use the
321// two-step initialization (calling the lightweight constructor and then
322// ChannelProxy::Init separately) process.
323class TwoStepServer : public Worker {
324 public:
325 explicit TwoStepServer(bool create_pipe_now)
326 : Worker(Channel::MODE_SERVER, "simpler_server"),
327 create_pipe_now_(create_pipe_now) { }
328
329 void Run() {
330 SendAnswerToLife(false, base::kNoTimeout, true);
331 Done();
332 }
333
334 virtual SyncChannel* CreateChannel() {
335 SyncChannel* channel = new SyncChannel(
336 this, ipc_thread().message_loop_proxy(), shutdown_event());
337 channel->Init(channel_name(), mode(), create_pipe_now_);
338 return channel;
339 }
340
341 bool create_pipe_now_;
342};
343
344class TwoStepClient : public Worker {
345 public:
346 TwoStepClient(bool create_pipe_now)
347 : Worker(Channel::MODE_CLIENT, "simple_client"),
348 create_pipe_now_(create_pipe_now) { }
349
350 void OnAnswer(int* answer) {
351 *answer = 42;
352 Done();
353 }
354
355 virtual SyncChannel* CreateChannel() {
356 SyncChannel* channel = new SyncChannel(
357 this, ipc_thread().message_loop_proxy(), shutdown_event());
358 channel->Init(channel_name(), mode(), create_pipe_now_);
359 return channel;
360 }
361
362 bool create_pipe_now_;
363};
364
365void TwoStep(bool create_server_pipe_now, bool create_client_pipe_now) {
366 std::vector<Worker*> workers;
367 workers.push_back(new TwoStepServer(create_server_pipe_now));
368 workers.push_back(new TwoStepClient(create_client_pipe_now));
369 RunTest(workers);
370}
371
372} // namespace
373
374// Tests basic two-step initialization, where you call the lightweight
375// constructor then Init.
376TEST_F(IPCSyncChannelTest, TwoStepInitialization) {
377 TwoStep(false, false);
378 TwoStep(false, true);
379 TwoStep(true, false);
380 TwoStep(true, true);
381}
382
383
384//-----------------------------------------------------------------------------
385
386namespace {
387
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900388class DelayClient : public Worker {
389 public:
390 DelayClient() : Worker(Channel::MODE_CLIENT, "delay_client") { }
391
392 void OnAnswerDelay(Message* reply_msg) {
393 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
394 Send(reply_msg);
395 Done();
396 }
397};
398
399void DelayReply(bool pump_during_send) {
400 std::vector<Worker*> workers;
401 workers.push_back(new SimpleServer(pump_during_send));
402 workers.push_back(new DelayClient());
403 RunTest(workers);
404}
405
406} // namespace
407
408// Tests that asynchronous replies work
409TEST_F(IPCSyncChannelTest, DelayReply) {
410 DelayReply(false);
411 DelayReply(true);
412}
413
414//-----------------------------------------------------------------------------
415
416namespace {
417
418class NoHangServer : public Worker {
419 public:
dilmah@chromium.org92b4c8e2011-07-27 02:25:25 +0900420 NoHangServer(WaitableEvent* got_first_reply, bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900421 : Worker(Channel::MODE_SERVER, "no_hang_server"),
422 got_first_reply_(got_first_reply),
423 pump_during_send_(pump_during_send) { }
424 void Run() {
425 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
426 got_first_reply_->Signal();
427
428 SendAnswerToLife(pump_during_send_, base::kNoTimeout, false);
429 Done();
430 }
431
432 WaitableEvent* got_first_reply_;
433 bool pump_during_send_;
434};
435
436class NoHangClient : public Worker {
437 public:
438 explicit NoHangClient(WaitableEvent* got_first_reply)
439 : Worker(Channel::MODE_CLIENT, "no_hang_client"),
440 got_first_reply_(got_first_reply) { }
441
442 virtual void OnAnswerDelay(Message* reply_msg) {
443 // Use the DELAY_REPLY macro so that we can force the reply to be sent
444 // before this function returns (when the channel will be reset).
445 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
446 Send(reply_msg);
447 got_first_reply_->Wait();
448 CloseChannel();
449 Done();
450 }
451
452 WaitableEvent* got_first_reply_;
453};
454
455void NoHang(bool pump_during_send) {
456 WaitableEvent got_first_reply(false, false);
457 std::vector<Worker*> workers;
458 workers.push_back(new NoHangServer(&got_first_reply, pump_during_send));
459 workers.push_back(new NoHangClient(&got_first_reply));
460 RunTest(workers);
461}
462
463} // namespace
464
465// Tests that caller doesn't hang if receiver dies
466TEST_F(IPCSyncChannelTest, NoHang) {
467 NoHang(false);
468 NoHang(true);
469}
470
471//-----------------------------------------------------------------------------
472
473namespace {
474
475class UnblockServer : public Worker {
476 public:
jam@chromium.orgebd07182009-12-01 11:34:18 +0900477 UnblockServer(bool pump_during_send, bool delete_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900478 : Worker(Channel::MODE_SERVER, "unblock_server"),
jam@chromium.orgebd07182009-12-01 11:34:18 +0900479 pump_during_send_(pump_during_send),
480 delete_during_send_(delete_during_send) { }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900481 void Run() {
jam@chromium.orgebd07182009-12-01 11:34:18 +0900482 if (delete_during_send_) {
483 // Use custom code since race conditions mean the answer may or may not be
484 // available.
485 int answer = 0;
486 SyncMessage* msg = new SyncChannelTestMsg_AnswerToLife(&answer);
487 if (pump_during_send_)
488 msg->EnableMessagePumping();
489 Send(msg);
490 } else {
491 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
492 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900493 Done();
494 }
495
jam@chromium.orgebd07182009-12-01 11:34:18 +0900496 void OnDoubleDelay(int in, Message* reply_msg) {
497 SyncChannelTestMsg_Double::WriteReplyParams(reply_msg, in * 2);
498 Send(reply_msg);
499 if (delete_during_send_)
500 ResetChannel();
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900501 }
502
503 bool pump_during_send_;
jam@chromium.orgebd07182009-12-01 11:34:18 +0900504 bool delete_during_send_;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900505};
506
507class UnblockClient : public Worker {
508 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900509 explicit UnblockClient(bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900510 : Worker(Channel::MODE_CLIENT, "unblock_client"),
511 pump_during_send_(pump_during_send) { }
512
513 void OnAnswer(int* answer) {
514 SendDouble(pump_during_send_, true);
515 *answer = 42;
516 Done();
517 }
518
519 bool pump_during_send_;
520};
521
jam@chromium.orgebd07182009-12-01 11:34:18 +0900522void Unblock(bool server_pump, bool client_pump, bool delete_during_send) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900523 std::vector<Worker*> workers;
jam@chromium.orgebd07182009-12-01 11:34:18 +0900524 workers.push_back(new UnblockServer(server_pump, delete_during_send));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900525 workers.push_back(new UnblockClient(client_pump));
526 RunTest(workers);
527}
528
529} // namespace
530
531// Tests that the caller unblocks to answer a sync message from the receiver.
532TEST_F(IPCSyncChannelTest, Unblock) {
jam@chromium.orgebd07182009-12-01 11:34:18 +0900533 Unblock(false, false, false);
534 Unblock(false, true, false);
535 Unblock(true, false, false);
536 Unblock(true, true, false);
537}
538
539//-----------------------------------------------------------------------------
540
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +0900541// Tests that the the SyncChannel object can be deleted during a Send.
jam@chromium.orgebd07182009-12-01 11:34:18 +0900542TEST_F(IPCSyncChannelTest, ChannelDeleteDuringSend) {
543 Unblock(false, false, true);
544 Unblock(false, true, true);
545 Unblock(true, false, true);
546 Unblock(true, true, true);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900547}
548
549//-----------------------------------------------------------------------------
550
551namespace {
552
553class RecursiveServer : public Worker {
554 public:
dilmah@chromium.org92b4c8e2011-07-27 02:25:25 +0900555 RecursiveServer(bool expected_send_result, bool pump_first, bool pump_second)
556 : Worker(Channel::MODE_SERVER, "recursive_server"),
557 expected_send_result_(expected_send_result),
558 pump_first_(pump_first), pump_second_(pump_second) {}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900559 void Run() {
560 SendDouble(pump_first_, expected_send_result_);
561 Done();
562 }
563
564 void OnDouble(int in, int* out) {
565 *out = in * 2;
566 SendAnswerToLife(pump_second_, base::kNoTimeout, expected_send_result_);
567 }
568
569 bool expected_send_result_, pump_first_, pump_second_;
570};
571
572class RecursiveClient : public Worker {
573 public:
dilmah@chromium.org92b4c8e2011-07-27 02:25:25 +0900574 RecursiveClient(bool pump_during_send, bool close_channel)
575 : Worker(Channel::MODE_CLIENT, "recursive_client"),
576 pump_during_send_(pump_during_send), close_channel_(close_channel) {}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900577
578 void OnDoubleDelay(int in, Message* reply_msg) {
579 SendDouble(pump_during_send_, !close_channel_);
580 if (close_channel_) {
581 delete reply_msg;
582 } else {
583 SyncChannelTestMsg_Double::WriteReplyParams(reply_msg, in * 2);
584 Send(reply_msg);
585 }
586 Done();
587 }
588
589 void OnAnswerDelay(Message* reply_msg) {
590 if (close_channel_) {
591 delete reply_msg;
592 CloseChannel();
593 } else {
594 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
595 Send(reply_msg);
596 }
597 }
598
599 bool pump_during_send_, close_channel_;
600};
601
602void Recursive(
603 bool server_pump_first, bool server_pump_second, bool client_pump) {
604 std::vector<Worker*> workers;
605 workers.push_back(
606 new RecursiveServer(true, server_pump_first, server_pump_second));
607 workers.push_back(new RecursiveClient(client_pump, false));
608 RunTest(workers);
609}
610
611} // namespace
612
613// Tests a server calling Send while another Send is pending.
614TEST_F(IPCSyncChannelTest, Recursive) {
615 Recursive(false, false, false);
616 Recursive(false, false, true);
617 Recursive(false, true, false);
618 Recursive(false, true, true);
619 Recursive(true, false, false);
620 Recursive(true, false, true);
621 Recursive(true, true, false);
622 Recursive(true, true, true);
623}
624
625//-----------------------------------------------------------------------------
626
627namespace {
628
629void RecursiveNoHang(
630 bool server_pump_first, bool server_pump_second, bool client_pump) {
631 std::vector<Worker*> workers;
632 workers.push_back(
633 new RecursiveServer(false, server_pump_first, server_pump_second));
634 workers.push_back(new RecursiveClient(client_pump, true));
635 RunTest(workers);
636}
637
638} // namespace
639
640// Tests that if a caller makes a sync call during an existing sync call and
641// the receiver dies, neither of the Send() calls hang.
642TEST_F(IPCSyncChannelTest, RecursiveNoHang) {
643 RecursiveNoHang(false, false, false);
644 RecursiveNoHang(false, false, true);
645 RecursiveNoHang(false, true, false);
646 RecursiveNoHang(false, true, true);
647 RecursiveNoHang(true, false, false);
648 RecursiveNoHang(true, false, true);
649 RecursiveNoHang(true, true, false);
650 RecursiveNoHang(true, true, true);
651}
652
653//-----------------------------------------------------------------------------
654
655namespace {
656
657class MultipleServer1 : public Worker {
658 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900659 explicit MultipleServer1(bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900660 : Worker("test_channel1", Channel::MODE_SERVER),
661 pump_during_send_(pump_during_send) { }
662
663 void Run() {
664 SendDouble(pump_during_send_, true);
665 Done();
666 }
667
668 bool pump_during_send_;
669};
670
671class MultipleClient1 : public Worker {
672 public:
673 MultipleClient1(WaitableEvent* client1_msg_received,
674 WaitableEvent* client1_can_reply) :
675 Worker("test_channel1", Channel::MODE_CLIENT),
676 client1_msg_received_(client1_msg_received),
677 client1_can_reply_(client1_can_reply) { }
678
679 void OnDouble(int in, int* out) {
680 client1_msg_received_->Signal();
681 *out = in * 2;
682 client1_can_reply_->Wait();
683 Done();
684 }
685
686 private:
687 WaitableEvent *client1_msg_received_, *client1_can_reply_;
688};
689
690class MultipleServer2 : public Worker {
691 public:
692 MultipleServer2() : Worker("test_channel2", Channel::MODE_SERVER) { }
693
694 void OnAnswer(int* result) {
695 *result = 42;
696 Done();
697 }
698};
699
700class MultipleClient2 : public Worker {
701 public:
702 MultipleClient2(
703 WaitableEvent* client1_msg_received, WaitableEvent* client1_can_reply,
704 bool pump_during_send)
705 : Worker("test_channel2", Channel::MODE_CLIENT),
706 client1_msg_received_(client1_msg_received),
707 client1_can_reply_(client1_can_reply),
708 pump_during_send_(pump_during_send) { }
709
710 void Run() {
711 client1_msg_received_->Wait();
712 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
713 client1_can_reply_->Signal();
714 Done();
715 }
716
717 private:
718 WaitableEvent *client1_msg_received_, *client1_can_reply_;
719 bool pump_during_send_;
720};
721
722void Multiple(bool server_pump, bool client_pump) {
723 std::vector<Worker*> workers;
724
725 // A shared worker thread so that server1 and server2 run on one thread.
726 base::Thread worker_thread("Multiple");
727 ASSERT_TRUE(worker_thread.Start());
728
729 // Server1 sends a sync msg to client1, which blocks the reply until
730 // server2 (which runs on the same worker thread as server1) responds
731 // to a sync msg from client2.
732 WaitableEvent client1_msg_received(false, false);
733 WaitableEvent client1_can_reply(false, false);
734
735 Worker* worker;
736
737 worker = new MultipleServer2();
738 worker->OverrideThread(&worker_thread);
739 workers.push_back(worker);
740
741 worker = new MultipleClient2(
742 &client1_msg_received, &client1_can_reply, client_pump);
743 workers.push_back(worker);
744
745 worker = new MultipleServer1(server_pump);
746 worker->OverrideThread(&worker_thread);
747 workers.push_back(worker);
748
749 worker = new MultipleClient1(
750 &client1_msg_received, &client1_can_reply);
751 workers.push_back(worker);
752
753 RunTest(workers);
754}
755
756} // namespace
757
758// Tests that multiple SyncObjects on the same listener thread can unblock each
759// other.
760TEST_F(IPCSyncChannelTest, Multiple) {
761 Multiple(false, false);
762 Multiple(false, true);
763 Multiple(true, false);
764 Multiple(true, true);
765}
766
767//-----------------------------------------------------------------------------
768
769namespace {
770
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900771// This class provides server side functionality to test the case where
772// multiple sync channels are in use on the same thread on the client and
773// nested calls are issued.
774class QueuedReplyServer : public Worker {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900775 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900776 QueuedReplyServer(base::Thread* listener_thread,
777 const std::string& channel_name,
778 const std::string& reply_text)
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900779 : Worker(channel_name, Channel::MODE_SERVER),
780 reply_text_(reply_text) {
781 Worker::OverrideThread(listener_thread);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900782 }
783
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900784 virtual void OnNestedTestMsg(Message* reply_msg) {
pkasting@chromium.orgfcdd54b2010-10-20 08:50:00 +0900785 VLOG(1) << __FUNCTION__ << " Sending reply: " << reply_text_;
786 SyncChannelNestedTestMsg_String::WriteReplyParams(reply_msg, reply_text_);
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900787 Send(reply_msg);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900788 Done();
789 }
790
791 private:
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900792 std::string reply_text_;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900793};
794
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900795// The QueuedReplyClient class provides functionality to test the case where
796// multiple sync channels are in use on the same thread and they make nested
797// sync calls, i.e. while the first channel waits for a response it makes a
798// sync call on another channel.
799// The callstack should unwind correctly, i.e. the outermost call should
800// complete first, and so on.
801class QueuedReplyClient : public Worker {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900802 public:
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900803 QueuedReplyClient(base::Thread* listener_thread,
804 const std::string& channel_name,
805 const std::string& expected_text,
806 bool pump_during_send)
807 : Worker(channel_name, Channel::MODE_CLIENT),
thomasvl@google.com9a242072010-07-23 23:18:59 +0900808 pump_during_send_(pump_during_send),
809 expected_text_(expected_text) {
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900810 Worker::OverrideThread(listener_thread);
811 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900812
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900813 virtual void Run() {
814 std::string response;
815 SyncMessage* msg = new SyncChannelNestedTestMsg_String(&response);
816 if (pump_during_send_)
817 msg->EnableMessagePumping();
818 bool result = Send(msg);
819 DCHECK(result);
jam@chromium.orgebd07182009-12-01 11:34:18 +0900820 DCHECK_EQ(response, expected_text_);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900821
pkasting@chromium.orgfcdd54b2010-10-20 08:50:00 +0900822 VLOG(1) << __FUNCTION__ << " Received reply: " << response;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900823 Done();
824 }
825
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900826 private:
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900827 bool pump_during_send_;
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900828 std::string expected_text_;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900829};
830
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900831void QueuedReply(bool client_pump) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900832 std::vector<Worker*> workers;
833
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900834 // A shared worker thread for servers
835 base::Thread server_worker_thread("QueuedReply_ServerListener");
836 ASSERT_TRUE(server_worker_thread.Start());
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900837
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900838 base::Thread client_worker_thread("QueuedReply_ClientListener");
839 ASSERT_TRUE(client_worker_thread.Start());
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900840
841 Worker* worker;
842
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900843 worker = new QueuedReplyServer(&server_worker_thread,
844 "QueuedReply_Server1",
845 "Got first message");
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900846 workers.push_back(worker);
847
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900848 worker = new QueuedReplyServer(&server_worker_thread,
849 "QueuedReply_Server2",
850 "Got second message");
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900851 workers.push_back(worker);
852
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900853 worker = new QueuedReplyClient(&client_worker_thread,
854 "QueuedReply_Server1",
855 "Got first message",
856 client_pump);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900857 workers.push_back(worker);
858
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900859 worker = new QueuedReplyClient(&client_worker_thread,
860 "QueuedReply_Server2",
861 "Got second message",
862 client_pump);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900863 workers.push_back(worker);
864
865 RunTest(workers);
866}
867
868} // namespace
869
870// While a blocking send is in progress, the listener thread might answer other
871// synchronous messages. This tests that if during the response to another
872// message the reply to the original messages comes, it is queued up correctly
873// and the original Send is unblocked later.
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900874// We also test that the send call stacks unwind correctly when the channel
875// pumps messages while waiting for a response.
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900876TEST_F(IPCSyncChannelTest, QueuedReply) {
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900877 QueuedReply(false);
878 QueuedReply(true);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900879}
880
881//-----------------------------------------------------------------------------
882
883namespace {
884
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900885class ChattyClient : public Worker {
886 public:
887 ChattyClient() :
888 Worker(Channel::MODE_CLIENT, "chatty_client") { }
889
890 void OnAnswer(int* answer) {
891 // The PostMessage limit is 10k. Send 20% more than that.
892 const int kMessageLimit = 10000;
893 const int kMessagesToSend = kMessageLimit * 120 / 100;
894 for (int i = 0; i < kMessagesToSend; ++i) {
895 if (!SendDouble(false, true))
896 break;
897 }
898 *answer = 42;
899 Done();
900 }
901};
902
903void ChattyServer(bool pump_during_send) {
904 std::vector<Worker*> workers;
jam@chromium.orgebd07182009-12-01 11:34:18 +0900905 workers.push_back(new UnblockServer(pump_during_send, false));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900906 workers.push_back(new ChattyClient());
907 RunTest(workers);
908}
909
910} // namespace
911
912// Tests http://b/1093251 - that sending lots of sync messages while
913// the receiver is waiting for a sync reply does not overflow the PostMessage
914// queue.
915TEST_F(IPCSyncChannelTest, ChattyServer) {
916 ChattyServer(false);
917 ChattyServer(true);
918}
919
920//------------------------------------------------------------------------------
921
922namespace {
923
924class TimeoutServer : public Worker {
925 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900926 TimeoutServer(int timeout_ms,
927 std::vector<bool> timeout_seq,
928 bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900929 : Worker(Channel::MODE_SERVER, "timeout_server"),
930 timeout_ms_(timeout_ms),
931 timeout_seq_(timeout_seq),
932 pump_during_send_(pump_during_send) {
933 }
934
935 void Run() {
936 for (std::vector<bool>::const_iterator iter = timeout_seq_.begin();
937 iter != timeout_seq_.end(); ++iter) {
938 SendAnswerToLife(pump_during_send_, timeout_ms_, !*iter);
939 }
940 Done();
941 }
942
943 private:
944 int timeout_ms_;
945 std::vector<bool> timeout_seq_;
946 bool pump_during_send_;
947};
948
949class UnresponsiveClient : public Worker {
950 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900951 explicit UnresponsiveClient(std::vector<bool> timeout_seq)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900952 : Worker(Channel::MODE_CLIENT, "unresponsive_client"),
953 timeout_seq_(timeout_seq) {
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900954 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900955
956 void OnAnswerDelay(Message* reply_msg) {
957 DCHECK(!timeout_seq_.empty());
958 if (!timeout_seq_[0]) {
959 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
960 Send(reply_msg);
961 } else {
962 // Don't reply.
963 delete reply_msg;
964 }
965 timeout_seq_.erase(timeout_seq_.begin());
966 if (timeout_seq_.empty())
967 Done();
968 }
969
970 private:
971 // Whether we should time-out or respond to the various messages we receive.
972 std::vector<bool> timeout_seq_;
973};
974
975void SendWithTimeoutOK(bool pump_during_send) {
976 std::vector<Worker*> workers;
977 std::vector<bool> timeout_seq;
978 timeout_seq.push_back(false);
979 timeout_seq.push_back(false);
980 timeout_seq.push_back(false);
981 workers.push_back(new TimeoutServer(5000, timeout_seq, pump_during_send));
982 workers.push_back(new SimpleClient());
983 RunTest(workers);
984}
985
986void SendWithTimeoutTimeout(bool pump_during_send) {
987 std::vector<Worker*> workers;
988 std::vector<bool> timeout_seq;
989 timeout_seq.push_back(true);
990 timeout_seq.push_back(false);
991 timeout_seq.push_back(false);
992 workers.push_back(new TimeoutServer(100, timeout_seq, pump_during_send));
993 workers.push_back(new UnresponsiveClient(timeout_seq));
994 RunTest(workers);
995}
996
997void SendWithTimeoutMixedOKAndTimeout(bool pump_during_send) {
998 std::vector<Worker*> workers;
999 std::vector<bool> timeout_seq;
1000 timeout_seq.push_back(true);
1001 timeout_seq.push_back(false);
1002 timeout_seq.push_back(false);
1003 timeout_seq.push_back(true);
1004 timeout_seq.push_back(false);
1005 workers.push_back(new TimeoutServer(100, timeout_seq, pump_during_send));
1006 workers.push_back(new UnresponsiveClient(timeout_seq));
1007 RunTest(workers);
1008}
1009
1010} // namespace
1011
1012// Tests that SendWithTimeout does not time-out if the response comes back fast
1013// enough.
1014TEST_F(IPCSyncChannelTest, SendWithTimeoutOK) {
1015 SendWithTimeoutOK(false);
1016 SendWithTimeoutOK(true);
1017}
1018
1019// Tests that SendWithTimeout does time-out.
1020TEST_F(IPCSyncChannelTest, SendWithTimeoutTimeout) {
1021 SendWithTimeoutTimeout(false);
1022 SendWithTimeoutTimeout(true);
1023}
1024
1025// Sends some message that time-out and some that succeed.
phajdan.jr@chromium.orga8e3c092011-01-19 17:11:56 +09001026// Crashes flakily, http://crbug.com/70075.
1027TEST_F(IPCSyncChannelTest, DISABLED_SendWithTimeoutMixedOKAndTimeout) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001028 SendWithTimeoutMixedOKAndTimeout(false);
1029 SendWithTimeoutMixedOKAndTimeout(true);
1030}
1031
1032//------------------------------------------------------------------------------
1033
1034namespace {
1035
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001036void NestedCallback(Worker* server) {
1037 // Sleep a bit so that we wake up after the reply has been received.
tedvessenes@gmail.com30dbbaa2012-01-13 09:11:01 +09001038 base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(250));
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001039 server->SendAnswerToLife(true, base::kNoTimeout, true);
1040}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001041
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001042bool timeout_occurred = false;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001043
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001044void TimeoutCallback() {
1045 timeout_occurred = true;
1046}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001047
1048class DoneEventRaceServer : public Worker {
1049 public:
1050 DoneEventRaceServer()
1051 : Worker(Channel::MODE_SERVER, "done_event_race_server") { }
1052
1053 void Run() {
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001054 MessageLoop::current()->PostTask(FROM_HERE,
1055 base::Bind(&NestedCallback, this));
1056 MessageLoop::current()->PostDelayedTask(
tedvessenes@gmail.com56b33702012-03-07 13:41:40 +09001057 FROM_HERE,
1058 base::Bind(&TimeoutCallback),
1059 base::TimeDelta::FromSeconds(9));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001060 // Even though we have a timeout on the Send, it will succeed since for this
1061 // bug, the reply message comes back and is deserialized, however the done
1062 // event wasn't set. So we indirectly use the timeout task to notice if a
1063 // timeout occurred.
1064 SendAnswerToLife(true, 10000, true);
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001065 DCHECK(!timeout_occurred);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001066 Done();
1067 }
1068};
1069
1070} // namespace
1071
1072// Tests http://b/1474092 - that if after the done_event is set but before
1073// OnObjectSignaled is called another message is sent out, then after its
1074// reply comes back OnObjectSignaled will be called for the first message.
1075TEST_F(IPCSyncChannelTest, DoneEventRace) {
1076 std::vector<Worker*> workers;
1077 workers.push_back(new DoneEventRaceServer());
1078 workers.push_back(new SimpleClient());
1079 RunTest(workers);
1080}
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001081
1082//-----------------------------------------------------------------------------
1083
1084namespace {
1085
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001086class TestSyncMessageFilter : public SyncMessageFilter {
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001087 public:
1088 TestSyncMessageFilter(base::WaitableEvent* shutdown_event, Worker* worker)
1089 : SyncMessageFilter(shutdown_event),
1090 worker_(worker),
1091 thread_("helper_thread") {
1092 base::Thread::Options options;
1093 options.message_loop_type = MessageLoop::TYPE_DEFAULT;
1094 thread_.StartWithOptions(options);
1095 }
1096
1097 virtual void OnFilterAdded(Channel* channel) {
1098 SyncMessageFilter::OnFilterAdded(channel);
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001099 thread_.message_loop()->PostTask(
1100 FROM_HERE,
1101 base::Bind(&TestSyncMessageFilter::SendMessageOnHelperThread, this));
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001102 }
1103
1104 void SendMessageOnHelperThread() {
1105 int answer = 0;
1106 bool result = Send(new SyncChannelTestMsg_AnswerToLife(&answer));
1107 DCHECK(result);
1108 DCHECK_EQ(answer, 42);
1109
1110 worker_->Done();
1111 }
1112
1113 Worker* worker_;
1114 base::Thread thread_;
1115};
1116
1117class SyncMessageFilterServer : public Worker {
1118 public:
1119 SyncMessageFilterServer()
1120 : Worker(Channel::MODE_SERVER, "sync_message_filter_server") {
1121 filter_ = new TestSyncMessageFilter(shutdown_event(), this);
1122 }
1123
1124 void Run() {
1125 channel()->AddFilter(filter_.get());
1126 }
1127
1128 scoped_refptr<TestSyncMessageFilter> filter_;
1129};
1130
ananta@chromium.org999f2972010-09-03 06:45:50 +09001131// This class provides functionality to test the case that a Send on the sync
1132// channel does not crash after the channel has been closed.
1133class ServerSendAfterClose : public Worker {
1134 public:
1135 ServerSendAfterClose()
1136 : Worker(Channel::MODE_SERVER, "simpler_server"),
1137 send_result_(true) {
1138 }
1139
1140 bool SendDummy() {
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001141 ListenerThread()->message_loop()->PostTask(
ajwong@chromium.orgc1e632d2012-01-06 11:37:17 +09001142 FROM_HERE, base::Bind(base::IgnoreResult(&ServerSendAfterClose::Send),
1143 this, new SyncChannelTestMsg_NoArgs));
ananta@chromium.org999f2972010-09-03 06:45:50 +09001144 return true;
1145 }
1146
1147 bool send_result() const {
1148 return send_result_;
1149 }
1150
1151 private:
1152 virtual void Run() {
1153 CloseChannel();
1154 Done();
1155 }
1156
1157 bool Send(Message* msg) {
1158 send_result_ = Worker::Send(msg);
1159 Done();
1160 return send_result_;
1161 }
1162
1163 bool send_result_;
1164};
1165
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001166} // namespace
1167
1168// Tests basic synchronous call
1169TEST_F(IPCSyncChannelTest, SyncMessageFilter) {
1170 std::vector<Worker*> workers;
1171 workers.push_back(new SyncMessageFilterServer());
1172 workers.push_back(new SimpleClient());
1173 RunTest(workers);
1174}
ananta@chromium.org999f2972010-09-03 06:45:50 +09001175
1176// Test the case when the channel is closed and a Send is attempted after that.
1177TEST_F(IPCSyncChannelTest, SendAfterClose) {
1178 ServerSendAfterClose server;
1179 server.Start();
1180
1181 server.done_event()->Wait();
1182 server.done_event()->Reset();
1183
1184 server.SendDummy();
1185 server.done_event()->Wait();
1186
1187 EXPECT_FALSE(server.send_result());
1188}
1189
piman@google.com0cbefaa2011-04-08 12:38:21 +09001190//-----------------------------------------------------------------------------
ananta@chromium.org999f2972010-09-03 06:45:50 +09001191
piman@google.com0cbefaa2011-04-08 12:38:21 +09001192namespace {
1193
1194class RestrictedDispatchServer : public Worker {
1195 public:
piman@chromium.org5a00b882012-03-31 06:29:30 +09001196 RestrictedDispatchServer(WaitableEvent* sent_ping_event,
1197 WaitableEvent* wait_event)
piman@google.com0cbefaa2011-04-08 12:38:21 +09001198 : Worker("restricted_channel", Channel::MODE_SERVER),
piman@chromium.org5a00b882012-03-31 06:29:30 +09001199 sent_ping_event_(sent_ping_event),
1200 wait_event_(wait_event) { }
piman@google.com0cbefaa2011-04-08 12:38:21 +09001201
1202 void OnDoPing(int ping) {
1203 // Send an asynchronous message that unblocks the caller.
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001204 Message* msg = new SyncChannelTestMsg_Ping(ping);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001205 msg->set_unblock(true);
1206 Send(msg);
1207 // Signal the event after the message has been sent on the channel, on the
1208 // IPC thread.
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001209 ipc_thread().message_loop()->PostTask(
1210 FROM_HERE, base::Bind(&RestrictedDispatchServer::OnPingSent, this));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001211 }
1212
piman@chromium.org5a00b882012-03-31 06:29:30 +09001213 void OnPingTTL(int ping, int* out) {
1214 *out = ping;
1215 wait_event_->Wait();
1216 }
1217
piman@google.com0cbefaa2011-04-08 12:38:21 +09001218 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1219
1220 private:
1221 bool OnMessageReceived(const Message& message) {
1222 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchServer, message)
1223 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
piman@chromium.org5a00b882012-03-31 06:29:30 +09001224 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_PingTTL, OnPingTTL)
piman@google.com0cbefaa2011-04-08 12:38:21 +09001225 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, Done)
1226 IPC_END_MESSAGE_MAP()
1227 return true;
1228 }
1229
1230 void OnPingSent() {
1231 sent_ping_event_->Signal();
1232 }
1233
1234 void OnNoArgs() { }
1235 WaitableEvent* sent_ping_event_;
piman@chromium.org5a00b882012-03-31 06:29:30 +09001236 WaitableEvent* wait_event_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001237};
1238
1239class NonRestrictedDispatchServer : public Worker {
1240 public:
piman@chromium.org5a00b882012-03-31 06:29:30 +09001241 NonRestrictedDispatchServer(WaitableEvent* signal_event)
1242 : Worker("non_restricted_channel", Channel::MODE_SERVER),
1243 signal_event_(signal_event) {}
1244
1245 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1246
1247 void OnDoPingTTL(int ping) {
1248 int value = 0;
1249 Send(new SyncChannelTestMsg_PingTTL(ping, &value));
1250 signal_event_->Signal();
1251 }
piman@google.com0cbefaa2011-04-08 12:38:21 +09001252
1253 private:
1254 bool OnMessageReceived(const Message& message) {
1255 IPC_BEGIN_MESSAGE_MAP(NonRestrictedDispatchServer, message)
1256 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1257 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, Done)
1258 IPC_END_MESSAGE_MAP()
1259 return true;
1260 }
1261
1262 void OnNoArgs() { }
piman@chromium.org5a00b882012-03-31 06:29:30 +09001263 WaitableEvent* signal_event_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001264};
1265
1266class RestrictedDispatchClient : public Worker {
1267 public:
1268 RestrictedDispatchClient(WaitableEvent* sent_ping_event,
1269 RestrictedDispatchServer* server,
piman@chromium.org5a00b882012-03-31 06:29:30 +09001270 NonRestrictedDispatchServer* server2,
piman@google.com0cbefaa2011-04-08 12:38:21 +09001271 int* success)
1272 : Worker("restricted_channel", Channel::MODE_CLIENT),
1273 ping_(0),
1274 server_(server),
piman@chromium.org5a00b882012-03-31 06:29:30 +09001275 server2_(server2),
piman@google.com0cbefaa2011-04-08 12:38:21 +09001276 success_(success),
1277 sent_ping_event_(sent_ping_event) {}
1278
1279 void Run() {
1280 // Incoming messages from our channel should only be dispatched when we
1281 // send a message on that same channel.
piman@chromium.org5a00b882012-03-31 06:29:30 +09001282 channel()->SetRestrictDispatchChannelGroup(1);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001283
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001284 server_->ListenerThread()->message_loop()->PostTask(
1285 FROM_HERE, base::Bind(&RestrictedDispatchServer::OnDoPing, server_, 1));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001286 sent_ping_event_->Wait();
1287 Send(new SyncChannelTestMsg_NoArgs);
1288 if (ping_ == 1)
1289 ++*success_;
1290 else
1291 LOG(ERROR) << "Send failed to dispatch incoming message on same channel";
1292
piman@chromium.org5a00b882012-03-31 06:29:30 +09001293 non_restricted_channel_.reset(new SyncChannel(
piman@google.com0cbefaa2011-04-08 12:38:21 +09001294 "non_restricted_channel", Channel::MODE_CLIENT, this,
jam@chromium.org06d18442011-05-03 03:00:49 +09001295 ipc_thread().message_loop_proxy(), true, shutdown_event()));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001296
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001297 server_->ListenerThread()->message_loop()->PostTask(
1298 FROM_HERE, base::Bind(&RestrictedDispatchServer::OnDoPing, server_, 2));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001299 sent_ping_event_->Wait();
1300 // Check that the incoming message is *not* dispatched when sending on the
1301 // non restricted channel.
1302 // TODO(piman): there is a possibility of a false positive race condition
1303 // here, if the message that was posted on the server-side end of the pipe
1304 // is not visible yet on the client side, but I don't know how to solve this
1305 // without hooking into the internals of SyncChannel. I haven't seen it in
1306 // practice (i.e. not setting SetRestrictDispatchToSameChannel does cause
1307 // the following to fail).
piman@chromium.org5a00b882012-03-31 06:29:30 +09001308 non_restricted_channel_->Send(new SyncChannelTestMsg_NoArgs);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001309 if (ping_ == 1)
1310 ++*success_;
1311 else
1312 LOG(ERROR) << "Send dispatched message from restricted channel";
1313
1314 Send(new SyncChannelTestMsg_NoArgs);
1315 if (ping_ == 2)
1316 ++*success_;
1317 else
1318 LOG(ERROR) << "Send failed to dispatch incoming message on same channel";
1319
piman@chromium.org5a00b882012-03-31 06:29:30 +09001320 // Check that the incoming message on the non-restricted channel is
1321 // dispatched when sending on the restricted channel.
1322 server2_->ListenerThread()->message_loop()->PostTask(
1323 FROM_HERE,
1324 base::Bind(&NonRestrictedDispatchServer::OnDoPingTTL, server2_, 3));
1325 int value = 0;
1326 Send(new SyncChannelTestMsg_PingTTL(4, &value));
1327 if (ping_ == 3 && value == 4)
1328 ++*success_;
1329 else
1330 LOG(ERROR) << "Send failed to dispatch message from unrestricted channel";
1331
1332 non_restricted_channel_->Send(new SyncChannelTestMsg_Done);
1333 non_restricted_channel_.reset();
piman@google.com0cbefaa2011-04-08 12:38:21 +09001334 Send(new SyncChannelTestMsg_Done);
1335 Done();
1336 }
1337
1338 private:
1339 bool OnMessageReceived(const Message& message) {
1340 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchClient, message)
1341 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Ping, OnPing)
piman@chromium.org5a00b882012-03-31 06:29:30 +09001342 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelTestMsg_PingTTL, OnPingTTL)
piman@google.com0cbefaa2011-04-08 12:38:21 +09001343 IPC_END_MESSAGE_MAP()
1344 return true;
1345 }
1346
1347 void OnPing(int ping) {
1348 ping_ = ping;
1349 }
1350
piman@chromium.org5a00b882012-03-31 06:29:30 +09001351 void OnPingTTL(int ping, IPC::Message* reply) {
1352 ping_ = ping;
1353 // This message comes from the NonRestrictedDispatchServer, we have to send
1354 // the reply back manually.
1355 SyncChannelTestMsg_PingTTL::WriteReplyParams(reply, ping);
1356 non_restricted_channel_->Send(reply);
1357 }
1358
piman@google.com0cbefaa2011-04-08 12:38:21 +09001359 int ping_;
1360 RestrictedDispatchServer* server_;
piman@chromium.org5a00b882012-03-31 06:29:30 +09001361 NonRestrictedDispatchServer* server2_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001362 int* success_;
1363 WaitableEvent* sent_ping_event_;
piman@chromium.org5a00b882012-03-31 06:29:30 +09001364 scoped_ptr<SyncChannel> non_restricted_channel_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001365};
1366
1367} // namespace
1368
1369TEST_F(IPCSyncChannelTest, RestrictedDispatch) {
1370 WaitableEvent sent_ping_event(false, false);
piman@chromium.org5a00b882012-03-31 06:29:30 +09001371 WaitableEvent wait_event(false, false);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001372 RestrictedDispatchServer* server =
piman@chromium.org5a00b882012-03-31 06:29:30 +09001373 new RestrictedDispatchServer(&sent_ping_event, &wait_event);
1374 NonRestrictedDispatchServer* server2 =
1375 new NonRestrictedDispatchServer(&wait_event);
1376
piman@google.com0cbefaa2011-04-08 12:38:21 +09001377 int success = 0;
1378 std::vector<Worker*> workers;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001379 workers.push_back(server);
piman@chromium.org5a00b882012-03-31 06:29:30 +09001380 workers.push_back(server2);
1381 workers.push_back(new RestrictedDispatchClient(
1382 &sent_ping_event, server, server2, &success));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001383 RunTest(workers);
piman@chromium.org5a00b882012-03-31 06:29:30 +09001384 EXPECT_EQ(4, success);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001385}
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001386
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001387//-----------------------------------------------------------------------------
1388
1389// This test case inspired by crbug.com/108491
1390// We create two servers that use the same ListenerThread but have
1391// SetRestrictDispatchToSameChannel set to true.
1392// We create clients, then use some specific WaitableEvent wait/signalling to
1393// ensure that messages get dispatched in a way that causes a deadlock due to
1394// a nested dispatch and an eligible message in a higher-level dispatch's
1395// delayed_queue. Specifically, we start with client1 about so send an
1396// unblocking message to server1, while the shared listener thread for the
1397// servers server1 and server2 is about to send a non-unblocking message to
1398// client1. At the same time, client2 will be about to send an unblocking
1399// message to server2. Server1 will handle the client1->server1 message by
1400// telling server2 to send a non-unblocking message to client2.
1401// What should happen is that the send to server2 should find the pending,
1402// same-context client2->server2 message to dispatch, causing client2 to
1403// unblock then handle the server2->client2 message, so that the shared
1404// servers' listener thread can then respond to the client1->server1 message.
1405// Then client1 can handle the non-unblocking server1->client1 message.
1406// The old code would end up in a state where the server2->client2 message is
1407// sent, but the client2->server2 message (which is eligible for dispatch, and
1408// which is what client2 is waiting for) is stashed in a local delayed_queue
1409// that has server1's channel context, causing a deadlock.
1410// WaitableEvents in the events array are used to:
1411// event 0: indicate to client1 that server listener is in OnDoServerTask
1412// event 1: indicate to client1 that client2 listener is in OnDoClient2Task
1413// event 2: indicate to server1 that client2 listener is in OnDoClient2Task
1414// event 3: indicate to client2 that server listener is in OnDoServerTask
1415
1416namespace {
1417
1418class RestrictedDispatchDeadlockServer : public Worker {
1419 public:
1420 RestrictedDispatchDeadlockServer(int server_num,
1421 WaitableEvent* server_ready_event,
1422 WaitableEvent** events,
1423 RestrictedDispatchDeadlockServer* peer)
1424 : Worker(server_num == 1 ? "channel1" : "channel2", Channel::MODE_SERVER),
1425 server_num_(server_num),
1426 server_ready_event_(server_ready_event),
1427 events_(events),
1428 peer_(peer),
1429 client_kicked_(false) { }
1430
1431 void OnDoServerTask() {
1432 events_[3]->Signal();
1433 events_[2]->Wait();
1434 events_[0]->Signal();
1435 SendMessageToClient();
1436 }
1437
1438 void Run() {
piman@chromium.org5a00b882012-03-31 06:29:30 +09001439 channel()->SetRestrictDispatchChannelGroup(1);
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001440 server_ready_event_->Signal();
1441 }
1442
1443 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1444
1445 private:
1446 bool OnMessageReceived(const Message& message) {
1447 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchDeadlockServer, message)
1448 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1449 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, Done)
1450 IPC_END_MESSAGE_MAP()
1451 return true;
1452 }
1453
1454 void OnNoArgs() {
1455 if (server_num_ == 1) {
1456 DCHECK(peer_ != NULL);
1457 peer_->SendMessageToClient();
1458 }
1459 }
1460
1461 void SendMessageToClient() {
1462 Message* msg = new SyncChannelTestMsg_NoArgs;
1463 msg->set_unblock(false);
1464 DCHECK(!msg->should_unblock());
1465 Send(msg);
1466 }
1467
1468 int server_num_;
1469 WaitableEvent* server_ready_event_;
1470 WaitableEvent** events_;
1471 RestrictedDispatchDeadlockServer* peer_;
1472 bool client_kicked_;
1473};
1474
1475class RestrictedDispatchDeadlockClient2 : public Worker {
1476 public:
1477 RestrictedDispatchDeadlockClient2(RestrictedDispatchDeadlockServer* server,
1478 WaitableEvent* server_ready_event,
1479 WaitableEvent** events)
1480 : Worker("channel2", Channel::MODE_CLIENT),
1481 server_(server),
1482 server_ready_event_(server_ready_event),
1483 events_(events),
1484 received_msg_(false),
1485 received_noarg_reply_(false),
1486 done_issued_(false) {}
1487
1488 void Run() {
1489 server_ready_event_->Wait();
1490 }
1491
1492 void OnDoClient2Task() {
1493 events_[3]->Wait();
1494 events_[1]->Signal();
1495 events_[2]->Signal();
1496 DCHECK(received_msg_ == false);
1497
1498 Message* message = new SyncChannelTestMsg_NoArgs;
1499 message->set_unblock(true);
1500 Send(message);
1501 received_noarg_reply_ = true;
1502 }
1503
1504 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1505 private:
1506 bool OnMessageReceived(const Message& message) {
1507 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchDeadlockClient2, message)
1508 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1509 IPC_END_MESSAGE_MAP()
1510 return true;
1511 }
1512
1513 void OnNoArgs() {
1514 received_msg_ = true;
1515 PossiblyDone();
1516 }
1517
1518 void PossiblyDone() {
1519 if (received_noarg_reply_ && received_msg_) {
1520 DCHECK(done_issued_ == false);
1521 done_issued_ = true;
1522 Send(new SyncChannelTestMsg_Done);
1523 Done();
1524 }
1525 }
1526
1527 RestrictedDispatchDeadlockServer* server_;
1528 WaitableEvent* server_ready_event_;
1529 WaitableEvent** events_;
1530 bool received_msg_;
1531 bool received_noarg_reply_;
1532 bool done_issued_;
1533};
1534
1535class RestrictedDispatchDeadlockClient1 : public Worker {
1536 public:
1537 RestrictedDispatchDeadlockClient1(RestrictedDispatchDeadlockServer* server,
1538 RestrictedDispatchDeadlockClient2* peer,
1539 WaitableEvent* server_ready_event,
1540 WaitableEvent** events)
1541 : Worker("channel1", Channel::MODE_CLIENT),
1542 server_(server),
1543 peer_(peer),
1544 server_ready_event_(server_ready_event),
1545 events_(events),
1546 received_msg_(false),
1547 received_noarg_reply_(false),
1548 done_issued_(false) {}
1549
1550 void Run() {
1551 server_ready_event_->Wait();
1552 server_->ListenerThread()->message_loop()->PostTask(
1553 FROM_HERE,
1554 base::Bind(&RestrictedDispatchDeadlockServer::OnDoServerTask, server_));
1555 peer_->ListenerThread()->message_loop()->PostTask(
1556 FROM_HERE,
1557 base::Bind(&RestrictedDispatchDeadlockClient2::OnDoClient2Task, peer_));
1558 events_[0]->Wait();
1559 events_[1]->Wait();
1560 DCHECK(received_msg_ == false);
1561
1562 Message* message = new SyncChannelTestMsg_NoArgs;
1563 message->set_unblock(true);
1564 Send(message);
1565 received_noarg_reply_ = true;
1566 PossiblyDone();
1567 }
1568
1569 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1570 private:
1571 bool OnMessageReceived(const Message& message) {
1572 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchDeadlockClient1, message)
1573 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1574 IPC_END_MESSAGE_MAP()
1575 return true;
1576 }
1577
1578 void OnNoArgs() {
1579 received_msg_ = true;
1580 PossiblyDone();
1581 }
1582
1583 void PossiblyDone() {
1584 if (received_noarg_reply_ && received_msg_) {
1585 DCHECK(done_issued_ == false);
1586 done_issued_ = true;
1587 Send(new SyncChannelTestMsg_Done);
1588 Done();
1589 }
1590 }
1591
1592 RestrictedDispatchDeadlockServer* server_;
1593 RestrictedDispatchDeadlockClient2* peer_;
1594 WaitableEvent* server_ready_event_;
1595 WaitableEvent** events_;
1596 bool received_msg_;
1597 bool received_noarg_reply_;
1598 bool done_issued_;
1599};
1600
1601} // namespace
1602
1603TEST_F(IPCSyncChannelTest, RestrictedDispatchDeadlock) {
1604 std::vector<Worker*> workers;
1605
1606 // A shared worker thread so that server1 and server2 run on one thread.
1607 base::Thread worker_thread("RestrictedDispatchDeadlock");
1608 ASSERT_TRUE(worker_thread.Start());
1609
1610 WaitableEvent server1_ready(false, false);
1611 WaitableEvent server2_ready(false, false);
1612
1613 WaitableEvent event0(false, false);
1614 WaitableEvent event1(false, false);
1615 WaitableEvent event2(false, false);
1616 WaitableEvent event3(false, false);
1617 WaitableEvent* events[4] = {&event0, &event1, &event2, &event3};
1618
1619 RestrictedDispatchDeadlockServer* server1;
1620 RestrictedDispatchDeadlockServer* server2;
1621 RestrictedDispatchDeadlockClient1* client1;
1622 RestrictedDispatchDeadlockClient2* client2;
1623
1624 server2 = new RestrictedDispatchDeadlockServer(2, &server2_ready, events,
1625 NULL);
1626 server2->OverrideThread(&worker_thread);
1627 workers.push_back(server2);
1628
1629 client2 = new RestrictedDispatchDeadlockClient2(server2, &server2_ready,
1630 events);
1631 workers.push_back(client2);
1632
1633 server1 = new RestrictedDispatchDeadlockServer(1, &server1_ready, events,
1634 server2);
1635 server1->OverrideThread(&worker_thread);
1636 workers.push_back(server1);
1637
1638 client1 = new RestrictedDispatchDeadlockClient1(server1, client2,
1639 &server1_ready, events);
1640 workers.push_back(client1);
1641
1642 RunTest(workers);
1643}
1644
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001645//-----------------------------------------------------------------------------
1646
piman@chromium.org5a00b882012-03-31 06:29:30 +09001647// This test case inspired by crbug.com/120530
1648// We create 4 workers that pipe to each other W1->W2->W3->W4->W1 then we send a
1649// message that recurses through 3, 4 or 5 steps to make sure, say, W1 can
1650// re-enter when called from W4 while it's sending a message to W2.
1651// The first worker drives the whole test so it must be treated specially.
1652namespace {
1653
1654class RestrictedDispatchPipeWorker : public Worker {
1655 public:
1656 RestrictedDispatchPipeWorker(
1657 const std::string &channel1,
1658 WaitableEvent* event1,
1659 const std::string &channel2,
1660 WaitableEvent* event2,
1661 int group,
1662 int* success)
1663 : Worker(channel1, Channel::MODE_SERVER),
1664 event1_(event1),
1665 event2_(event2),
1666 other_channel_name_(channel2),
1667 group_(group),
1668 success_(success) {
1669 }
1670
1671 void OnPingTTL(int ping, int* ret) {
1672 *ret = 0;
1673 if (!ping)
1674 return;
1675 other_channel_->Send(new SyncChannelTestMsg_PingTTL(ping - 1, ret));
1676 ++*ret;
1677 }
1678
1679 void OnDone() {
1680 if (is_first())
1681 return;
1682 other_channel_->Send(new SyncChannelTestMsg_Done);
1683 other_channel_.reset();
1684 Done();
1685 }
1686
1687 void Run() {
1688 channel()->SetRestrictDispatchChannelGroup(group_);
1689 if (is_first())
1690 event1_->Signal();
1691 event2_->Wait();
1692 other_channel_.reset(new SyncChannel(
1693 other_channel_name_, Channel::MODE_CLIENT, this,
1694 ipc_thread().message_loop_proxy(), true, shutdown_event()));
1695 other_channel_->SetRestrictDispatchChannelGroup(group_);
1696 if (!is_first()) {
1697 event1_->Signal();
1698 return;
1699 }
1700 *success_ = 0;
1701 int value = 0;
1702 OnPingTTL(3, &value);
1703 *success_ += (value == 3);
1704 OnPingTTL(4, &value);
1705 *success_ += (value == 4);
1706 OnPingTTL(5, &value);
1707 *success_ += (value == 5);
1708 other_channel_->Send(new SyncChannelTestMsg_Done);
1709 other_channel_.reset();
1710 Done();
1711 }
1712
1713 bool is_first() { return !!success_; }
1714
1715 private:
1716 bool OnMessageReceived(const Message& message) {
1717 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchPipeWorker, message)
1718 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_PingTTL, OnPingTTL)
1719 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, OnDone)
1720 IPC_END_MESSAGE_MAP()
1721 return true;
1722 }
1723
1724 scoped_ptr<SyncChannel> other_channel_;
1725 WaitableEvent* event1_;
1726 WaitableEvent* event2_;
1727 std::string other_channel_name_;
1728 int group_;
1729 int* success_;
1730};
1731
1732} // namespace
1733
1734TEST_F(IPCSyncChannelTest, RestrictedDispatch4WayDeadlock) {
1735 int success = 0;
1736 std::vector<Worker*> workers;
1737 WaitableEvent event0(true, false);
1738 WaitableEvent event1(true, false);
1739 WaitableEvent event2(true, false);
1740 WaitableEvent event3(true, false);
1741 workers.push_back(new RestrictedDispatchPipeWorker(
1742 "channel0", &event0, "channel1", &event1, 1, &success));
1743 workers.push_back(new RestrictedDispatchPipeWorker(
1744 "channel1", &event1, "channel2", &event2, 2, NULL));
1745 workers.push_back(new RestrictedDispatchPipeWorker(
1746 "channel2", &event2, "channel3", &event3, 3, NULL));
1747 workers.push_back(new RestrictedDispatchPipeWorker(
1748 "channel3", &event3, "channel0", &event0, 4, NULL));
1749 RunTest(workers);
1750 EXPECT_EQ(3, success);
1751}
1752
piman@chromium.orgde4e75e2012-04-11 05:07:53 +09001753
1754//-----------------------------------------------------------------------------
1755//
1756// This test case inspired by crbug.com/122443
1757// We want to make sure a reply message with the unblock flag set correctly
1758// behaves as a reply, not a regular message.
1759// We have 3 workers. Server1 will send a message to Server2 (which will block),
1760// during which it will dispatch a message comming from Client, at which point
1761// it will send another message to Server2. While sending that second message it
1762// will receive a reply from Server1 with the unblock flag.
1763
1764namespace {
1765
1766class ReentrantReplyServer1 : public Worker {
1767 public:
1768 ReentrantReplyServer1(WaitableEvent* server_ready)
1769 : Worker("reentrant_reply1", Channel::MODE_SERVER),
1770 server_ready_(server_ready) { }
1771
1772 void Run() {
1773 server2_channel_.reset(new SyncChannel(
1774 "reentrant_reply2", Channel::MODE_CLIENT, this,
1775 ipc_thread().message_loop_proxy(), true, shutdown_event()));
1776 server_ready_->Signal();
1777 Message* msg = new SyncChannelTestMsg_Reentrant1();
1778 server2_channel_->Send(msg);
1779 server2_channel_.reset();
1780 Done();
1781 }
1782
1783 private:
1784 bool OnMessageReceived(const Message& message) {
1785 IPC_BEGIN_MESSAGE_MAP(ReentrantReplyServer1, message)
1786 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Reentrant2, OnReentrant2)
1787 IPC_REPLY_HANDLER(OnReply)
1788 IPC_END_MESSAGE_MAP()
1789 return true;
1790 }
1791
1792 void OnReentrant2() {
1793 Message* msg = new SyncChannelTestMsg_Reentrant3();
1794 server2_channel_->Send(msg);
1795 }
1796
1797 void OnReply(const Message& message) {
1798 // If we get here, the Send() will never receive the reply (thus would
1799 // hang), so abort instead.
1800 LOG(FATAL) << "Reply message was dispatched";
1801 }
1802
1803 WaitableEvent* server_ready_;
1804 scoped_ptr<SyncChannel> server2_channel_;
1805};
1806
1807class ReentrantReplyServer2 : public Worker {
1808 public:
1809 ReentrantReplyServer2()
1810 : Worker("reentrant_reply2", Channel::MODE_SERVER),
1811 reply_(NULL) { }
1812
1813 private:
1814 bool OnMessageReceived(const Message& message) {
1815 IPC_BEGIN_MESSAGE_MAP(ReentrantReplyServer2, message)
1816 IPC_MESSAGE_HANDLER_DELAY_REPLY(
1817 SyncChannelTestMsg_Reentrant1, OnReentrant1)
1818 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Reentrant3, OnReentrant3)
1819 IPC_END_MESSAGE_MAP()
1820 return true;
1821 }
1822
1823 void OnReentrant1(Message* reply) {
1824 DCHECK(!reply_);
1825 reply_ = reply;
1826 }
1827
1828 void OnReentrant3() {
1829 DCHECK(reply_);
1830 Message* reply = reply_;
1831 reply_ = NULL;
1832 reply->set_unblock(true);
1833 Send(reply);
1834 Done();
1835 }
1836
1837 Message* reply_;
1838};
1839
1840class ReentrantReplyClient : public Worker {
1841 public:
1842 ReentrantReplyClient(WaitableEvent* server_ready)
1843 : Worker("reentrant_reply1", Channel::MODE_CLIENT),
1844 server_ready_(server_ready) { }
1845
1846 void Run() {
1847 server_ready_->Wait();
1848 Send(new SyncChannelTestMsg_Reentrant2());
1849 Done();
1850 }
1851
1852 private:
1853 WaitableEvent* server_ready_;
1854};
1855
1856} // namespace
1857
1858TEST_F(IPCSyncChannelTest, ReentrantReply) {
1859 std::vector<Worker*> workers;
1860 WaitableEvent server_ready(false, false);
1861 workers.push_back(new ReentrantReplyServer2());
1862 workers.push_back(new ReentrantReplyServer1(&server_ready));
1863 workers.push_back(new ReentrantReplyClient(&server_ready));
1864 RunTest(workers);
1865}
1866
piman@chromium.org5a00b882012-03-31 06:29:30 +09001867//-----------------------------------------------------------------------------
1868
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001869// Generate a validated channel ID using Channel::GenerateVerifiedChannelID().
1870namespace {
1871
1872class VerifiedServer : public Worker {
1873 public:
1874 VerifiedServer(base::Thread* listener_thread,
1875 const std::string& channel_name,
1876 const std::string& reply_text)
1877 : Worker(channel_name, Channel::MODE_SERVER),
1878 reply_text_(reply_text) {
1879 Worker::OverrideThread(listener_thread);
1880 }
1881
1882 virtual void OnNestedTestMsg(Message* reply_msg) {
1883 VLOG(1) << __FUNCTION__ << " Sending reply: " << reply_text_;
1884 SyncChannelNestedTestMsg_String::WriteReplyParams(reply_msg, reply_text_);
1885 Send(reply_msg);
jschuh@chromium.orga5cd0762012-04-05 11:38:34 +09001886 ASSERT_EQ(channel()->peer_pid(), base::GetCurrentProcId());
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001887 Done();
1888 }
1889
1890 private:
1891 std::string reply_text_;
1892};
1893
1894class VerifiedClient : public Worker {
1895 public:
1896 VerifiedClient(base::Thread* listener_thread,
1897 const std::string& channel_name,
1898 const std::string& expected_text)
1899 : Worker(channel_name, Channel::MODE_CLIENT),
1900 expected_text_(expected_text) {
1901 Worker::OverrideThread(listener_thread);
1902 }
1903
1904 virtual void Run() {
1905 std::string response;
1906 SyncMessage* msg = new SyncChannelNestedTestMsg_String(&response);
1907 bool result = Send(msg);
1908 DCHECK(result);
1909 DCHECK_EQ(response, expected_text_);
1910
1911 VLOG(1) << __FUNCTION__ << " Received reply: " << response;
jschuh@chromium.orga5cd0762012-04-05 11:38:34 +09001912 ASSERT_EQ(channel()->peer_pid(), base::GetCurrentProcId());
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001913 Done();
1914 }
1915
1916 private:
1917 bool pump_during_send_;
1918 std::string expected_text_;
1919};
1920
1921void Verified() {
1922 std::vector<Worker*> workers;
1923
1924 // A shared worker thread for servers
1925 base::Thread server_worker_thread("Verified_ServerListener");
1926 ASSERT_TRUE(server_worker_thread.Start());
1927
1928 base::Thread client_worker_thread("Verified_ClientListener");
1929 ASSERT_TRUE(client_worker_thread.Start());
1930
1931 std::string channel_id = Channel::GenerateVerifiedChannelID("Verified");
1932 Worker* worker;
1933
1934 worker = new VerifiedServer(&server_worker_thread,
1935 channel_id,
1936 "Got first message");
1937 workers.push_back(worker);
1938
1939 worker = new VerifiedClient(&client_worker_thread,
1940 channel_id,
1941 "Got first message");
1942 workers.push_back(worker);
1943
1944 RunTest(workers);
1945
1946#if defined(OS_WIN)
1947#endif
1948}
1949
1950} // namespace
1951
1952// Windows needs to send an out-of-band secret to verify the client end of the
1953// channel. Test that we still connect correctly in that case.
1954TEST_F(IPCSyncChannelTest, Verified) {
1955 Verified();
1956}
1957
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001958} // namespace IPC