blob: 81f5d11917c08d20ef415ab7014ba58329be8f29 [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"
dilmah@chromium.orgdc4b9702011-07-20 07:13:24 +090017#include "base/stl_util.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090018#include "base/string_util.h"
timurrrr@chromium.orgf39c3ff2010-05-14 17:24:42 +090019#include "base/third_party/dynamic_annotations/dynamic_annotations.h"
brettw@google.com7c5cc672011-01-01 11:17:08 +090020#include "base/threading/platform_thread.h"
brettw@chromium.org5b5f5e02011-01-01 10:01:06 +090021#include "base/threading/thread.h"
brettw@chromium.org5238c7d2011-01-02 15:05:39 +090022#include "base/synchronization/waitable_event.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090023#include "ipc/ipc_message.h"
jabdelmalek@google.comeb921652010-04-07 05:33:36 +090024#include "ipc/ipc_sync_message_filter.h"
jam@chromium.org86a8de12010-12-09 08:34:16 +090025#include "ipc/ipc_sync_message_unittest.h"
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090026#include "testing/gtest/include/gtest/gtest.h"
27
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090028using base::WaitableEvent;
29
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +090030namespace IPC {
31
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090032namespace {
33
34// Base class for a "process" with listener and IPC threads.
35class Worker : public Channel::Listener, public Message::Sender {
36 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),
timurrrr@chromium.org03100a82009-10-27 20:28:58 +090045 shutdown_event_(true, false) {
46 // The data race on vfptr is real but is very hard
47 // to suppress using standard Valgrind mechanism (suppressions).
48 // We have to use ANNOTATE_BENIGN_RACE to hide the reports and
49 // make ThreadSanitizer bots green.
50 ANNOTATE_BENIGN_RACE(this, "Race on vfptr, http://crbug.com/25841");
51 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090052
53 // Will create a named channel and use this name for the threads' name.
54 Worker(const std::string& channel_name, Channel::Mode mode)
55 : done_(new WaitableEvent(false, false)),
56 channel_created_(new WaitableEvent(false, false)),
57 channel_name_(channel_name),
58 mode_(mode),
59 ipc_thread_((channel_name + "_ipc").c_str()),
60 listener_thread_((channel_name + "_listener").c_str()),
61 overrided_thread_(NULL),
timurrrr@chromium.org03100a82009-10-27 20:28:58 +090062 shutdown_event_(true, false) {
63 // The data race on vfptr is real but is very hard
64 // to suppress using standard Valgrind mechanism (suppressions).
65 // We have to use ANNOTATE_BENIGN_RACE to hide the reports and
66 // make ThreadSanitizer bots green.
67 ANNOTATE_BENIGN_RACE(this, "Race on vfptr, http://crbug.com/25841");
68 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090069
70 // The IPC thread needs to outlive SyncChannel, so force the correct order of
71 // destruction.
72 virtual ~Worker() {
73 WaitableEvent listener_done(false, false), ipc_done(false, false);
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +090074 ListenerThread()->message_loop()->PostTask(
75 FROM_HERE, base::Bind(&Worker::OnListenerThreadShutdown1, this,
76 &listener_done, &ipc_done));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090077 listener_done.Wait();
78 ipc_done.Wait();
79 ipc_thread_.Stop();
80 listener_thread_.Stop();
81 }
82 void AddRef() { }
83 void Release() { }
darin@chromium.org5cb996e2009-09-30 13:29:20 +090084 static bool ImplementsThreadSafeReferenceCounting() { return true; }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090085 bool Send(Message* msg) { return channel_->Send(msg); }
86 bool SendWithTimeout(Message* msg, int timeout_ms) {
87 return channel_->SendWithTimeout(msg, timeout_ms);
88 }
89 void WaitForChannelCreation() { channel_created_->Wait(); }
90 void CloseChannel() {
91 DCHECK(MessageLoop::current() == ListenerThread()->message_loop());
92 channel_->Close();
93 }
94 void Start() {
95 StartThread(&listener_thread_, MessageLoop::TYPE_DEFAULT);
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +090096 ListenerThread()->message_loop()->PostTask(
97 FROM_HERE, base::Bind(&Worker::OnStart, this));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +090098 }
99 void OverrideThread(base::Thread* overrided_thread) {
100 DCHECK(overrided_thread_ == NULL);
101 overrided_thread_ = overrided_thread;
102 }
103 bool SendAnswerToLife(bool pump, int timeout, bool succeed) {
104 int answer = 0;
105 SyncMessage* msg = new SyncChannelTestMsg_AnswerToLife(&answer);
106 if (pump)
107 msg->EnableMessagePumping();
108 bool result = SendWithTimeout(msg, timeout);
jam@chromium.orgebd07182009-12-01 11:34:18 +0900109 DCHECK_EQ(result, succeed);
110 DCHECK_EQ(answer, (succeed ? 42 : 0));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900111 return result;
112 }
113 bool SendDouble(bool pump, bool succeed) {
114 int answer = 0;
115 SyncMessage* msg = new SyncChannelTestMsg_Double(5, &answer);
116 if (pump)
117 msg->EnableMessagePumping();
118 bool result = Send(msg);
jam@chromium.orgebd07182009-12-01 11:34:18 +0900119 DCHECK_EQ(result, succeed);
120 DCHECK_EQ(answer, (succeed ? 10 : 0));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900121 return result;
122 }
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900123 const std::string& channel_name() { return channel_name_; }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900124 Channel::Mode mode() { return mode_; }
125 WaitableEvent* done_event() { return done_.get(); }
jabdelmalek@google.comeb921652010-04-07 05:33:36 +0900126 WaitableEvent* shutdown_event() { return &shutdown_event_; }
jam@chromium.orgebd07182009-12-01 11:34:18 +0900127 void ResetChannel() { channel_.reset(); }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900128 // Derived classes need to call this when they've completed their part of
129 // the test.
130 void Done() { done_->Signal(); }
jabdelmalek@google.comeb921652010-04-07 05:33:36 +0900131
132 protected:
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +0900133 SyncChannel* channel() { return channel_.get(); }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900134 // Functions for dervied classes to implement if they wish.
135 virtual void Run() { }
136 virtual void OnAnswer(int* answer) { NOTREACHED(); }
137 virtual void OnAnswerDelay(Message* reply_msg) {
138 // The message handler map below can only take one entry for
139 // SyncChannelTestMsg_AnswerToLife, so since some classes want
140 // the normal version while other want the delayed reply, we
141 // call the normal version if the derived class didn't override
142 // this function.
143 int answer;
144 OnAnswer(&answer);
145 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, answer);
146 Send(reply_msg);
147 }
148 virtual void OnDouble(int in, int* out) { NOTREACHED(); }
149 virtual void OnDoubleDelay(int in, Message* reply_msg) {
150 int result;
151 OnDouble(in, &result);
152 SyncChannelTestMsg_Double::WriteReplyParams(reply_msg, result);
153 Send(reply_msg);
154 }
155
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900156 virtual void OnNestedTestMsg(Message* reply_msg) {
157 NOTREACHED();
158 }
159
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900160 virtual SyncChannel* CreateChannel() {
161 return new SyncChannel(
162 channel_name_, mode_, this, ipc_thread_.message_loop_proxy(), true,
163 &shutdown_event_);
164 }
165
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900166 base::Thread* ListenerThread() {
167 return overrided_thread_ ? overrided_thread_ : &listener_thread_;
168 }
ananta@chromium.org999f2972010-09-03 06:45:50 +0900169
piman@google.com0cbefaa2011-04-08 12:38:21 +0900170 const base::Thread& ipc_thread() const { return ipc_thread_; }
171
ananta@chromium.org999f2972010-09-03 06:45:50 +0900172 private:
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900173 // Called on the listener thread to create the sync channel.
174 void OnStart() {
175 // Link ipc_thread_, listener_thread_ and channel_ altogether.
176 StartThread(&ipc_thread_, MessageLoop::TYPE_IO);
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900177 channel_.reset(CreateChannel());
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900178 channel_created_->Signal();
179 Run();
180 }
181
182 void OnListenerThreadShutdown1(WaitableEvent* listener_event,
183 WaitableEvent* ipc_event) {
184 // SyncChannel needs to be destructed on the thread that it was created on.
185 channel_.reset();
186
187 MessageLoop::current()->RunAllPending();
188
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +0900189 ipc_thread_.message_loop()->PostTask(
190 FROM_HERE, base::Bind(&Worker::OnIPCThreadShutdown, this,
191 listener_event, ipc_event));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900192 }
193
194 void OnIPCThreadShutdown(WaitableEvent* listener_event,
195 WaitableEvent* ipc_event) {
196 MessageLoop::current()->RunAllPending();
197 ipc_event->Signal();
198
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +0900199 listener_thread_.message_loop()->PostTask(
200 FROM_HERE, base::Bind(&Worker::OnListenerThreadShutdown2, this,
201 listener_event));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900202 }
203
204 void OnListenerThreadShutdown2(WaitableEvent* listener_event) {
205 MessageLoop::current()->RunAllPending();
206 listener_event->Signal();
207 }
208
jam@chromium.org8a2c7842010-12-24 15:19:28 +0900209 bool OnMessageReceived(const Message& message) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900210 IPC_BEGIN_MESSAGE_MAP(Worker, message)
211 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelTestMsg_Double, OnDoubleDelay)
212 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelTestMsg_AnswerToLife,
213 OnAnswerDelay)
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900214 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelNestedTestMsg_String,
215 OnNestedTestMsg)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900216 IPC_END_MESSAGE_MAP()
jam@chromium.org8a2c7842010-12-24 15:19:28 +0900217 return true;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900218 }
219
220 void StartThread(base::Thread* thread, MessageLoop::Type type) {
221 base::Thread::Options options;
222 options.message_loop_type = type;
223 thread->StartWithOptions(options);
224 }
225
226 scoped_ptr<WaitableEvent> done_;
227 scoped_ptr<WaitableEvent> channel_created_;
228 std::string channel_name_;
229 Channel::Mode mode_;
230 scoped_ptr<SyncChannel> channel_;
231 base::Thread ipc_thread_;
232 base::Thread listener_thread_;
233 base::Thread* overrided_thread_;
234
235 base::WaitableEvent shutdown_event_;
236
tfarina@chromium.orgb73eaee2010-06-07 11:10:18 +0900237 DISALLOW_COPY_AND_ASSIGN(Worker);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900238};
239
240
241// Starts the test with the given workers. This function deletes the workers
242// when it's done.
243void RunTest(std::vector<Worker*> workers) {
244 // First we create the workers that are channel servers, or else the other
245 // workers' channel initialization might fail because the pipe isn't created..
246 for (size_t i = 0; i < workers.size(); ++i) {
dmaclach@chromium.orgf146c292011-02-04 05:35:09 +0900247 if (workers[i]->mode() & Channel::MODE_SERVER_FLAG) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900248 workers[i]->Start();
249 workers[i]->WaitForChannelCreation();
250 }
251 }
252
253 // now create the clients
254 for (size_t i = 0; i < workers.size(); ++i) {
dmaclach@chromium.orgf146c292011-02-04 05:35:09 +0900255 if (workers[i]->mode() & Channel::MODE_CLIENT_FLAG)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900256 workers[i]->Start();
257 }
258
259 // wait for all the workers to finish
260 for (size_t i = 0; i < workers.size(); ++i)
261 workers[i]->done_event()->Wait();
262
263 STLDeleteContainerPointers(workers.begin(), workers.end());
264}
265
266} // namespace
267
268class IPCSyncChannelTest : public testing::Test {
269 private:
270 MessageLoop message_loop_;
271};
272
273//-----------------------------------------------------------------------------
274
275namespace {
276
277class SimpleServer : public Worker {
278 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900279 explicit SimpleServer(bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900280 : Worker(Channel::MODE_SERVER, "simpler_server"),
281 pump_during_send_(pump_during_send) { }
282 void Run() {
283 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
284 Done();
285 }
286
287 bool pump_during_send_;
288};
289
290class SimpleClient : public Worker {
291 public:
292 SimpleClient() : Worker(Channel::MODE_CLIENT, "simple_client") { }
293
294 void OnAnswer(int* answer) {
295 *answer = 42;
296 Done();
297 }
298};
299
300void Simple(bool pump_during_send) {
301 std::vector<Worker*> workers;
302 workers.push_back(new SimpleServer(pump_during_send));
303 workers.push_back(new SimpleClient());
304 RunTest(workers);
305}
306
307} // namespace
308
309// Tests basic synchronous call
310TEST_F(IPCSyncChannelTest, Simple) {
311 Simple(false);
312 Simple(true);
313}
314
315//-----------------------------------------------------------------------------
316
317namespace {
318
kkania@chromium.org9ccb4692011-11-16 10:06:46 +0900319// Worker classes which override how the sync channel is created to use the
320// two-step initialization (calling the lightweight constructor and then
321// ChannelProxy::Init separately) process.
322class TwoStepServer : public Worker {
323 public:
324 explicit TwoStepServer(bool create_pipe_now)
325 : Worker(Channel::MODE_SERVER, "simpler_server"),
326 create_pipe_now_(create_pipe_now) { }
327
328 void Run() {
329 SendAnswerToLife(false, base::kNoTimeout, true);
330 Done();
331 }
332
333 virtual SyncChannel* CreateChannel() {
334 SyncChannel* channel = new SyncChannel(
335 this, ipc_thread().message_loop_proxy(), shutdown_event());
336 channel->Init(channel_name(), mode(), create_pipe_now_);
337 return channel;
338 }
339
340 bool create_pipe_now_;
341};
342
343class TwoStepClient : public Worker {
344 public:
345 TwoStepClient(bool create_pipe_now)
346 : Worker(Channel::MODE_CLIENT, "simple_client"),
347 create_pipe_now_(create_pipe_now) { }
348
349 void OnAnswer(int* answer) {
350 *answer = 42;
351 Done();
352 }
353
354 virtual SyncChannel* CreateChannel() {
355 SyncChannel* channel = new SyncChannel(
356 this, ipc_thread().message_loop_proxy(), shutdown_event());
357 channel->Init(channel_name(), mode(), create_pipe_now_);
358 return channel;
359 }
360
361 bool create_pipe_now_;
362};
363
364void TwoStep(bool create_server_pipe_now, bool create_client_pipe_now) {
365 std::vector<Worker*> workers;
366 workers.push_back(new TwoStepServer(create_server_pipe_now));
367 workers.push_back(new TwoStepClient(create_client_pipe_now));
368 RunTest(workers);
369}
370
371} // namespace
372
373// Tests basic two-step initialization, where you call the lightweight
374// constructor then Init.
375TEST_F(IPCSyncChannelTest, TwoStepInitialization) {
376 TwoStep(false, false);
377 TwoStep(false, true);
378 TwoStep(true, false);
379 TwoStep(true, true);
380}
381
382
383//-----------------------------------------------------------------------------
384
385namespace {
386
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900387class DelayClient : public Worker {
388 public:
389 DelayClient() : Worker(Channel::MODE_CLIENT, "delay_client") { }
390
391 void OnAnswerDelay(Message* reply_msg) {
392 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
393 Send(reply_msg);
394 Done();
395 }
396};
397
398void DelayReply(bool pump_during_send) {
399 std::vector<Worker*> workers;
400 workers.push_back(new SimpleServer(pump_during_send));
401 workers.push_back(new DelayClient());
402 RunTest(workers);
403}
404
405} // namespace
406
407// Tests that asynchronous replies work
408TEST_F(IPCSyncChannelTest, DelayReply) {
409 DelayReply(false);
410 DelayReply(true);
411}
412
413//-----------------------------------------------------------------------------
414
415namespace {
416
417class NoHangServer : public Worker {
418 public:
dilmah@chromium.org92b4c8e2011-07-27 02:25:25 +0900419 NoHangServer(WaitableEvent* got_first_reply, bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900420 : Worker(Channel::MODE_SERVER, "no_hang_server"),
421 got_first_reply_(got_first_reply),
422 pump_during_send_(pump_during_send) { }
423 void Run() {
424 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
425 got_first_reply_->Signal();
426
427 SendAnswerToLife(pump_during_send_, base::kNoTimeout, false);
428 Done();
429 }
430
431 WaitableEvent* got_first_reply_;
432 bool pump_during_send_;
433};
434
435class NoHangClient : public Worker {
436 public:
437 explicit NoHangClient(WaitableEvent* got_first_reply)
438 : Worker(Channel::MODE_CLIENT, "no_hang_client"),
439 got_first_reply_(got_first_reply) { }
440
441 virtual void OnAnswerDelay(Message* reply_msg) {
442 // Use the DELAY_REPLY macro so that we can force the reply to be sent
443 // before this function returns (when the channel will be reset).
444 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
445 Send(reply_msg);
446 got_first_reply_->Wait();
447 CloseChannel();
448 Done();
449 }
450
451 WaitableEvent* got_first_reply_;
452};
453
454void NoHang(bool pump_during_send) {
455 WaitableEvent got_first_reply(false, false);
456 std::vector<Worker*> workers;
457 workers.push_back(new NoHangServer(&got_first_reply, pump_during_send));
458 workers.push_back(new NoHangClient(&got_first_reply));
459 RunTest(workers);
460}
461
462} // namespace
463
464// Tests that caller doesn't hang if receiver dies
465TEST_F(IPCSyncChannelTest, NoHang) {
466 NoHang(false);
467 NoHang(true);
468}
469
470//-----------------------------------------------------------------------------
471
472namespace {
473
474class UnblockServer : public Worker {
475 public:
jam@chromium.orgebd07182009-12-01 11:34:18 +0900476 UnblockServer(bool pump_during_send, bool delete_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900477 : Worker(Channel::MODE_SERVER, "unblock_server"),
jam@chromium.orgebd07182009-12-01 11:34:18 +0900478 pump_during_send_(pump_during_send),
479 delete_during_send_(delete_during_send) { }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900480 void Run() {
jam@chromium.orgebd07182009-12-01 11:34:18 +0900481 if (delete_during_send_) {
482 // Use custom code since race conditions mean the answer may or may not be
483 // available.
484 int answer = 0;
485 SyncMessage* msg = new SyncChannelTestMsg_AnswerToLife(&answer);
486 if (pump_during_send_)
487 msg->EnableMessagePumping();
488 Send(msg);
489 } else {
490 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
491 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900492 Done();
493 }
494
jam@chromium.orgebd07182009-12-01 11:34:18 +0900495 void OnDoubleDelay(int in, Message* reply_msg) {
496 SyncChannelTestMsg_Double::WriteReplyParams(reply_msg, in * 2);
497 Send(reply_msg);
498 if (delete_during_send_)
499 ResetChannel();
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900500 }
501
502 bool pump_during_send_;
jam@chromium.orgebd07182009-12-01 11:34:18 +0900503 bool delete_during_send_;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900504};
505
506class UnblockClient : public Worker {
507 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900508 explicit UnblockClient(bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900509 : Worker(Channel::MODE_CLIENT, "unblock_client"),
510 pump_during_send_(pump_during_send) { }
511
512 void OnAnswer(int* answer) {
513 SendDouble(pump_during_send_, true);
514 *answer = 42;
515 Done();
516 }
517
518 bool pump_during_send_;
519};
520
jam@chromium.orgebd07182009-12-01 11:34:18 +0900521void Unblock(bool server_pump, bool client_pump, bool delete_during_send) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900522 std::vector<Worker*> workers;
jam@chromium.orgebd07182009-12-01 11:34:18 +0900523 workers.push_back(new UnblockServer(server_pump, delete_during_send));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900524 workers.push_back(new UnblockClient(client_pump));
525 RunTest(workers);
526}
527
528} // namespace
529
530// Tests that the caller unblocks to answer a sync message from the receiver.
531TEST_F(IPCSyncChannelTest, Unblock) {
jam@chromium.orgebd07182009-12-01 11:34:18 +0900532 Unblock(false, false, false);
533 Unblock(false, true, false);
534 Unblock(true, false, false);
535 Unblock(true, true, false);
536}
537
538//-----------------------------------------------------------------------------
539
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +0900540// Tests that the the SyncChannel object can be deleted during a Send.
jam@chromium.orgebd07182009-12-01 11:34:18 +0900541TEST_F(IPCSyncChannelTest, ChannelDeleteDuringSend) {
542 Unblock(false, false, true);
543 Unblock(false, true, true);
544 Unblock(true, false, true);
545 Unblock(true, true, true);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900546}
547
548//-----------------------------------------------------------------------------
549
550namespace {
551
552class RecursiveServer : public Worker {
553 public:
dilmah@chromium.org92b4c8e2011-07-27 02:25:25 +0900554 RecursiveServer(bool expected_send_result, bool pump_first, bool pump_second)
555 : Worker(Channel::MODE_SERVER, "recursive_server"),
556 expected_send_result_(expected_send_result),
557 pump_first_(pump_first), pump_second_(pump_second) {}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900558 void Run() {
559 SendDouble(pump_first_, expected_send_result_);
560 Done();
561 }
562
563 void OnDouble(int in, int* out) {
564 *out = in * 2;
565 SendAnswerToLife(pump_second_, base::kNoTimeout, expected_send_result_);
566 }
567
568 bool expected_send_result_, pump_first_, pump_second_;
569};
570
571class RecursiveClient : public Worker {
572 public:
dilmah@chromium.org92b4c8e2011-07-27 02:25:25 +0900573 RecursiveClient(bool pump_during_send, bool close_channel)
574 : Worker(Channel::MODE_CLIENT, "recursive_client"),
575 pump_during_send_(pump_during_send), close_channel_(close_channel) {}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900576
577 void OnDoubleDelay(int in, Message* reply_msg) {
578 SendDouble(pump_during_send_, !close_channel_);
579 if (close_channel_) {
580 delete reply_msg;
581 } else {
582 SyncChannelTestMsg_Double::WriteReplyParams(reply_msg, in * 2);
583 Send(reply_msg);
584 }
585 Done();
586 }
587
588 void OnAnswerDelay(Message* reply_msg) {
589 if (close_channel_) {
590 delete reply_msg;
591 CloseChannel();
592 } else {
593 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
594 Send(reply_msg);
595 }
596 }
597
598 bool pump_during_send_, close_channel_;
599};
600
601void Recursive(
602 bool server_pump_first, bool server_pump_second, bool client_pump) {
603 std::vector<Worker*> workers;
604 workers.push_back(
605 new RecursiveServer(true, server_pump_first, server_pump_second));
606 workers.push_back(new RecursiveClient(client_pump, false));
607 RunTest(workers);
608}
609
610} // namespace
611
612// Tests a server calling Send while another Send is pending.
613TEST_F(IPCSyncChannelTest, Recursive) {
614 Recursive(false, false, false);
615 Recursive(false, false, true);
616 Recursive(false, true, false);
617 Recursive(false, true, true);
618 Recursive(true, false, false);
619 Recursive(true, false, true);
620 Recursive(true, true, false);
621 Recursive(true, true, true);
622}
623
624//-----------------------------------------------------------------------------
625
626namespace {
627
628void RecursiveNoHang(
629 bool server_pump_first, bool server_pump_second, bool client_pump) {
630 std::vector<Worker*> workers;
631 workers.push_back(
632 new RecursiveServer(false, server_pump_first, server_pump_second));
633 workers.push_back(new RecursiveClient(client_pump, true));
634 RunTest(workers);
635}
636
637} // namespace
638
639// Tests that if a caller makes a sync call during an existing sync call and
640// the receiver dies, neither of the Send() calls hang.
641TEST_F(IPCSyncChannelTest, RecursiveNoHang) {
642 RecursiveNoHang(false, false, false);
643 RecursiveNoHang(false, false, true);
644 RecursiveNoHang(false, true, false);
645 RecursiveNoHang(false, true, true);
646 RecursiveNoHang(true, false, false);
647 RecursiveNoHang(true, false, true);
648 RecursiveNoHang(true, true, false);
649 RecursiveNoHang(true, true, true);
650}
651
652//-----------------------------------------------------------------------------
653
654namespace {
655
656class MultipleServer1 : public Worker {
657 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900658 explicit MultipleServer1(bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900659 : Worker("test_channel1", Channel::MODE_SERVER),
660 pump_during_send_(pump_during_send) { }
661
662 void Run() {
663 SendDouble(pump_during_send_, true);
664 Done();
665 }
666
667 bool pump_during_send_;
668};
669
670class MultipleClient1 : public Worker {
671 public:
672 MultipleClient1(WaitableEvent* client1_msg_received,
673 WaitableEvent* client1_can_reply) :
674 Worker("test_channel1", Channel::MODE_CLIENT),
675 client1_msg_received_(client1_msg_received),
676 client1_can_reply_(client1_can_reply) { }
677
678 void OnDouble(int in, int* out) {
679 client1_msg_received_->Signal();
680 *out = in * 2;
681 client1_can_reply_->Wait();
682 Done();
683 }
684
685 private:
686 WaitableEvent *client1_msg_received_, *client1_can_reply_;
687};
688
689class MultipleServer2 : public Worker {
690 public:
691 MultipleServer2() : Worker("test_channel2", Channel::MODE_SERVER) { }
692
693 void OnAnswer(int* result) {
694 *result = 42;
695 Done();
696 }
697};
698
699class MultipleClient2 : public Worker {
700 public:
701 MultipleClient2(
702 WaitableEvent* client1_msg_received, WaitableEvent* client1_can_reply,
703 bool pump_during_send)
704 : Worker("test_channel2", Channel::MODE_CLIENT),
705 client1_msg_received_(client1_msg_received),
706 client1_can_reply_(client1_can_reply),
707 pump_during_send_(pump_during_send) { }
708
709 void Run() {
710 client1_msg_received_->Wait();
711 SendAnswerToLife(pump_during_send_, base::kNoTimeout, true);
712 client1_can_reply_->Signal();
713 Done();
714 }
715
716 private:
717 WaitableEvent *client1_msg_received_, *client1_can_reply_;
718 bool pump_during_send_;
719};
720
721void Multiple(bool server_pump, bool client_pump) {
722 std::vector<Worker*> workers;
723
724 // A shared worker thread so that server1 and server2 run on one thread.
725 base::Thread worker_thread("Multiple");
726 ASSERT_TRUE(worker_thread.Start());
727
728 // Server1 sends a sync msg to client1, which blocks the reply until
729 // server2 (which runs on the same worker thread as server1) responds
730 // to a sync msg from client2.
731 WaitableEvent client1_msg_received(false, false);
732 WaitableEvent client1_can_reply(false, false);
733
734 Worker* worker;
735
736 worker = new MultipleServer2();
737 worker->OverrideThread(&worker_thread);
738 workers.push_back(worker);
739
740 worker = new MultipleClient2(
741 &client1_msg_received, &client1_can_reply, client_pump);
742 workers.push_back(worker);
743
744 worker = new MultipleServer1(server_pump);
745 worker->OverrideThread(&worker_thread);
746 workers.push_back(worker);
747
748 worker = new MultipleClient1(
749 &client1_msg_received, &client1_can_reply);
750 workers.push_back(worker);
751
752 RunTest(workers);
753}
754
755} // namespace
756
757// Tests that multiple SyncObjects on the same listener thread can unblock each
758// other.
759TEST_F(IPCSyncChannelTest, Multiple) {
760 Multiple(false, false);
761 Multiple(false, true);
762 Multiple(true, false);
763 Multiple(true, true);
764}
765
766//-----------------------------------------------------------------------------
767
768namespace {
769
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900770// This class provides server side functionality to test the case where
771// multiple sync channels are in use on the same thread on the client and
772// nested calls are issued.
773class QueuedReplyServer : public Worker {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900774 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900775 QueuedReplyServer(base::Thread* listener_thread,
776 const std::string& channel_name,
777 const std::string& reply_text)
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900778 : Worker(channel_name, Channel::MODE_SERVER),
779 reply_text_(reply_text) {
780 Worker::OverrideThread(listener_thread);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900781 }
782
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900783 virtual void OnNestedTestMsg(Message* reply_msg) {
pkasting@chromium.orgfcdd54b2010-10-20 08:50:00 +0900784 VLOG(1) << __FUNCTION__ << " Sending reply: " << reply_text_;
785 SyncChannelNestedTestMsg_String::WriteReplyParams(reply_msg, reply_text_);
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900786 Send(reply_msg);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900787 Done();
788 }
789
790 private:
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900791 std::string reply_text_;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900792};
793
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900794// The QueuedReplyClient class provides functionality to test the case where
795// multiple sync channels are in use on the same thread and they make nested
796// sync calls, i.e. while the first channel waits for a response it makes a
797// sync call on another channel.
798// The callstack should unwind correctly, i.e. the outermost call should
799// complete first, and so on.
800class QueuedReplyClient : public Worker {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900801 public:
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900802 QueuedReplyClient(base::Thread* listener_thread,
803 const std::string& channel_name,
804 const std::string& expected_text,
805 bool pump_during_send)
806 : Worker(channel_name, Channel::MODE_CLIENT),
thomasvl@google.com9a242072010-07-23 23:18:59 +0900807 pump_during_send_(pump_during_send),
808 expected_text_(expected_text) {
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900809 Worker::OverrideThread(listener_thread);
810 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900811
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900812 virtual void Run() {
813 std::string response;
814 SyncMessage* msg = new SyncChannelNestedTestMsg_String(&response);
815 if (pump_during_send_)
816 msg->EnableMessagePumping();
817 bool result = Send(msg);
818 DCHECK(result);
jam@chromium.orgebd07182009-12-01 11:34:18 +0900819 DCHECK_EQ(response, expected_text_);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900820
pkasting@chromium.orgfcdd54b2010-10-20 08:50:00 +0900821 VLOG(1) << __FUNCTION__ << " Received reply: " << response;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900822 Done();
823 }
824
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900825 private:
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900826 bool pump_during_send_;
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900827 std::string expected_text_;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900828};
829
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900830void QueuedReply(bool client_pump) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900831 std::vector<Worker*> workers;
832
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900833 // A shared worker thread for servers
834 base::Thread server_worker_thread("QueuedReply_ServerListener");
835 ASSERT_TRUE(server_worker_thread.Start());
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900836
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900837 base::Thread client_worker_thread("QueuedReply_ClientListener");
838 ASSERT_TRUE(client_worker_thread.Start());
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900839
840 Worker* worker;
841
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900842 worker = new QueuedReplyServer(&server_worker_thread,
843 "QueuedReply_Server1",
844 "Got first message");
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900845 workers.push_back(worker);
846
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900847 worker = new QueuedReplyServer(&server_worker_thread,
848 "QueuedReply_Server2",
849 "Got second message");
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900850 workers.push_back(worker);
851
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900852 worker = new QueuedReplyClient(&client_worker_thread,
853 "QueuedReply_Server1",
854 "Got first message",
855 client_pump);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900856 workers.push_back(worker);
857
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900858 worker = new QueuedReplyClient(&client_worker_thread,
859 "QueuedReply_Server2",
860 "Got second message",
861 client_pump);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900862 workers.push_back(worker);
863
864 RunTest(workers);
865}
866
867} // namespace
868
869// While a blocking send is in progress, the listener thread might answer other
870// synchronous messages. This tests that if during the response to another
871// message the reply to the original messages comes, it is queued up correctly
872// and the original Send is unblocked later.
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900873// We also test that the send call stacks unwind correctly when the channel
874// pumps messages while waiting for a response.
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900875TEST_F(IPCSyncChannelTest, QueuedReply) {
ananta@chromium.org31b338f2009-10-15 01:22:02 +0900876 QueuedReply(false);
877 QueuedReply(true);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900878}
879
880//-----------------------------------------------------------------------------
881
882namespace {
883
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900884class ChattyClient : public Worker {
885 public:
886 ChattyClient() :
887 Worker(Channel::MODE_CLIENT, "chatty_client") { }
888
889 void OnAnswer(int* answer) {
890 // The PostMessage limit is 10k. Send 20% more than that.
891 const int kMessageLimit = 10000;
892 const int kMessagesToSend = kMessageLimit * 120 / 100;
893 for (int i = 0; i < kMessagesToSend; ++i) {
894 if (!SendDouble(false, true))
895 break;
896 }
897 *answer = 42;
898 Done();
899 }
900};
901
902void ChattyServer(bool pump_during_send) {
903 std::vector<Worker*> workers;
jam@chromium.orgebd07182009-12-01 11:34:18 +0900904 workers.push_back(new UnblockServer(pump_during_send, false));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900905 workers.push_back(new ChattyClient());
906 RunTest(workers);
907}
908
909} // namespace
910
911// Tests http://b/1093251 - that sending lots of sync messages while
912// the receiver is waiting for a sync reply does not overflow the PostMessage
913// queue.
914TEST_F(IPCSyncChannelTest, ChattyServer) {
915 ChattyServer(false);
916 ChattyServer(true);
917}
918
919//------------------------------------------------------------------------------
920
921namespace {
922
923class TimeoutServer : public Worker {
924 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900925 TimeoutServer(int timeout_ms,
926 std::vector<bool> timeout_seq,
927 bool pump_during_send)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900928 : Worker(Channel::MODE_SERVER, "timeout_server"),
929 timeout_ms_(timeout_ms),
930 timeout_seq_(timeout_seq),
931 pump_during_send_(pump_during_send) {
932 }
933
934 void Run() {
935 for (std::vector<bool>::const_iterator iter = timeout_seq_.begin();
936 iter != timeout_seq_.end(); ++iter) {
937 SendAnswerToLife(pump_during_send_, timeout_ms_, !*iter);
938 }
939 Done();
940 }
941
942 private:
943 int timeout_ms_;
944 std::vector<bool> timeout_seq_;
945 bool pump_during_send_;
946};
947
948class UnresponsiveClient : public Worker {
949 public:
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900950 explicit UnresponsiveClient(std::vector<bool> timeout_seq)
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900951 : Worker(Channel::MODE_CLIENT, "unresponsive_client"),
952 timeout_seq_(timeout_seq) {
thestig@chromium.org60bd7c02010-07-23 14:23:13 +0900953 }
agl@chromium.org1c6dcf22009-07-23 08:57:21 +0900954
955 void OnAnswerDelay(Message* reply_msg) {
956 DCHECK(!timeout_seq_.empty());
957 if (!timeout_seq_[0]) {
958 SyncChannelTestMsg_AnswerToLife::WriteReplyParams(reply_msg, 42);
959 Send(reply_msg);
960 } else {
961 // Don't reply.
962 delete reply_msg;
963 }
964 timeout_seq_.erase(timeout_seq_.begin());
965 if (timeout_seq_.empty())
966 Done();
967 }
968
969 private:
970 // Whether we should time-out or respond to the various messages we receive.
971 std::vector<bool> timeout_seq_;
972};
973
974void SendWithTimeoutOK(bool pump_during_send) {
975 std::vector<Worker*> workers;
976 std::vector<bool> timeout_seq;
977 timeout_seq.push_back(false);
978 timeout_seq.push_back(false);
979 timeout_seq.push_back(false);
980 workers.push_back(new TimeoutServer(5000, timeout_seq, pump_during_send));
981 workers.push_back(new SimpleClient());
982 RunTest(workers);
983}
984
985void SendWithTimeoutTimeout(bool pump_during_send) {
986 std::vector<Worker*> workers;
987 std::vector<bool> timeout_seq;
988 timeout_seq.push_back(true);
989 timeout_seq.push_back(false);
990 timeout_seq.push_back(false);
991 workers.push_back(new TimeoutServer(100, timeout_seq, pump_during_send));
992 workers.push_back(new UnresponsiveClient(timeout_seq));
993 RunTest(workers);
994}
995
996void SendWithTimeoutMixedOKAndTimeout(bool pump_during_send) {
997 std::vector<Worker*> workers;
998 std::vector<bool> timeout_seq;
999 timeout_seq.push_back(true);
1000 timeout_seq.push_back(false);
1001 timeout_seq.push_back(false);
1002 timeout_seq.push_back(true);
1003 timeout_seq.push_back(false);
1004 workers.push_back(new TimeoutServer(100, timeout_seq, pump_during_send));
1005 workers.push_back(new UnresponsiveClient(timeout_seq));
1006 RunTest(workers);
1007}
1008
1009} // namespace
1010
1011// Tests that SendWithTimeout does not time-out if the response comes back fast
1012// enough.
1013TEST_F(IPCSyncChannelTest, SendWithTimeoutOK) {
1014 SendWithTimeoutOK(false);
1015 SendWithTimeoutOK(true);
1016}
1017
1018// Tests that SendWithTimeout does time-out.
1019TEST_F(IPCSyncChannelTest, SendWithTimeoutTimeout) {
1020 SendWithTimeoutTimeout(false);
1021 SendWithTimeoutTimeout(true);
1022}
1023
1024// Sends some message that time-out and some that succeed.
phajdan.jr@chromium.orga8e3c092011-01-19 17:11:56 +09001025// Crashes flakily, http://crbug.com/70075.
1026TEST_F(IPCSyncChannelTest, DISABLED_SendWithTimeoutMixedOKAndTimeout) {
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001027 SendWithTimeoutMixedOKAndTimeout(false);
1028 SendWithTimeoutMixedOKAndTimeout(true);
1029}
1030
1031//------------------------------------------------------------------------------
1032
1033namespace {
1034
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001035void NestedCallback(Worker* server) {
1036 // Sleep a bit so that we wake up after the reply has been received.
tedvessenes@gmail.com30dbbaa2012-01-13 09:11:01 +09001037 base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(250));
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001038 server->SendAnswerToLife(true, base::kNoTimeout, true);
1039}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001040
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001041bool timeout_occurred = false;
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001042
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001043void TimeoutCallback() {
1044 timeout_occurred = true;
1045}
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001046
1047class DoneEventRaceServer : public Worker {
1048 public:
1049 DoneEventRaceServer()
1050 : Worker(Channel::MODE_SERVER, "done_event_race_server") { }
1051
1052 void Run() {
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001053 MessageLoop::current()->PostTask(FROM_HERE,
1054 base::Bind(&NestedCallback, this));
1055 MessageLoop::current()->PostDelayedTask(
tedvessenes@gmail.com56b33702012-03-07 13:41:40 +09001056 FROM_HERE,
1057 base::Bind(&TimeoutCallback),
1058 base::TimeDelta::FromSeconds(9));
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001059 // Even though we have a timeout on the Send, it will succeed since for this
1060 // bug, the reply message comes back and is deserialized, however the done
1061 // event wasn't set. So we indirectly use the timeout task to notice if a
1062 // timeout occurred.
1063 SendAnswerToLife(true, 10000, true);
jhawkins@chromium.orgdc0f37f2011-11-29 03:35:39 +09001064 DCHECK(!timeout_occurred);
agl@chromium.org1c6dcf22009-07-23 08:57:21 +09001065 Done();
1066 }
1067};
1068
1069} // namespace
1070
1071// Tests http://b/1474092 - that if after the done_event is set but before
1072// OnObjectSignaled is called another message is sent out, then after its
1073// reply comes back OnObjectSignaled will be called for the first message.
1074TEST_F(IPCSyncChannelTest, DoneEventRace) {
1075 std::vector<Worker*> workers;
1076 workers.push_back(new DoneEventRaceServer());
1077 workers.push_back(new SimpleClient());
1078 RunTest(workers);
1079}
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001080
1081//-----------------------------------------------------------------------------
1082
1083namespace {
1084
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001085class TestSyncMessageFilter : public SyncMessageFilter {
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001086 public:
1087 TestSyncMessageFilter(base::WaitableEvent* shutdown_event, Worker* worker)
1088 : SyncMessageFilter(shutdown_event),
1089 worker_(worker),
1090 thread_("helper_thread") {
1091 base::Thread::Options options;
1092 options.message_loop_type = MessageLoop::TYPE_DEFAULT;
1093 thread_.StartWithOptions(options);
1094 }
1095
1096 virtual void OnFilterAdded(Channel* channel) {
1097 SyncMessageFilter::OnFilterAdded(channel);
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001098 thread_.message_loop()->PostTask(
1099 FROM_HERE,
1100 base::Bind(&TestSyncMessageFilter::SendMessageOnHelperThread, this));
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001101 }
1102
1103 void SendMessageOnHelperThread() {
1104 int answer = 0;
1105 bool result = Send(new SyncChannelTestMsg_AnswerToLife(&answer));
1106 DCHECK(result);
1107 DCHECK_EQ(answer, 42);
1108
1109 worker_->Done();
1110 }
1111
1112 Worker* worker_;
1113 base::Thread thread_;
1114};
1115
1116class SyncMessageFilterServer : public Worker {
1117 public:
1118 SyncMessageFilterServer()
1119 : Worker(Channel::MODE_SERVER, "sync_message_filter_server") {
1120 filter_ = new TestSyncMessageFilter(shutdown_event(), this);
1121 }
1122
1123 void Run() {
1124 channel()->AddFilter(filter_.get());
1125 }
1126
1127 scoped_refptr<TestSyncMessageFilter> filter_;
1128};
1129
ananta@chromium.org999f2972010-09-03 06:45:50 +09001130// This class provides functionality to test the case that a Send on the sync
1131// channel does not crash after the channel has been closed.
1132class ServerSendAfterClose : public Worker {
1133 public:
1134 ServerSendAfterClose()
1135 : Worker(Channel::MODE_SERVER, "simpler_server"),
1136 send_result_(true) {
1137 }
1138
1139 bool SendDummy() {
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001140 ListenerThread()->message_loop()->PostTask(
ajwong@chromium.orgc1e632d2012-01-06 11:37:17 +09001141 FROM_HERE, base::Bind(base::IgnoreResult(&ServerSendAfterClose::Send),
1142 this, new SyncChannelTestMsg_NoArgs));
ananta@chromium.org999f2972010-09-03 06:45:50 +09001143 return true;
1144 }
1145
1146 bool send_result() const {
1147 return send_result_;
1148 }
1149
1150 private:
1151 virtual void Run() {
1152 CloseChannel();
1153 Done();
1154 }
1155
1156 bool Send(Message* msg) {
1157 send_result_ = Worker::Send(msg);
1158 Done();
1159 return send_result_;
1160 }
1161
1162 bool send_result_;
1163};
1164
jabdelmalek@google.comeb921652010-04-07 05:33:36 +09001165} // namespace
1166
1167// Tests basic synchronous call
1168TEST_F(IPCSyncChannelTest, SyncMessageFilter) {
1169 std::vector<Worker*> workers;
1170 workers.push_back(new SyncMessageFilterServer());
1171 workers.push_back(new SimpleClient());
1172 RunTest(workers);
1173}
ananta@chromium.org999f2972010-09-03 06:45:50 +09001174
1175// Test the case when the channel is closed and a Send is attempted after that.
1176TEST_F(IPCSyncChannelTest, SendAfterClose) {
1177 ServerSendAfterClose server;
1178 server.Start();
1179
1180 server.done_event()->Wait();
1181 server.done_event()->Reset();
1182
1183 server.SendDummy();
1184 server.done_event()->Wait();
1185
1186 EXPECT_FALSE(server.send_result());
1187}
1188
piman@google.com0cbefaa2011-04-08 12:38:21 +09001189//-----------------------------------------------------------------------------
ananta@chromium.org999f2972010-09-03 06:45:50 +09001190
piman@google.com0cbefaa2011-04-08 12:38:21 +09001191namespace {
1192
1193class RestrictedDispatchServer : public Worker {
1194 public:
piman@chromium.org5a00b882012-03-31 06:29:30 +09001195 RestrictedDispatchServer(WaitableEvent* sent_ping_event,
1196 WaitableEvent* wait_event)
piman@google.com0cbefaa2011-04-08 12:38:21 +09001197 : Worker("restricted_channel", Channel::MODE_SERVER),
piman@chromium.org5a00b882012-03-31 06:29:30 +09001198 sent_ping_event_(sent_ping_event),
1199 wait_event_(wait_event) { }
piman@google.com0cbefaa2011-04-08 12:38:21 +09001200
1201 void OnDoPing(int ping) {
1202 // Send an asynchronous message that unblocks the caller.
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001203 Message* msg = new SyncChannelTestMsg_Ping(ping);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001204 msg->set_unblock(true);
1205 Send(msg);
1206 // Signal the event after the message has been sent on the channel, on the
1207 // IPC thread.
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001208 ipc_thread().message_loop()->PostTask(
1209 FROM_HERE, base::Bind(&RestrictedDispatchServer::OnPingSent, this));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001210 }
1211
piman@chromium.org5a00b882012-03-31 06:29:30 +09001212 void OnPingTTL(int ping, int* out) {
1213 *out = ping;
1214 wait_event_->Wait();
1215 }
1216
piman@google.com0cbefaa2011-04-08 12:38:21 +09001217 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1218
1219 private:
1220 bool OnMessageReceived(const Message& message) {
1221 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchServer, message)
1222 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
piman@chromium.org5a00b882012-03-31 06:29:30 +09001223 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_PingTTL, OnPingTTL)
piman@google.com0cbefaa2011-04-08 12:38:21 +09001224 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, Done)
1225 IPC_END_MESSAGE_MAP()
1226 return true;
1227 }
1228
1229 void OnPingSent() {
1230 sent_ping_event_->Signal();
1231 }
1232
1233 void OnNoArgs() { }
1234 WaitableEvent* sent_ping_event_;
piman@chromium.org5a00b882012-03-31 06:29:30 +09001235 WaitableEvent* wait_event_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001236};
1237
1238class NonRestrictedDispatchServer : public Worker {
1239 public:
piman@chromium.org5a00b882012-03-31 06:29:30 +09001240 NonRestrictedDispatchServer(WaitableEvent* signal_event)
1241 : Worker("non_restricted_channel", Channel::MODE_SERVER),
1242 signal_event_(signal_event) {}
1243
1244 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1245
1246 void OnDoPingTTL(int ping) {
1247 int value = 0;
1248 Send(new SyncChannelTestMsg_PingTTL(ping, &value));
1249 signal_event_->Signal();
1250 }
piman@google.com0cbefaa2011-04-08 12:38:21 +09001251
1252 private:
1253 bool OnMessageReceived(const Message& message) {
1254 IPC_BEGIN_MESSAGE_MAP(NonRestrictedDispatchServer, message)
1255 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1256 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, Done)
1257 IPC_END_MESSAGE_MAP()
1258 return true;
1259 }
1260
1261 void OnNoArgs() { }
piman@chromium.org5a00b882012-03-31 06:29:30 +09001262 WaitableEvent* signal_event_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001263};
1264
1265class RestrictedDispatchClient : public Worker {
1266 public:
1267 RestrictedDispatchClient(WaitableEvent* sent_ping_event,
1268 RestrictedDispatchServer* server,
piman@chromium.org5a00b882012-03-31 06:29:30 +09001269 NonRestrictedDispatchServer* server2,
piman@google.com0cbefaa2011-04-08 12:38:21 +09001270 int* success)
1271 : Worker("restricted_channel", Channel::MODE_CLIENT),
1272 ping_(0),
1273 server_(server),
piman@chromium.org5a00b882012-03-31 06:29:30 +09001274 server2_(server2),
piman@google.com0cbefaa2011-04-08 12:38:21 +09001275 success_(success),
1276 sent_ping_event_(sent_ping_event) {}
1277
1278 void Run() {
1279 // Incoming messages from our channel should only be dispatched when we
1280 // send a message on that same channel.
piman@chromium.org5a00b882012-03-31 06:29:30 +09001281 channel()->SetRestrictDispatchChannelGroup(1);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001282
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001283 server_->ListenerThread()->message_loop()->PostTask(
1284 FROM_HERE, base::Bind(&RestrictedDispatchServer::OnDoPing, server_, 1));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001285 sent_ping_event_->Wait();
1286 Send(new SyncChannelTestMsg_NoArgs);
1287 if (ping_ == 1)
1288 ++*success_;
1289 else
1290 LOG(ERROR) << "Send failed to dispatch incoming message on same channel";
1291
piman@chromium.org5a00b882012-03-31 06:29:30 +09001292 non_restricted_channel_.reset(new SyncChannel(
piman@google.com0cbefaa2011-04-08 12:38:21 +09001293 "non_restricted_channel", Channel::MODE_CLIENT, this,
jam@chromium.org06d18442011-05-03 03:00:49 +09001294 ipc_thread().message_loop_proxy(), true, shutdown_event()));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001295
jhawkins@chromium.org9827bd12011-11-13 06:16:41 +09001296 server_->ListenerThread()->message_loop()->PostTask(
1297 FROM_HERE, base::Bind(&RestrictedDispatchServer::OnDoPing, server_, 2));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001298 sent_ping_event_->Wait();
1299 // Check that the incoming message is *not* dispatched when sending on the
1300 // non restricted channel.
1301 // TODO(piman): there is a possibility of a false positive race condition
1302 // here, if the message that was posted on the server-side end of the pipe
1303 // is not visible yet on the client side, but I don't know how to solve this
1304 // without hooking into the internals of SyncChannel. I haven't seen it in
1305 // practice (i.e. not setting SetRestrictDispatchToSameChannel does cause
1306 // the following to fail).
piman@chromium.org5a00b882012-03-31 06:29:30 +09001307 non_restricted_channel_->Send(new SyncChannelTestMsg_NoArgs);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001308 if (ping_ == 1)
1309 ++*success_;
1310 else
1311 LOG(ERROR) << "Send dispatched message from restricted channel";
1312
1313 Send(new SyncChannelTestMsg_NoArgs);
1314 if (ping_ == 2)
1315 ++*success_;
1316 else
1317 LOG(ERROR) << "Send failed to dispatch incoming message on same channel";
1318
piman@chromium.org5a00b882012-03-31 06:29:30 +09001319 // Check that the incoming message on the non-restricted channel is
1320 // dispatched when sending on the restricted channel.
1321 server2_->ListenerThread()->message_loop()->PostTask(
1322 FROM_HERE,
1323 base::Bind(&NonRestrictedDispatchServer::OnDoPingTTL, server2_, 3));
1324 int value = 0;
1325 Send(new SyncChannelTestMsg_PingTTL(4, &value));
1326 if (ping_ == 3 && value == 4)
1327 ++*success_;
1328 else
1329 LOG(ERROR) << "Send failed to dispatch message from unrestricted channel";
1330
1331 non_restricted_channel_->Send(new SyncChannelTestMsg_Done);
1332 non_restricted_channel_.reset();
piman@google.com0cbefaa2011-04-08 12:38:21 +09001333 Send(new SyncChannelTestMsg_Done);
1334 Done();
1335 }
1336
1337 private:
1338 bool OnMessageReceived(const Message& message) {
1339 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchClient, message)
1340 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Ping, OnPing)
piman@chromium.org5a00b882012-03-31 06:29:30 +09001341 IPC_MESSAGE_HANDLER_DELAY_REPLY(SyncChannelTestMsg_PingTTL, OnPingTTL)
piman@google.com0cbefaa2011-04-08 12:38:21 +09001342 IPC_END_MESSAGE_MAP()
1343 return true;
1344 }
1345
1346 void OnPing(int ping) {
1347 ping_ = ping;
1348 }
1349
piman@chromium.org5a00b882012-03-31 06:29:30 +09001350 void OnPingTTL(int ping, IPC::Message* reply) {
1351 ping_ = ping;
1352 // This message comes from the NonRestrictedDispatchServer, we have to send
1353 // the reply back manually.
1354 SyncChannelTestMsg_PingTTL::WriteReplyParams(reply, ping);
1355 non_restricted_channel_->Send(reply);
1356 }
1357
piman@google.com0cbefaa2011-04-08 12:38:21 +09001358 int ping_;
1359 RestrictedDispatchServer* server_;
piman@chromium.org5a00b882012-03-31 06:29:30 +09001360 NonRestrictedDispatchServer* server2_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001361 int* success_;
1362 WaitableEvent* sent_ping_event_;
piman@chromium.org5a00b882012-03-31 06:29:30 +09001363 scoped_ptr<SyncChannel> non_restricted_channel_;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001364};
1365
1366} // namespace
1367
1368TEST_F(IPCSyncChannelTest, RestrictedDispatch) {
1369 WaitableEvent sent_ping_event(false, false);
piman@chromium.org5a00b882012-03-31 06:29:30 +09001370 WaitableEvent wait_event(false, false);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001371 RestrictedDispatchServer* server =
piman@chromium.org5a00b882012-03-31 06:29:30 +09001372 new RestrictedDispatchServer(&sent_ping_event, &wait_event);
1373 NonRestrictedDispatchServer* server2 =
1374 new NonRestrictedDispatchServer(&wait_event);
1375
piman@google.com0cbefaa2011-04-08 12:38:21 +09001376 int success = 0;
1377 std::vector<Worker*> workers;
piman@google.com0cbefaa2011-04-08 12:38:21 +09001378 workers.push_back(server);
piman@chromium.org5a00b882012-03-31 06:29:30 +09001379 workers.push_back(server2);
1380 workers.push_back(new RestrictedDispatchClient(
1381 &sent_ping_event, server, server2, &success));
piman@google.com0cbefaa2011-04-08 12:38:21 +09001382 RunTest(workers);
piman@chromium.org5a00b882012-03-31 06:29:30 +09001383 EXPECT_EQ(4, success);
piman@google.com0cbefaa2011-04-08 12:38:21 +09001384}
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001385
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001386//-----------------------------------------------------------------------------
1387
1388// This test case inspired by crbug.com/108491
1389// We create two servers that use the same ListenerThread but have
1390// SetRestrictDispatchToSameChannel set to true.
1391// We create clients, then use some specific WaitableEvent wait/signalling to
1392// ensure that messages get dispatched in a way that causes a deadlock due to
1393// a nested dispatch and an eligible message in a higher-level dispatch's
1394// delayed_queue. Specifically, we start with client1 about so send an
1395// unblocking message to server1, while the shared listener thread for the
1396// servers server1 and server2 is about to send a non-unblocking message to
1397// client1. At the same time, client2 will be about to send an unblocking
1398// message to server2. Server1 will handle the client1->server1 message by
1399// telling server2 to send a non-unblocking message to client2.
1400// What should happen is that the send to server2 should find the pending,
1401// same-context client2->server2 message to dispatch, causing client2 to
1402// unblock then handle the server2->client2 message, so that the shared
1403// servers' listener thread can then respond to the client1->server1 message.
1404// Then client1 can handle the non-unblocking server1->client1 message.
1405// The old code would end up in a state where the server2->client2 message is
1406// sent, but the client2->server2 message (which is eligible for dispatch, and
1407// which is what client2 is waiting for) is stashed in a local delayed_queue
1408// that has server1's channel context, causing a deadlock.
1409// WaitableEvents in the events array are used to:
1410// event 0: indicate to client1 that server listener is in OnDoServerTask
1411// event 1: indicate to client1 that client2 listener is in OnDoClient2Task
1412// event 2: indicate to server1 that client2 listener is in OnDoClient2Task
1413// event 3: indicate to client2 that server listener is in OnDoServerTask
1414
1415namespace {
1416
1417class RestrictedDispatchDeadlockServer : public Worker {
1418 public:
1419 RestrictedDispatchDeadlockServer(int server_num,
1420 WaitableEvent* server_ready_event,
1421 WaitableEvent** events,
1422 RestrictedDispatchDeadlockServer* peer)
1423 : Worker(server_num == 1 ? "channel1" : "channel2", Channel::MODE_SERVER),
1424 server_num_(server_num),
1425 server_ready_event_(server_ready_event),
1426 events_(events),
1427 peer_(peer),
1428 client_kicked_(false) { }
1429
1430 void OnDoServerTask() {
1431 events_[3]->Signal();
1432 events_[2]->Wait();
1433 events_[0]->Signal();
1434 SendMessageToClient();
1435 }
1436
1437 void Run() {
piman@chromium.org5a00b882012-03-31 06:29:30 +09001438 channel()->SetRestrictDispatchChannelGroup(1);
jhorwich@chromium.org3c3a8132012-01-12 07:39:54 +09001439 server_ready_event_->Signal();
1440 }
1441
1442 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1443
1444 private:
1445 bool OnMessageReceived(const Message& message) {
1446 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchDeadlockServer, message)
1447 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1448 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, Done)
1449 IPC_END_MESSAGE_MAP()
1450 return true;
1451 }
1452
1453 void OnNoArgs() {
1454 if (server_num_ == 1) {
1455 DCHECK(peer_ != NULL);
1456 peer_->SendMessageToClient();
1457 }
1458 }
1459
1460 void SendMessageToClient() {
1461 Message* msg = new SyncChannelTestMsg_NoArgs;
1462 msg->set_unblock(false);
1463 DCHECK(!msg->should_unblock());
1464 Send(msg);
1465 }
1466
1467 int server_num_;
1468 WaitableEvent* server_ready_event_;
1469 WaitableEvent** events_;
1470 RestrictedDispatchDeadlockServer* peer_;
1471 bool client_kicked_;
1472};
1473
1474class RestrictedDispatchDeadlockClient2 : public Worker {
1475 public:
1476 RestrictedDispatchDeadlockClient2(RestrictedDispatchDeadlockServer* server,
1477 WaitableEvent* server_ready_event,
1478 WaitableEvent** events)
1479 : Worker("channel2", Channel::MODE_CLIENT),
1480 server_(server),
1481 server_ready_event_(server_ready_event),
1482 events_(events),
1483 received_msg_(false),
1484 received_noarg_reply_(false),
1485 done_issued_(false) {}
1486
1487 void Run() {
1488 server_ready_event_->Wait();
1489 }
1490
1491 void OnDoClient2Task() {
1492 events_[3]->Wait();
1493 events_[1]->Signal();
1494 events_[2]->Signal();
1495 DCHECK(received_msg_ == false);
1496
1497 Message* message = new SyncChannelTestMsg_NoArgs;
1498 message->set_unblock(true);
1499 Send(message);
1500 received_noarg_reply_ = true;
1501 }
1502
1503 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1504 private:
1505 bool OnMessageReceived(const Message& message) {
1506 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchDeadlockClient2, message)
1507 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1508 IPC_END_MESSAGE_MAP()
1509 return true;
1510 }
1511
1512 void OnNoArgs() {
1513 received_msg_ = true;
1514 PossiblyDone();
1515 }
1516
1517 void PossiblyDone() {
1518 if (received_noarg_reply_ && received_msg_) {
1519 DCHECK(done_issued_ == false);
1520 done_issued_ = true;
1521 Send(new SyncChannelTestMsg_Done);
1522 Done();
1523 }
1524 }
1525
1526 RestrictedDispatchDeadlockServer* server_;
1527 WaitableEvent* server_ready_event_;
1528 WaitableEvent** events_;
1529 bool received_msg_;
1530 bool received_noarg_reply_;
1531 bool done_issued_;
1532};
1533
1534class RestrictedDispatchDeadlockClient1 : public Worker {
1535 public:
1536 RestrictedDispatchDeadlockClient1(RestrictedDispatchDeadlockServer* server,
1537 RestrictedDispatchDeadlockClient2* peer,
1538 WaitableEvent* server_ready_event,
1539 WaitableEvent** events)
1540 : Worker("channel1", Channel::MODE_CLIENT),
1541 server_(server),
1542 peer_(peer),
1543 server_ready_event_(server_ready_event),
1544 events_(events),
1545 received_msg_(false),
1546 received_noarg_reply_(false),
1547 done_issued_(false) {}
1548
1549 void Run() {
1550 server_ready_event_->Wait();
1551 server_->ListenerThread()->message_loop()->PostTask(
1552 FROM_HERE,
1553 base::Bind(&RestrictedDispatchDeadlockServer::OnDoServerTask, server_));
1554 peer_->ListenerThread()->message_loop()->PostTask(
1555 FROM_HERE,
1556 base::Bind(&RestrictedDispatchDeadlockClient2::OnDoClient2Task, peer_));
1557 events_[0]->Wait();
1558 events_[1]->Wait();
1559 DCHECK(received_msg_ == false);
1560
1561 Message* message = new SyncChannelTestMsg_NoArgs;
1562 message->set_unblock(true);
1563 Send(message);
1564 received_noarg_reply_ = true;
1565 PossiblyDone();
1566 }
1567
1568 base::Thread* ListenerThread() { return Worker::ListenerThread(); }
1569 private:
1570 bool OnMessageReceived(const Message& message) {
1571 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchDeadlockClient1, message)
1572 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_NoArgs, OnNoArgs)
1573 IPC_END_MESSAGE_MAP()
1574 return true;
1575 }
1576
1577 void OnNoArgs() {
1578 received_msg_ = true;
1579 PossiblyDone();
1580 }
1581
1582 void PossiblyDone() {
1583 if (received_noarg_reply_ && received_msg_) {
1584 DCHECK(done_issued_ == false);
1585 done_issued_ = true;
1586 Send(new SyncChannelTestMsg_Done);
1587 Done();
1588 }
1589 }
1590
1591 RestrictedDispatchDeadlockServer* server_;
1592 RestrictedDispatchDeadlockClient2* peer_;
1593 WaitableEvent* server_ready_event_;
1594 WaitableEvent** events_;
1595 bool received_msg_;
1596 bool received_noarg_reply_;
1597 bool done_issued_;
1598};
1599
1600} // namespace
1601
1602TEST_F(IPCSyncChannelTest, RestrictedDispatchDeadlock) {
1603 std::vector<Worker*> workers;
1604
1605 // A shared worker thread so that server1 and server2 run on one thread.
1606 base::Thread worker_thread("RestrictedDispatchDeadlock");
1607 ASSERT_TRUE(worker_thread.Start());
1608
1609 WaitableEvent server1_ready(false, false);
1610 WaitableEvent server2_ready(false, false);
1611
1612 WaitableEvent event0(false, false);
1613 WaitableEvent event1(false, false);
1614 WaitableEvent event2(false, false);
1615 WaitableEvent event3(false, false);
1616 WaitableEvent* events[4] = {&event0, &event1, &event2, &event3};
1617
1618 RestrictedDispatchDeadlockServer* server1;
1619 RestrictedDispatchDeadlockServer* server2;
1620 RestrictedDispatchDeadlockClient1* client1;
1621 RestrictedDispatchDeadlockClient2* client2;
1622
1623 server2 = new RestrictedDispatchDeadlockServer(2, &server2_ready, events,
1624 NULL);
1625 server2->OverrideThread(&worker_thread);
1626 workers.push_back(server2);
1627
1628 client2 = new RestrictedDispatchDeadlockClient2(server2, &server2_ready,
1629 events);
1630 workers.push_back(client2);
1631
1632 server1 = new RestrictedDispatchDeadlockServer(1, &server1_ready, events,
1633 server2);
1634 server1->OverrideThread(&worker_thread);
1635 workers.push_back(server1);
1636
1637 client1 = new RestrictedDispatchDeadlockClient1(server1, client2,
1638 &server1_ready, events);
1639 workers.push_back(client1);
1640
1641 RunTest(workers);
1642}
1643
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001644//-----------------------------------------------------------------------------
1645
piman@chromium.org5a00b882012-03-31 06:29:30 +09001646// This test case inspired by crbug.com/120530
1647// We create 4 workers that pipe to each other W1->W2->W3->W4->W1 then we send a
1648// message that recurses through 3, 4 or 5 steps to make sure, say, W1 can
1649// re-enter when called from W4 while it's sending a message to W2.
1650// The first worker drives the whole test so it must be treated specially.
1651namespace {
1652
1653class RestrictedDispatchPipeWorker : public Worker {
1654 public:
1655 RestrictedDispatchPipeWorker(
1656 const std::string &channel1,
1657 WaitableEvent* event1,
1658 const std::string &channel2,
1659 WaitableEvent* event2,
1660 int group,
1661 int* success)
1662 : Worker(channel1, Channel::MODE_SERVER),
1663 event1_(event1),
1664 event2_(event2),
1665 other_channel_name_(channel2),
1666 group_(group),
1667 success_(success) {
1668 }
1669
1670 void OnPingTTL(int ping, int* ret) {
1671 *ret = 0;
1672 if (!ping)
1673 return;
1674 other_channel_->Send(new SyncChannelTestMsg_PingTTL(ping - 1, ret));
1675 ++*ret;
1676 }
1677
1678 void OnDone() {
1679 if (is_first())
1680 return;
1681 other_channel_->Send(new SyncChannelTestMsg_Done);
1682 other_channel_.reset();
1683 Done();
1684 }
1685
1686 void Run() {
1687 channel()->SetRestrictDispatchChannelGroup(group_);
1688 if (is_first())
1689 event1_->Signal();
1690 event2_->Wait();
1691 other_channel_.reset(new SyncChannel(
1692 other_channel_name_, Channel::MODE_CLIENT, this,
1693 ipc_thread().message_loop_proxy(), true, shutdown_event()));
1694 other_channel_->SetRestrictDispatchChannelGroup(group_);
1695 if (!is_first()) {
1696 event1_->Signal();
1697 return;
1698 }
1699 *success_ = 0;
1700 int value = 0;
1701 OnPingTTL(3, &value);
1702 *success_ += (value == 3);
1703 OnPingTTL(4, &value);
1704 *success_ += (value == 4);
1705 OnPingTTL(5, &value);
1706 *success_ += (value == 5);
1707 other_channel_->Send(new SyncChannelTestMsg_Done);
1708 other_channel_.reset();
1709 Done();
1710 }
1711
1712 bool is_first() { return !!success_; }
1713
1714 private:
1715 bool OnMessageReceived(const Message& message) {
1716 IPC_BEGIN_MESSAGE_MAP(RestrictedDispatchPipeWorker, message)
1717 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_PingTTL, OnPingTTL)
1718 IPC_MESSAGE_HANDLER(SyncChannelTestMsg_Done, OnDone)
1719 IPC_END_MESSAGE_MAP()
1720 return true;
1721 }
1722
1723 scoped_ptr<SyncChannel> other_channel_;
1724 WaitableEvent* event1_;
1725 WaitableEvent* event2_;
1726 std::string other_channel_name_;
1727 int group_;
1728 int* success_;
1729};
1730
1731} // namespace
1732
1733TEST_F(IPCSyncChannelTest, RestrictedDispatch4WayDeadlock) {
1734 int success = 0;
1735 std::vector<Worker*> workers;
1736 WaitableEvent event0(true, false);
1737 WaitableEvent event1(true, false);
1738 WaitableEvent event2(true, false);
1739 WaitableEvent event3(true, false);
1740 workers.push_back(new RestrictedDispatchPipeWorker(
1741 "channel0", &event0, "channel1", &event1, 1, &success));
1742 workers.push_back(new RestrictedDispatchPipeWorker(
1743 "channel1", &event1, "channel2", &event2, 2, NULL));
1744 workers.push_back(new RestrictedDispatchPipeWorker(
1745 "channel2", &event2, "channel3", &event3, 3, NULL));
1746 workers.push_back(new RestrictedDispatchPipeWorker(
1747 "channel3", &event3, "channel0", &event0, 4, NULL));
1748 RunTest(workers);
1749 EXPECT_EQ(3, success);
1750}
1751
1752//-----------------------------------------------------------------------------
1753
jschuh@chromium.orgf75a4d12012-03-17 11:20:46 +09001754// Generate a validated channel ID using Channel::GenerateVerifiedChannelID().
1755namespace {
1756
1757class VerifiedServer : public Worker {
1758 public:
1759 VerifiedServer(base::Thread* listener_thread,
1760 const std::string& channel_name,
1761 const std::string& reply_text)
1762 : Worker(channel_name, Channel::MODE_SERVER),
1763 reply_text_(reply_text) {
1764 Worker::OverrideThread(listener_thread);
1765 }
1766
1767 virtual void OnNestedTestMsg(Message* reply_msg) {
1768 VLOG(1) << __FUNCTION__ << " Sending reply: " << reply_text_;
1769 SyncChannelNestedTestMsg_String::WriteReplyParams(reply_msg, reply_text_);
1770 Send(reply_msg);
1771 Done();
1772 }
1773
1774 private:
1775 std::string reply_text_;
1776};
1777
1778class VerifiedClient : public Worker {
1779 public:
1780 VerifiedClient(base::Thread* listener_thread,
1781 const std::string& channel_name,
1782 const std::string& expected_text)
1783 : Worker(channel_name, Channel::MODE_CLIENT),
1784 expected_text_(expected_text) {
1785 Worker::OverrideThread(listener_thread);
1786 }
1787
1788 virtual void Run() {
1789 std::string response;
1790 SyncMessage* msg = new SyncChannelNestedTestMsg_String(&response);
1791 bool result = Send(msg);
1792 DCHECK(result);
1793 DCHECK_EQ(response, expected_text_);
1794
1795 VLOG(1) << __FUNCTION__ << " Received reply: " << response;
1796 Done();
1797 }
1798
1799 private:
1800 bool pump_during_send_;
1801 std::string expected_text_;
1802};
1803
1804void Verified() {
1805 std::vector<Worker*> workers;
1806
1807 // A shared worker thread for servers
1808 base::Thread server_worker_thread("Verified_ServerListener");
1809 ASSERT_TRUE(server_worker_thread.Start());
1810
1811 base::Thread client_worker_thread("Verified_ClientListener");
1812 ASSERT_TRUE(client_worker_thread.Start());
1813
1814 std::string channel_id = Channel::GenerateVerifiedChannelID("Verified");
1815 Worker* worker;
1816
1817 worker = new VerifiedServer(&server_worker_thread,
1818 channel_id,
1819 "Got first message");
1820 workers.push_back(worker);
1821
1822 worker = new VerifiedClient(&client_worker_thread,
1823 channel_id,
1824 "Got first message");
1825 workers.push_back(worker);
1826
1827 RunTest(workers);
1828
1829#if defined(OS_WIN)
1830#endif
1831}
1832
1833} // namespace
1834
1835// Windows needs to send an out-of-band secret to verify the client end of the
1836// channel. Test that we still connect correctly in that case.
1837TEST_F(IPCSyncChannelTest, Verified) {
1838 Verified();
1839}
1840
kushi.p@gmail.com5948bf42011-05-14 07:42:41 +09001841} // namespace IPC