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

Side by Side Diff: base/task_scheduler/sequence_unittest.cc

Issue 2785943004: DCHECK tasks posted by TaskRunner::PostTask (Closed)
Patch Set: s/CHECK/DCHECK/ except for TaskQueueImpl Created 3 years, 8 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/task_scheduler/sequence.h" 5 #include "base/task_scheduler/sequence.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "base/memory/ptr_util.h" 11 #include "base/memory/ptr_util.h"
12 #include "base/test/gtest_util.h" 12 #include "base/test/gtest_util.h"
13 #include "base/time/time.h" 13 #include "base/time/time.h"
14 #include "testing/gtest/include/gtest/gtest.h" 14 #include "testing/gtest/include/gtest/gtest.h"
15 15
16 namespace base { 16 namespace base {
17 namespace internal { 17 namespace internal {
18 18
19 namespace { 19 namespace {
20 20
21 21
22 class TaskSchedulerSequenceTest : public testing::Test { 22 class TaskSchedulerSequenceTest : public testing::Test {
23 public: 23 public:
24 TaskSchedulerSequenceTest() 24 TaskSchedulerSequenceTest()
25 : task_a_owned_( 25 : task_a_owned_(
26 new Task(FROM_HERE, 26 new Task(FROM_HERE,
27 Closure(), 27 Bind([] {}),
28 TaskTraits().WithPriority(TaskPriority::BACKGROUND), 28 TaskTraits().WithPriority(TaskPriority::BACKGROUND),
29 TimeDelta())), 29 TimeDelta())),
30 task_b_owned_( 30 task_b_owned_(
31 new Task(FROM_HERE, 31 new Task(FROM_HERE,
32 Closure(), 32 Bind([] {}),
33 TaskTraits().WithPriority(TaskPriority::USER_VISIBLE), 33 TaskTraits().WithPriority(TaskPriority::USER_VISIBLE),
34 TimeDelta())), 34 TimeDelta())),
35 task_c_owned_( 35 task_c_owned_(
36 new Task(FROM_HERE, 36 new Task(FROM_HERE,
37 Closure(), 37 Bind([] {}),
38 TaskTraits().WithPriority(TaskPriority::USER_BLOCKING), 38 TaskTraits().WithPriority(TaskPriority::USER_BLOCKING),
39 TimeDelta())), 39 TimeDelta())),
40 task_d_owned_( 40 task_d_owned_(
41 new Task(FROM_HERE, 41 new Task(FROM_HERE,
42 Closure(), 42 Bind([] {}),
43 TaskTraits().WithPriority(TaskPriority::USER_BLOCKING), 43 TaskTraits().WithPriority(TaskPriority::USER_BLOCKING),
44 TimeDelta())), 44 TimeDelta())),
45 task_e_owned_( 45 task_e_owned_(
46 new Task(FROM_HERE, 46 new Task(FROM_HERE,
47 Closure(), 47 Bind([] {}),
48 TaskTraits().WithPriority(TaskPriority::BACKGROUND), 48 TaskTraits().WithPriority(TaskPriority::BACKGROUND),
49 TimeDelta())), 49 TimeDelta())),
50 task_a_(task_a_owned_.get()), 50 task_a_(task_a_owned_.get()),
51 task_b_(task_b_owned_.get()), 51 task_b_(task_b_owned_.get()),
52 task_c_(task_c_owned_.get()), 52 task_c_(task_c_owned_.get()),
53 task_d_(task_d_owned_.get()), 53 task_d_(task_d_owned_.get()),
54 task_e_(task_e_owned_.get()) {} 54 task_e_(task_e_owned_.get()) {}
55 55
56 protected: 56 protected:
57 // Tasks to be handed off to a Sequence for testing. 57 // Tasks to be handed off to a Sequence for testing.
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 } 217 }
218 218
219 // Verify that a DCHECK fires if TakeTask() is called on an empty sequence. 219 // Verify that a DCHECK fires if TakeTask() is called on an empty sequence.
220 TEST_F(TaskSchedulerSequenceTest, TakeEmptySequence) { 220 TEST_F(TaskSchedulerSequenceTest, TakeEmptySequence) {
221 scoped_refptr<Sequence> sequence(new Sequence); 221 scoped_refptr<Sequence> sequence(new Sequence);
222 EXPECT_DCHECK_DEATH({ sequence->TakeTask(); }); 222 EXPECT_DCHECK_DEATH({ sequence->TakeTask(); });
223 } 223 }
224 224
225 } // namespace internal 225 } // namespace internal
226 } // namespace base 226 } // namespace base
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698