MemoryInfra: Report all provider metrics to UKM

Most of the metrics were added to UKM in:
https://chromium-review.googlesource.com/c/chromium/src/+/939904
Some metrics were missed due to privacy review being stale.
This CL adds the remaining metrics after updating privacy doc

BUG=730783

Change-Id: Iec4cc7d6cc4f90c7fb57a9bd8cad23eef7e3dce9
Reviewed-on: https://chromium-review.googlesource.com/961538Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Commit-Queue: Siddhartha S <ssid@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543497}

base/trace_event/memory_infra_background_whitelist.cc

@@ -61,15 +61,21 @@ const char* const kDumpProviderWhitelist[] = {
 // dump. So, the effective size will not be correct.
 const char* const kDumpProviderSummaryWhitelist[] = {
     "android::ResourceManagerImpl",
+    "AutocompleteController",
     "BlinkGC",
     "BlinkObjectCounters",
+    "BlobStorageContext",
     "ClientDiscardableSharedMemoryManager",
+    "DOMStorage",
+    "DownloadService",
     "DiscardableSharedMemoryManager",
     "gpu::BufferManager",
     "gpu::RenderbufferManager",
     "gpu::TextureManager",
+    "FontCaches",
     "HistoryReport",
     "IndexedDBBackingStore",
+    "InMemoryURLIndex",
     "JavaHeap",
     "LevelDB",
     "LeveldbValueStore",
@@ -237,8 +243,6 @@ const char* const kAllocatorDumpNameWhitelist[] = {
     "v8/isolate_0x?/heap_spaces/other_spaces",
     "v8/isolate_0x?/malloc",
     "v8/isolate_0x?/zapped_for_debug",
-    "winheap",
-    "winheap/allocated_objects",
     "site_storage/blob_storage/0x?",
     "site_storage/index_db/0x?",
     "site_storage/localstorage/0x?/cache_size",

chrome/browser/metrics/process_memory_metrics_emitter.cc

@@ -57,10 +57,14 @@ const Metric kAllocatorDumpNamesForMetrics[] = {
     {"blink_objects/Node", "NumberOfNodes",
      base::trace_event::MemoryAllocatorDump::kNameObjectCount,
      &Memory_Experimental::SetNumberOfNodes},
+    {"components/download", "DownloadService", kEffectiveSize,
+     &Memory_Experimental::SetDownloadService},
     {"discardable", "Discardable", kEffectiveSize,
      &Memory_Experimental::SetDiscardable},
     {"extensions/value_store", "Extensions.ValueStore", kEffectiveSize,
      &Memory_Experimental::SetExtensions_ValueStore},
+    {"font_caches", "FontCaches", kEffectiveSize,
+     &Memory_Experimental::SetFontCaches},
     {"gpu/gl", "CommandBuffer", kEffectiveSize,
      &Memory_Experimental::SetCommandBuffer},
     {"history", "History", kEffectiveSize, &Memory_Experimental::SetHistory},
@@ -75,6 +79,8 @@ const Metric kAllocatorDumpNamesForMetrics[] = {
     {"net", "Net", kEffectiveSize, &Memory_Experimental::SetNet},
     {"net/url_request_context", "Net.UrlRequestContext", kEffectiveSize,
      &Memory_Experimental::SetNet_UrlRequestContext},
+    {"omnibox", "OmniboxSuggestions", kEffectiveSize,
+     &Memory_Experimental::SetOmniboxSuggestions},
     {"partition_alloc", "PartitionAlloc", kEffectiveSize,
      &Memory_Experimental::SetPartitionAlloc},
     {"partition_alloc/allocated_objects", "PartitionAlloc.AllocatedObjects",
@@ -91,9 +97,10 @@ const Metric kAllocatorDumpNamesForMetrics[] = {
      kEffectiveSize, &Memory_Experimental::SetPartitionAlloc_Partitions_Layout},
     {"site_storage", "SiteStorage", kEffectiveSize,
      &Memory_Experimental::SetSiteStorage},
+    {"site_storage/blob_storage", "SiteStorage.BlobStorage", kEffectiveSize,
+     &Memory_Experimental::SetSiteStorage_BlobStorage},
     {"site_storage/index_db", "SiteStorage.IndexDB", kEffectiveSize,
      &Memory_Experimental::SetSiteStorage_IndexDB},
-    // TODO(ssid): This metric does not return total value.
     {"site_storage/localstorage", "SiteStorage.LocalStorage", kEffectiveSize,
      &Memory_Experimental::SetSiteStorage_LocalStorage},
     {"site_storage/session_storage", "SiteStorage.SessionStorage",

tools/metrics/ukm/ukm.xml

@@ -1461,11 +1461,22 @@ be describing additional metrics about the same event.
       Measure of memory consumed by Discardable memory service.
     </summary>
   </metric>
+  <metric name="DownloadService">
+    <summary>
+      Measure of memory used by download service.
+    </summary>
+  </metric>
   <metric name="Extensions.ValueStore">
     <summary>
       Measure of memory consumed by Key Value Store databases of extensions.
     </summary>
   </metric>
+  <metric name="FontCaches">
+    <summary>
+      Measure of memory used by font platform and shape caches in renderer
+      process.
+    </summary>
+  </metric>
   <metric name="History">
     <summary>
       Approximate measure of memory consumed by History service.
@@ -1534,6 +1545,11 @@ be describing additional metrics about the same event.
       The number of nodes that the associated renderer owns.
     </summary>
   </metric>
+  <metric name="OmniboxSuggestions">
+    <summary>
+      Measure of memory used due to URL indexing and autocomplete suggestions.
+    </summary>
+  </metric>
   <metric name="PartitionAlloc">
     <summary>
       Measure of memory allocated by PartitionAlloc allocator.
@@ -1597,6 +1613,11 @@ be describing additional metrics about the same event.
       Measure of memory used due to web storage APIs in browser process.
     </summary>
   </metric>
+  <metric name="SiteStorage.BlobStorage">
+    <summary>
+      Measure of memory used by in-memory blob file API in browser process.
+    </summary>
+  </metric>
   <metric name="SiteStorage.IndexDB">
     <summary>
       Measure of memory used due to IndexedDB API in browser process.