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

Unified Diff: runtime/bin/eventhandler_test.cc

Issue 2974233002: VM: Re-format to use at most one newline between functions (Closed)
Patch Set: Rebase and merge Created 3 years, 5 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
« no previous file with comments | « runtime/bin/eventhandler_macos.cc ('k') | runtime/bin/eventhandler_unsupported.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/bin/eventhandler_test.cc
diff --git a/runtime/bin/eventhandler_test.cc b/runtime/bin/eventhandler_test.cc
index cef2b94b9fadb06ff064a6babbbad25b133d1ada..76880634e6ec176ba14088e7b0bf49f49179c8ad 100644
--- a/runtime/bin/eventhandler_test.cc
+++ b/runtime/bin/eventhandler_test.cc
@@ -18,14 +18,12 @@ VM_UNIT_TEST_CASE(CircularLinkedList) {
EXPECT(list.HasHead());
EXPECT(list.head() == 1);
-
// Test: Inserts don't move head.
for (int i = 2; i <= 100; i++) {
list.Add(i);
EXPECT(list.head() == 1);
}
-
// Test: Rotate cycle through all elements in insertion order.
for (int i = 1; i <= 100; i++) {
EXPECT(list.HasHead());
@@ -33,7 +31,6 @@ VM_UNIT_TEST_CASE(CircularLinkedList) {
list.Rotate();
}
-
// Test: Removing head results in next element to be head.
for (int i = 1; i <= 100; i++) {
list.RemoveHead();
@@ -47,7 +44,6 @@ VM_UNIT_TEST_CASE(CircularLinkedList) {
// Test: Removing all items individually make list empty.
EXPECT(!list.HasHead());
-
// Test: Removing all items at once makes list empty.
for (int i = 1; i <= 100; i++) {
list.Add(i);
@@ -55,7 +51,6 @@ VM_UNIT_TEST_CASE(CircularLinkedList) {
list.RemoveAll();
EXPECT(!list.HasHead());
-
// Test: Remove individual items just deletes them without modifying head.
for (int i = 1; i <= 10; i++) {
list.Add(i);
@@ -69,12 +64,10 @@ VM_UNIT_TEST_CASE(CircularLinkedList) {
list.Rotate();
EXPECT(list.head() == 1);
-
// Test: Remove non-existent element leaves list un-changed.
list.Remove(4242);
EXPECT(list.head() == 1);
-
// Test: Remove head element individually moves head to next element.
list.Remove(1);
EXPECT(list.HasHead());
@@ -82,7 +75,6 @@ VM_UNIT_TEST_CASE(CircularLinkedList) {
list.Remove(10);
EXPECT(!list.HasHead());
-
// Test: Remove non-existent element from empty list works.
list.Remove(4242);
}
« no previous file with comments | « runtime/bin/eventhandler_macos.cc ('k') | runtime/bin/eventhandler_unsupported.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698