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

Unified Diff: base/trace_event/process_memory_dump_unittest.cc

Issue 2012883003: [tracing] Set whitelist values for dump provider supporting background mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@background_dump_names
Patch Set: Created 4 years, 6 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: base/trace_event/process_memory_dump_unittest.cc
diff --git a/base/trace_event/process_memory_dump_unittest.cc b/base/trace_event/process_memory_dump_unittest.cc
index e31aeaa2d6b7fd9c5ef7a9f21d53ca76c8719361..2e4c40f4482ae4b92dbd1d6de2a958ec0dc21279 100644
--- a/base/trace_event/process_memory_dump_unittest.cc
+++ b/base/trace_event/process_memory_dump_unittest.cc
@@ -270,9 +270,9 @@ TEST(ProcessMemoryDumpTest, BackgroundModeTest) {
EXPECT_NE(black_hole_mad,
pmd->CreateAllocatorDump("Whitelisted/TestName0xA1b2"));
EXPECT_NE(black_hole_mad,
- pmd->CreateAllocatorDump("Whitelisted/TestName_123"));
+ pmd->CreateAllocatorDump("Whitelisted/TestName123"));
EXPECT_NE(black_hole_mad,
- pmd->CreateAllocatorDump("Whitelisted_12/0xaB/TestName"));
+ pmd->CreateAllocatorDump("Whitelisted12/0xaB/TestName"));
// GetAllocatorDump is consistent.
EXPECT_EQ(black_hole_mad, pmd->GetAllocatorDump("NotWhitelisted/TestName"));

Powered by Google App Engine
This is Rietveld 408576698