OLD | NEW |
---|---|
1 // Copyright 2013 The Chromium Authors. All rights reserved. | 1 // Copyright 2013 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "base/barrier_closure.h" | 5 #include "base/barrier_closure.h" |
6 | 6 |
7 #include "base/bind.h" | 7 #include "base/bind.h" |
8 #include "testing/gtest/include/gtest/gtest.h" | 8 #include "testing/gtest/include/gtest/gtest.h" |
9 | 9 |
10 namespace { | 10 namespace { |
11 | 11 |
12 void Increment(int* count) { (*count)++; } | 12 void Increment(int* count) { (*count)++; } |
13 | 13 |
14 TEST(BarrierClosureTest, RunImmediatelyForZeroClosures) { | 14 TEST(BarrierClosureTest, RunImmediatelyForZeroClosures) { |
15 int count = 0; | 15 int count = 0; |
16 base::Closure doneClosure(base::Bind(&Increment, base::Unretained(&count))); | 16 base::Closure done_closure(base::Bind(&Increment, base::Unretained(&count))); |
17 | 17 |
18 base::Closure barrierClosure = base::BarrierClosure(0, doneClosure); | 18 base::Closure barrier_closure = base::BarrierClosure(0, done_closure); |
19 EXPECT_EQ(1, count); | 19 EXPECT_EQ(1, count); |
20 } | 20 } |
21 | 21 |
22 TEST(BarrierClosureTest, RunAfterNumClosures) { | 22 TEST(BarrierClosureTest, RunAfterNumClosures) { |
23 int count = 0; | 23 int count = 0; |
24 base::Closure doneClosure(base::Bind(&Increment, base::Unretained(&count))); | 24 base::Closure done_closure(base::Bind(&Increment, base::Unretained(&count))); |
25 | 25 |
26 base::Closure barrierClosure = base::BarrierClosure(2, doneClosure); | 26 base::Closure barier_closure = base::BarrierClosure(2, done_closure); |
danakj
2015/04/30 17:32:55
barrier
dzhioev (left Google)
2015/04/30 20:00:54
Done.
| |
27 EXPECT_EQ(0, count); | 27 EXPECT_EQ(0, count); |
28 | 28 |
29 barrierClosure.Run(); | 29 barier_closure.Run(); |
30 EXPECT_EQ(0, count); | 30 EXPECT_EQ(0, count); |
31 | 31 |
32 barrierClosure.Run(); | 32 barier_closure.Run(); |
33 EXPECT_EQ(1, count); | 33 EXPECT_EQ(1, count); |
34 } | 34 } |
35 | 35 |
36 void ReleaseClosure(base::Closure* closure) { | |
37 *closure = base::Closure(); | |
38 } | |
39 | |
40 // Tests a case when client releases a reference to BarierClosure in | |
41 // |done_closure|. | |
42 TEST(BarrierClosureTest, KeepingClosureAliveUntilDone) { | |
43 base::Closure barier_closure; | |
44 base::Closure done_closure = base::Bind(ReleaseClosure, &barier_closure); | |
45 barier_closure = base::BarrierClosure(1, done_closure); | |
danakj
2015/04/30 17:32:55
BarrierClosure makes a copy of done_closure in the
dzhioev (left Google)
2015/04/30 20:00:54
I change |barrier_closure|, not |done_closure|.
De
| |
46 barier_closure.Run(); | |
47 } | |
48 | |
36 } // namespace | 49 } // namespace |
OLD | NEW |