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

Unified Diff: src/processor/map_serializers_unittest.cc

Issue 2029953003: Adding support for overlapping ranges to RangeMap. (Closed) Base URL: https://chromium.googlesource.com/breakpad/breakpad.git@master
Patch Set: Created 4 years, 7 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: src/processor/map_serializers_unittest.cc
diff --git a/src/processor/map_serializers_unittest.cc b/src/processor/map_serializers_unittest.cc
index 0d872ec2e21a6a2267284bf616508c4ad4dc1fe5..b1a9e5224de15b8b0ac7e459281f883466b2f63e 100644
--- a/src/processor/map_serializers_unittest.cc
+++ b/src/processor/map_serializers_unittest.cc
@@ -237,7 +237,7 @@ TEST_F(TestRangeMapSerializer, MapWithOneRangeTestCase) {
};
uint32_t correct_size = sizeof(correct_data);
- range_map_.StoreRange(1, 10, 6);
+ range_map_.StoreRange(1, 0 /* delta */, 10, 6);
serialized_data_ = serializer_.Serialize(range_map_, &serialized_size_);
@@ -258,9 +258,9 @@ TEST_F(TestRangeMapSerializer, MapWithThreeRangesTestCase) {
};
uint32_t correct_size = sizeof(correct_data);
- ASSERT_TRUE(range_map_.StoreRange(2, 4, 1));
- ASSERT_TRUE(range_map_.StoreRange(6, 4, 2));
- ASSERT_TRUE(range_map_.StoreRange(10, 11, 3));
+ ASSERT_TRUE(range_map_.StoreRange(2, 0 /* delta */, 4, 1));
+ ASSERT_TRUE(range_map_.StoreRange(6, 0 /* delta */, 4, 2));
+ ASSERT_TRUE(range_map_.StoreRange(10, 0 /* delta */, 11, 3));
serialized_data_ = serializer_.Serialize(range_map_, &serialized_size_);

Powered by Google App Engine
This is Rietveld 408576698