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

Unified Diff: net/http/http_cache_unittest.cc

Issue 2772283003: Make url_request_context dumps include context names (Closed)
Patch Set: Revert back to an explicit list Created 3 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
« no previous file with comments | « net/base/sdch_manager_unittest.cc ('k') | net/url_request/url_request_context.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_cache_unittest.cc
diff --git a/net/http/http_cache_unittest.cc b/net/http/http_cache_unittest.cc
index 27222a905449abe15cd263149bb25c373a576a89..8fb10d15409c8211dc5c4d23da8472911ebedc18 100644
--- a/net/http/http_cache_unittest.cc
+++ b/net/http/http_cache_unittest.cc
@@ -8386,13 +8386,14 @@ TEST_P(HttpCacheMemoryDumpTest, DumpMemoryStats) {
std::unique_ptr<base::trace_event::ProcessMemoryDump> process_memory_dump(
new base::trace_event::ProcessMemoryDump(nullptr, dump_args));
base::trace_event::MemoryAllocatorDump* parent_dump =
- process_memory_dump->CreateAllocatorDump("net/url_request_context_0x123");
+ process_memory_dump->CreateAllocatorDump(
+ "net/url_request_context/main/0x123");
cache.http_cache()->DumpMemoryStats(process_memory_dump.get(),
parent_dump->absolute_name());
const base::trace_event::MemoryAllocatorDump* dump =
process_memory_dump->GetAllocatorDump(
- "net/url_request_context_0x123/http_cache");
+ "net/url_request_context/main/0x123/http_cache");
ASSERT_NE(nullptr, dump);
std::unique_ptr<base::Value> raw_attrs =
dump->attributes_for_testing()->ToBaseValue();
« no previous file with comments | « net/base/sdch_manager_unittest.cc ('k') | net/url_request/url_request_context.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698