Reland of Move memory-infra check from background tracing and whitelist memory dump args

chrome/common/trace_event_args_whitelist.cc

 // Copyright 2015 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 "chrome/common/trace_event_args_whitelist.h"
 
 #include "base/bind.h"
 #include "base/strings/pattern.h"
 #include "base/strings/string_tokenizer.h"
 #include "base/strings/string_util.h"
+#include "base/trace_event/trace_event.h"
 
 namespace {
 
 struct WhitelistEntry {
   const char* category_name;
   const char* event_name;
   const char* const* arg_name_filter;
 };
 
 const char* const kInputLatencyAllowedArgs[] = {"data", nullptr};
+const char* const kMemoryDumpAllowedArgs[] = {"dumps", nullptr};
 
 const WhitelistEntry kEventArgsWhitelist[] = {
     {"__metadata", "thread_name", nullptr},
     {"ipc", "SyncChannel::Send", nullptr},
     {"toplevel", "*", nullptr},
     {"latencyInfo", "*", kInputLatencyAllowedArgs},
+    // Redefined the string since MemoryDumpManager::kTraceCategory causes
+    // static initialization of this struct.

[Primiano Tucci (use gerrit), 2016/06/23 22:41:18]

^__^ Wow. Definitely didn't expect that.
Whaatever, LGTM

+    {TRACE_DISABLED_BY_DEFAULT("memory-infra"), "*", kMemoryDumpAllowedArgs},
     {nullptr, nullptr, nullptr}};
 
 const char* kMetadataWhitelist[] = {
   "clock-domain",
   "command_line",
   "config",
   "cpu-*",
   "field-trials",
   "gpu-*",
   "highres-ticks",
@@ skipping 47 matching lines @@
 }
 
 bool IsMetadataWhitelisted(const std::string& metadata_name) {
   for (auto key : kMetadataWhitelist) {
     if (base::MatchPattern(metadata_name, key)) {
       return true;
     }
   }
   return false;
 }