[Tracing] Use TracingCategoryObserver in runtime statistics

src/isolate.cc

Index: src/isolate.cc
diff --git a/src/isolate.cc b/src/isolate.cc
index cafe546191bba737b06692b79d23808ec40feee5..c1727b470d2343887d81f71cad016070bdcf57b0 100644
--- a/src/isolate.cc
+++ b/src/isolate.cc
@@ -9,6 +9,7 @@
 #include <fstream>  // NOLINT(readability/streams)
 #include <sstream>
 
+#include "include/v8-tracing.h"
 #include "src/ast/context-slot-cache.h"
 #include "src/base/hashmap.h"
 #include "src/base/platform/platform.h"
@@ -2737,8 +2738,8 @@ void Isolate::DumpAndResetCompilationStats() {
   turbo_statistics_ = nullptr;
   delete hstatistics_;
   hstatistics_ = nullptr;
-  if (FLAG_runtime_call_stats &&
-      !TRACE_EVENT_RUNTIME_CALL_STATS_TRACING_ENABLED()) {
+  if (V8_UNLIKELY(FLAG_runtime_stats ==
+                  v8::tracing::TracingCategoryObserver::ENABLED_BY_NATIVE)) {

[alph, 2016/10/31 20:43:54]

Why '==' ?
It seems to me that it should be '&' instead. We still want to print out the
counters even if tracing is enabled.

[lpy, 2016/10/31 20:54:19]

No we don't want to print counters if tracing is enabled, tracing will have a
higher priority, that means when tracing and command line are enabled, we ignore
command line.

     OFStream os(stdout);
     counters()->runtime_call_stats()->Print(os);
     counters()->runtime_call_stats()->Reset();