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

Side by Side Diff: chrome/common/trace_event_args_whitelist.cc

Issue 2092463002: Move memory-infra check from background tracing and whitelist memory dump args (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@enabled_modes
Patch Set: Created 4 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/common/trace_event_args_whitelist.h" 5 #include "chrome/common/trace_event_args_whitelist.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/strings/pattern.h" 8 #include "base/strings/pattern.h"
9 #include "base/strings/string_tokenizer.h" 9 #include "base/strings/string_tokenizer.h"
10 #include "base/strings/string_util.h" 10 #include "base/strings/string_util.h"
11 #include "base/trace_event/memory_dump_manager.h"
11 12
12 namespace { 13 namespace {
13 14
14 struct WhitelistEntry { 15 struct WhitelistEntry {
15 const char* category_name; 16 const char* category_name;
16 const char* event_name; 17 const char* event_name;
17 const char* const* arg_name_filter; 18 const char* const* arg_name_filter;
18 }; 19 };
19 20
20 const char* const kInputLatencyAllowedArgs[] = {"data", nullptr}; 21 const char* const kInputLatencyAllowedArgs[] = {"data", nullptr};
22 const char* const kMemoryDumpAllowedArgs[] = {"dumps", nullptr};
21 23
22 const WhitelistEntry kEventArgsWhitelist[] = { 24 const WhitelistEntry kEventArgsWhitelist[] = {
23 {"__metadata", "thread_name", nullptr}, 25 {"__metadata", "thread_name", nullptr},
24 {"ipc", "SyncChannel::Send", nullptr}, 26 {"ipc", "SyncChannel::Send", nullptr},
25 {"toplevel", "*", nullptr}, 27 {"toplevel", "*", nullptr},
26 {"latencyInfo", "*", kInputLatencyAllowedArgs}, 28 {"latencyInfo", "*", kInputLatencyAllowedArgs},
29 {base::trace_event::MemoryDumpManager::kTraceCategory, "*",
30 kMemoryDumpAllowedArgs},
27 {nullptr, nullptr, nullptr}}; 31 {nullptr, nullptr, nullptr}};
28 32
29 const char* kMetadataWhitelist[] = { 33 const char* kMetadataWhitelist[] = {
30 "clock-domain", 34 "clock-domain",
31 "command_line", 35 "command_line",
32 "config", 36 "config",
33 "cpu-*", 37 "cpu-*",
34 "field-trials", 38 "field-trials",
35 "gpu-*", 39 "gpu-*",
36 "highres-ticks", 40 "highres-ticks",
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 } 88 }
85 89
86 bool IsMetadataWhitelisted(const std::string& metadata_name) { 90 bool IsMetadataWhitelisted(const std::string& metadata_name) {
87 for (auto key : kMetadataWhitelist) { 91 for (auto key : kMetadataWhitelist) {
88 if (base::MatchPattern(metadata_name, key)) { 92 if (base::MatchPattern(metadata_name, key)) {
89 return true; 93 return true;
90 } 94 }
91 } 95 }
92 return false; 96 return false;
93 } 97 }
OLDNEW
« no previous file with comments | « base/trace_event/memory_dump_manager.cc ('k') | content/browser/tracing/background_tracing_manager_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698