Make URLRequestContext a MemoryDumpProvider (Abandoned)

net/base/sdch_manager.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/base/sdch_manager.h"
 
 #include <limits.h>
 
 #include <utility>
 
 #include "base/base64url.h"
 #include "base/logging.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
+#include "base/strings/stringprintf.h"
 #include "base/time/default_clock.h"
+#include "base/trace_event/memory_allocator_dump.h"
+#include "base/trace_event/process_memory_dump.h"
 #include "base/values.h"
 #include "crypto/sha2.h"
 #include "net/base/parse_number.h"
 #include "net/base/sdch_net_log_params.h"
 #include "net/base/sdch_observer.h"
 #include "net/url_request/url_request_http_job.h"
 
 namespace {
 
 void StripTrailingDot(GURL* gurl) {
@@ skipping 290 matching lines @@
 }
 
 void SdchManager::AddObserver(SdchObserver* observer) {
   observers_.AddObserver(observer);
 }
 
 void SdchManager::RemoveObserver(SdchObserver* observer) {
   observers_.RemoveObserver(observer);
 }
 
+void SdchManager::DumpMemoryStats(
+    base::trace_event::MemoryAllocatorDump* url_request_context_dump) const {

[ssid, 2016/12/02 21:36:35]

Please pass ProcessMemoryDump* here instead.

+  std::string name = base::StringPrintf("net/sdch_manager_%p", this);
+  base::trace_event::MemoryAllocatorDump* sdch_manager_dump =
+      url_request_context_dump->process_memory_dump()->GetAllocatorDump(name);

[ssid, 2016/12/02 21:36:35]

Please do not use process_memory_dump() method. I am removing this method here:
https://codereview.chromium.org/2545173002

You should be passing ProcessMemoryDump object itself to this method if you want
to change it.

+  // If |this| is shared with another URLRequestContext, add an Ownership Edge
+  // and return early.
+  if (sdch_manager_dump != nullptr) {
+    url_request_context_dump->process_memory_dump()->AddOwnershipEdge(
+        url_request_context_dump->guid(), sdch_manager_dump->guid());
+    return;
+  }
+  // If there are no dictionaries stored, return early without creating a new
+  // MemoryAllocatorDump.
+  size_t total_count = dictionaries_.size();
+  if (total_count == 0)
+    return;
+  sdch_manager_dump =
+      url_request_context_dump->process_memory_dump()->CreateAllocatorDump(
+          name);
+  url_request_context_dump->process_memory_dump()->AddOwnershipEdge(
+      url_request_context_dump->guid(), sdch_manager_dump->guid());
+  size_t total_size = 0;
+  for (const auto& dictionary : dictionaries_) {
+    total_size += dictionary.second->data.text().size();
+  }
+  sdch_manager_dump->AddScalar(
+      base::trace_event::MemoryAllocatorDump::kNameSize,
+      base::trace_event::MemoryAllocatorDump::kUnitsBytes, total_size);
+  sdch_manager_dump->AddScalar(
+      base::trace_event::MemoryAllocatorDump::kNameObjectCount,
+      base::trace_event::MemoryAllocatorDump::kUnitsObjects, total_count);
+}
+
 SdchProblemCode SdchManager::AddSdchDictionary(
     const std::string& dictionary_text,
     const GURL& dictionary_url,
     std::string* server_hash_p) {
   DCHECK(thread_checker_.CalledOnValidThread());
   std::string client_hash;
   std::string server_hash;
   GenerateHash(dictionary_text, &client_hash, &server_hash);
   if (dictionaries_.find(server_hash) != dictionaries_.end())
     return SDCH_DICTIONARY_ALREADY_LOADED;  // Already loaded.
@@ skipping 157 matching lines @@
       entry_dict->SetInteger("tries", it->second.count);
     entry_dict->SetInteger("reason", it->second.reason);
     entry_list->Append(std::move(entry_dict));
   }
   value->Set("blacklisted", std::move(entry_list));
 
   return std::move(value);
 }
 
 }  // namespace net