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

Side by Side Diff: Source/platform/scheduler/DoubleBufferedDequeTest.cpp

Issue 439923006: Prioritizing input and compositor tasks in the blink scheduler. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: change to pacify clang Created 6 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
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "config.h"
6 #include "platform/scheduler/DoubleBufferedDeque.h"
7
8 #include <gtest/gtest.h>
9
10 namespace {
11
12 using blink::DoubleBufferedDeque;
13
14 typedef testing::Test DoubleBufferedDequeTest;
15
16 TEST(DoubleBufferedDequeTest, TestIsEmpty)
17 {
18 DoubleBufferedDeque<int> queue;
19
20 EXPECT_TRUE(queue.isEmpty());
21 queue.append(1);
22 EXPECT_FALSE(queue.isEmpty());
23 }
24
25 TEST(DoubleBufferedDequeTest, TestIsEmptyAfterSwapBuffers)
26 {
27 DoubleBufferedDeque<int> queue;
28 queue.append(1);
29
30 queue.swapBuffers();
31 EXPECT_TRUE(queue.isEmpty());
32
33 queue.swapBuffers();
34 EXPECT_FALSE(queue.isEmpty());
35 }
36
37 TEST(DoubleBufferedDequeTest, TestDoubleBuffering)
38 {
39 DoubleBufferedDeque<int> queue;
Sami 2014/08/08 15:31:13 It's a minor thing but could you rewrite this test
alexclarke 2014/08/08 15:47:45 Done.
40 queue.append(1);
41
42 WTF::Deque<int>& firstBuffer = queue.swapBuffers();
43 queue.append(2);
44
45 WTF::Deque<int>& secondBuffer = queue.swapBuffers();
46
47 EXPECT_EQ(1, firstBuffer.takeFirst());
48 EXPECT_EQ(2, secondBuffer.takeFirst());
49
50 EXPECT_EQ(&firstBuffer, &queue.swapBuffers());
51 EXPECT_EQ(&secondBuffer, &queue.swapBuffers());
52 }
53
54 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698