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

Unified Diff: cc/base/tiling_data_unittest.cc

Issue 13206004: cc: Fix build issues for adding ‘chromium_code’: 1 to cc.gyp and cc_tests.gyp (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 9 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: cc/base/tiling_data_unittest.cc
diff --git a/cc/base/tiling_data_unittest.cc b/cc/base/tiling_data_unittest.cc
index ea141addc4b57b45d9fc22f718c4e13242595563..6b16609aece9f2838f421ebf3713e49d18096e63 100644
--- a/cc/base/tiling_data_unittest.cc
+++ b/cc/base/tiling_data_unittest.cc
@@ -954,7 +954,7 @@ void TestIterate(
}
EXPECT_TRUE(found);
}
- EXPECT_EQ(0, expected.size());
+ EXPECT_TRUE(expected.empty());
}
// Make sure this also works with a difference iterator and an empty ignore.
@@ -973,7 +973,7 @@ void TestIterate(
}
EXPECT_TRUE(found);
}
- EXPECT_EQ(0, expected.size());
+ EXPECT_TRUE(expected.empty());
}
}
@@ -1063,7 +1063,7 @@ void TestDiff(
}
// Sanity check the test.
- EXPECT_EQ(num_tiles, expected.size());
+ EXPECT_EQ(static_cast<size_t>(num_tiles), expected.size());
jamesr 2013/03/29 00:07:16 num_tiles is a size_t, according to the function s
danakj 2013/03/29 00:17:35 Oh.. hm.. it's not. removed.
for (TilingData::DifferenceIterator iter(&data, consider, ignore);
iter; ++iter) {
@@ -1078,7 +1078,7 @@ void TestDiff(
}
EXPECT_TRUE(found);
}
- EXPECT_EQ(0, expected.size());
+ EXPECT_TRUE(expected.empty());
}
TEST(TilingDataTest, DifferenceIteratorIgnoreGeometry) {

Powered by Google App Engine
This is Rietveld 408576698