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

Unified Diff: third_party/WebKit/Source/wtf/DequeTest.cpp

Issue 2579773002: Use WTF::Deque instead of std::queue in the blink scheduler (Closed)
Patch Set: Apply the fix Sami suggested Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/wtf/DequeTest.cpp
diff --git a/third_party/WebKit/Source/wtf/DequeTest.cpp b/third_party/WebKit/Source/wtf/DequeTest.cpp
index 4305cf25da20673940ff030db475a82d2e42e84a..6751a39805053757aa050efa941bfd4c21ea8bd9 100644
--- a/third_party/WebKit/Source/wtf/DequeTest.cpp
+++ b/third_party/WebKit/Source/wtf/DequeTest.cpp
@@ -603,6 +603,36 @@ TEST(DequeTest, RemoveWhileIterating) {
}
}
+struct Item {
+ Item(int value1, int value2) : value1(value1), value2(value2) {}
+ int value1;
+ int value2;
+};
+
+TEST(DequeTest, emplace_back) {
+ Deque<Item> deque;
+ deque.emplace_back(1, 2);
+ deque.emplace_back(3, 4);
+
+ EXPECT_EQ(2u, deque.size());
+ EXPECT_EQ(1, deque[0].value1);
+ EXPECT_EQ(2, deque[0].value2);
+ EXPECT_EQ(3, deque[1].value1);
+ EXPECT_EQ(4, deque[1].value2);
+}
+
+TEST(DequeTest, emplace_front) {
+ Deque<Item> deque;
+ deque.emplace_front(1, 2);
+ deque.emplace_front(3, 4);
+
+ EXPECT_EQ(2u, deque.size());
+ EXPECT_EQ(3, deque[0].value1);
+ EXPECT_EQ(4, deque[0].value2);
+ EXPECT_EQ(1, deque[1].value1);
+ EXPECT_EQ(2, deque[1].value2);
+}
+
} // anonymous namespace
} // namespace WTF
« third_party/WebKit/Source/wtf/Deque.h ('K') | « third_party/WebKit/Source/wtf/Deque.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698