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

Side by Side Diff: test/cctest/test-lock.cc

Issue 21087012: Simplify implementation of Mutex. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: No need to guard inclusion of errno.h Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2006-2008 the V8 project authors. All rights reserved. 1 // Copyright 2006-2008 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 22 matching lines...) Expand all
33 33
34 #include "platform.h" 34 #include "platform.h"
35 #include "cctest.h" 35 #include "cctest.h"
36 36
37 37
38 using namespace ::v8::internal; 38 using namespace ::v8::internal;
39 39
40 40
41 // Simple test of locking logic 41 // Simple test of locking logic
42 TEST(Simple) { 42 TEST(Simple) {
43 Mutex* mutex = OS::CreateMutex(); 43 Mutex mutex;
44 CHECK_EQ(0, mutex->Lock()); // acquire the lock with the right token 44 mutex.Lock(); // acquire the lock with the right token
45 CHECK_EQ(0, mutex->Unlock()); // can unlock with the right token 45 mutex.Unlock(); // can unlock with the right token
46 delete mutex;
47 }
48
49
50 TEST(MultiLock) {
51 Mutex* mutex = OS::CreateMutex();
52 CHECK_EQ(0, mutex->Lock());
53 CHECK_EQ(0, mutex->Unlock());
54 delete mutex;
55 } 46 }
56 47
57 48
58 TEST(ShallowLock) { 49 TEST(ShallowLock) {
59 Mutex* mutex = OS::CreateMutex(); 50 Mutex mutex;
60 CHECK_EQ(0, mutex->Lock()); 51 mutex.Lock();
61 CHECK_EQ(0, mutex->Unlock()); 52 mutex.Unlock();
62 CHECK_EQ(0, mutex->Lock()); 53 mutex.Lock();
63 CHECK_EQ(0, mutex->Unlock()); 54 mutex.Unlock();
64 delete mutex;
65 } 55 }
66 56
67 57
68 TEST(SemaphoreTimeout) { 58 TEST(SemaphoreTimeout) {
69 bool ok; 59 bool ok;
70 Semaphore* sem = OS::CreateSemaphore(0); 60 Semaphore* sem = OS::CreateSemaphore(0);
71 61
72 // Semaphore not signalled - timeout. 62 // Semaphore not signalled - timeout.
73 ok = sem->Wait(0); 63 ok = sem->Wait(0);
74 CHECK(!ok); 64 CHECK(!ok);
75 ok = sem->Wait(100); 65 ok = sem->Wait(100);
76 CHECK(!ok); 66 CHECK(!ok);
77 ok = sem->Wait(1000); 67 ok = sem->Wait(1000);
78 CHECK(!ok); 68 CHECK(!ok);
79 69
80 // Semaphore signalled - no timeout. 70 // Semaphore signalled - no timeout.
81 sem->Signal(); 71 sem->Signal();
82 ok = sem->Wait(0); 72 ok = sem->Wait(0);
83 sem->Signal(); 73 sem->Signal();
84 ok = sem->Wait(100); 74 ok = sem->Wait(100);
85 sem->Signal(); 75 sem->Signal();
86 ok = sem->Wait(1000); 76 ok = sem->Wait(1000);
87 CHECK(ok); 77 CHECK(ok);
88 delete sem; 78 delete sem;
89 } 79 }
OLDNEW
« src/mutex.h ('K') | « test/cctest/test-debug.cc ('k') | test/cctest/test-platform-linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698