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

Unified Diff: components/metrics/leak_detector/call_stack_table_unittest.cc

Issue 2417403002: Revert of Leak reports collect information about the last uptrend (Closed)
Patch Set: Created 4 years, 2 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: components/metrics/leak_detector/call_stack_table_unittest.cc
diff --git a/components/metrics/leak_detector/call_stack_table_unittest.cc b/components/metrics/leak_detector/call_stack_table_unittest.cc
index 253b2bc170e8f3cc473e76c044560ebff91a6cf0..f56f628499e34915851a5344a11c89fedf269287 100644
--- a/components/metrics/leak_detector/call_stack_table_unittest.cc
+++ b/components/metrics/leak_detector/call_stack_table_unittest.cc
@@ -360,99 +360,5 @@
EXPECT_EQ(stack2_, iter->value.call_stack());
}
-TEST_F(CallStackTableTest, GetLastUptrendInfo) {
- CallStackTable table(kDefaultLeakThreshold);
-
- // Add some |stack0_| and |stack1_|.
- for (int i = 0; i < 60; ++i)
- table.Add(stack0_);
- for (int i = 0; i < 60; ++i)
- table.Add(stack1_);
- table.UpdateLastDropInfo(100);
-
- size_t timestamp_delta;
- uint32_t count_delta;
- table.GetLastUptrendInfo(stack0_, 100, &timestamp_delta, &count_delta);
- EXPECT_EQ(0U, timestamp_delta);
- EXPECT_EQ(0U, count_delta);
-
- // Remove |stack0_| and add |stack1_|.
- for (int i = 0; i < 30; ++i)
- table.Remove(stack0_);
- for (int i = 0; i < 30; ++i)
- table.Add(stack1_);
- table.UpdateLastDropInfo(200);
-
- table.GetLastUptrendInfo(stack0_, 200, &timestamp_delta, &count_delta);
- EXPECT_EQ(0U, timestamp_delta);
- EXPECT_EQ(0U, count_delta);
-
- table.GetLastUptrendInfo(stack1_, 200, &timestamp_delta, &count_delta);
- EXPECT_EQ(100U, timestamp_delta);
- EXPECT_EQ(30U, count_delta);
-
- // Check if previous drop of |stack0_| was recorded and introduce |stack2_|.
- for (int i = 0; i < 30; ++i)
- table.Add(stack0_);
- for (int i = 0; i < 60; ++i)
- table.Add(stack2_);
- table.UpdateLastDropInfo(300);
-
- table.GetLastUptrendInfo(stack0_, 300, &timestamp_delta, &count_delta);
- EXPECT_EQ(100U, timestamp_delta);
- EXPECT_EQ(30U, count_delta);
-
- table.GetLastUptrendInfo(stack2_, 300, &timestamp_delta, &count_delta);
- EXPECT_EQ(0U, timestamp_delta);
- EXPECT_EQ(0U, count_delta);
-
- // Introduce more variation between updates. Decrease |stack2_| to 0.
- // All the history for |stack2_| should be forgotten.
- for (int i = 0; i < 30; ++i)
- table.Add(stack0_);
- for (int i = 0; i < 40; ++i)
- table.Remove(stack0_);
- for (int i = 0; i < 40; ++i)
- table.Add(stack1_);
- for (int i = 0; i < 30; ++i)
- table.Remove(stack1_);
- for (int i = 0; i < 30; ++i)
- table.Remove(stack2_);
- for (int i = 0; i < 30; ++i)
- table.Remove(stack2_);
- table.UpdateLastDropInfo(400);
-
- table.GetLastUptrendInfo(stack0_, 400, &timestamp_delta, &count_delta);
- EXPECT_EQ(0U, timestamp_delta);
- EXPECT_EQ(0U, count_delta);
-
- table.GetLastUptrendInfo(stack1_, 400, &timestamp_delta, &count_delta);
- EXPECT_EQ(300U, timestamp_delta);
- EXPECT_EQ(40U, count_delta);
-
- table.GetLastUptrendInfo(stack2_, 400, &timestamp_delta, &count_delta);
- EXPECT_EQ(400U, timestamp_delta);
- EXPECT_EQ(0U, count_delta);
-
- // Make a 0-sum sequence for |stack0_|. Introduce |stack2_| again.
- for (int i = 0; i < 30; ++i)
- table.Add(stack0_);
- for (int i = 0; i < 30; ++i)
- table.Remove(stack0_);
- for (int i = 0; i < 40; ++i)
- table.Add(stack2_);
- for (int i = 0; i < 30; ++i)
- table.Remove(stack2_);
- table.UpdateLastDropInfo(500);
-
- table.GetLastUptrendInfo(stack0_, 500, &timestamp_delta, &count_delta);
- EXPECT_EQ(100U, timestamp_delta);
- EXPECT_EQ(0U, count_delta);
-
- table.GetLastUptrendInfo(stack2_, 500, &timestamp_delta, &count_delta);
- EXPECT_EQ(0U, timestamp_delta);
- EXPECT_EQ(0U, count_delta);
-}
-
} // namespace leak_detector
} // namespace metrics
« no previous file with comments | « components/metrics/leak_detector/call_stack_table.cc ('k') | components/metrics/leak_detector/leak_detector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698