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

Unified Diff: tools/gn/trace.cc

Issue 265693003: Redo GN "args" command (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: tools/gn/trace.cc
diff --git a/tools/gn/trace.cc b/tools/gn/trace.cc
index acd735df365cef7707c9cf18c3efc6e29d5d1328..4dfb6ca2c07a3965fb487ade02da1ecf8df02162 100644
--- a/tools/gn/trace.cc
+++ b/tools/gn/trace.cc
@@ -167,8 +167,8 @@ void ScopedTrace::Done() {
}
void EnableTracing() {
- CHECK(!trace_log);
- trace_log = new TraceLog;
+ if (!trace_log)
+ trace_log = new TraceLog;
}
void AddTrace(TraceItem* item) {
@@ -204,6 +204,7 @@ std::string SummarizeTraces() {
case TraceItem::TRACE_CHECK_HEADER:
headers_checked++;
break;
+ case TraceItem::TRACE_SETUP:
case TraceItem::TRACE_FILE_LOAD:
case TraceItem::TRACE_FILE_WRITE:
case TraceItem::TRACE_DEFINE_TARGET:
@@ -265,6 +266,9 @@ void SaveTraces(const base::FilePath& file_name) {
out << ",\"cat\":";
switch (item.type()) {
+ case TraceItem::TRACE_SETUP:
+ out << "\"setup\"";
+ break;
case TraceItem::TRACE_FILE_LOAD:
out << "\"load\"";
break;
« tools/gn/command_args.cc ('K') | « tools/gn/trace.h ('k') | tools/gn/value.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698