[memory-infra] Whitelist TabRestoreService MDP.

base/trace_event/memory_infra_background_whitelist.cc

 // Copyright 2016 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 "base/trace_event/memory_infra_background_whitelist.h"
 
 #include <ctype.h>
 #include <string.h>
 
 #include <string>
@@ skipping 17 matching lines @@
     "LeveldbValueStore",
     "Malloc",
     "MemoryCache",
     "PartitionAlloc",
     "ProcessMemoryMetrics",
     "Skia",
     "Sql",
     "V8Isolate",
     "WinHeap",
     "SyncDirectory",
+    "TabRestoreServiceHelper",

[ssid, 2017/02/07 18:58:59]

TabRestore service creates dumps even if there are no entries. Can we fix that
before enabling on slow reports? We might have a column without size and I don't
know what the memory mapper does in this case and might show weird results in
dashboard.

[DmitrySkiba, 2017/02/07 23:17:04]

Done.

     nullptr  // End of list marker.
 };
 
 // A list of string names that are allowed for the memory allocator dumps in
 // background mode.
 const char* const kAllocatorDumpNameWhitelist[] = {
     "blink_gc",
     "blink_gc/allocated_objects",
     "discardable",
     "discardable/child_0x?",
@@ skipping 69 matching lines @@
     "sync/0x?/model_type/PRIORITY_PREFERENCE",
     "sync/0x?/model_type/READING_LIST",
     "sync/0x?/model_type/SEARCH_ENGINE",
     "sync/0x?/model_type/SESSION",
     "sync/0x?/model_type/SYNCED_NOTIFICATION",
     "sync/0x?/model_type/SYNCED_NOTIFICATION_APP_INFO",
     "sync/0x?/model_type/THEME",
     "sync/0x?/model_type/TYPED_URL",
     "sync/0x?/model_type/WALLET_METADATA",
     "sync/0x?/model_type/WIFI_CREDENTIAL",
+    "tab_restore/service_helper_0x?/entries/tab_0x?",
+    "tab_restore/service_helper_0x?/entries/window_0x?",

[ssid, 2017/02/07 18:58:59]

Is there a maximum number of service helpers and entries?

[DmitrySkiba, 2017/02/07 23:17:04]

Maximum value is 25, see TabRestoreServiceHelper::kMaxEntries

     nullptr  // End of list marker.
 };
 
 const char* const* g_dump_provider_whitelist = kDumpProviderWhitelist;
 const char* const* g_allocator_dump_name_whitelist =
     kAllocatorDumpNameWhitelist;
 
 }  // namespace
 
 bool IsMemoryDumpProviderWhitelisted(const char* mdp_name) {
@@ skipping 35 matching lines @@
 void SetDumpProviderWhitelistForTesting(const char* const* list) {
   g_dump_provider_whitelist = list;
 }
 
 void SetAllocatorDumpNameWhitelistForTesting(const char* const* list) {
   g_allocator_dump_name_whitelist = list;
 }
 
 }  // namespace trace_event
 }  // namespace base