Fix CompilerStats info

runtime/vm/timer.h

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #ifndef VM_TIMER_H_
 #define VM_TIMER_H_
 
 #include "platform/utils.h"
 #include "vm/allocation.h"
 #include "vm/flags.h"
@@ skipping 107 matching lines @@
   void PrintTimersToJSONProperty(JSONObject* jsobj);
 
  private:
 #define TIMER_FIELD(name, msg) Timer name##_;
   TIMER_LIST(TIMER_FIELD)
 #undef TIMER_FIELD
   bool padding_;
   DISALLOW_COPY_AND_ASSIGN(TimerList);
 };
 
+
 // The class TimerScope is used to start and stop a timer within a scope.
 // It is used as follows:
 // {
 //   TimerScope timer(FLAG_name_of_flag, timer, isolate);
 //   .....
 //   code that needs to be timed.
 //   ....
 // }
 class TimerScope : public StackResource {
  public:
   TimerScope(bool flag, Timer* timer, Isolate* isolate = NULL)
-      : StackResource(isolate), flag_(flag), nested_(false), timer_(timer) {
-    if (flag_) {
+      : StackResource(isolate),
+        nested_(false),
+        timer_(flag ? timer : NULL) {
+    if (timer_ != NULL) {
       if (!timer_->running()) {
         timer_->Start();
       } else {
         nested_ = true;
       }
     }
   }
   ~TimerScope() {
-    if (flag_) {
+    if (timer_ != NULL) {
       if (!nested_) {
         timer_->Stop();
       }
     }
   }
 
  private:
-  const bool flag_;
   bool nested_;
   Timer* const timer_;
 
   DISALLOW_ALLOCATION();
   DISALLOW_COPY_AND_ASSIGN(TimerScope);
 };
 
 
 class PauseTimerScope : public StackResource {
  public:
   PauseTimerScope(bool flag, Timer* timer, Isolate* isolate = NULL)
-      : StackResource(isolate), flag_(flag), nested_(false), timer_(timer) {
-    if (flag_) {
+      : StackResource(isolate),
+        nested_(false),
+        timer_(flag ? timer : NULL) {
+    if (timer_) {
       if (timer_->running()) {
         timer_->Stop();
       } else {
         nested_ = true;
       }
     }
   }
   ~PauseTimerScope() {
-    if (flag_) {
+    if (timer_) {
       if (!nested_) {
         timer_->Start();
       }
     }
   }
 
  private:
-  const bool flag_;
   bool nested_;
   Timer* const timer_;
 
   DISALLOW_ALLOCATION();
   DISALLOW_COPY_AND_ASSIGN(PauseTimerScope);
 };
 
 
 // Macros to deal with named timers in the isolate.
 #define START_TIMER(isolate, name)                                             \
 isolate->timer_list().name().Start();
 
 #define STOP_TIMER(isolate, name)                                              \
 isolate->timer_list().name().Stop();
 
 #define TIMERSCOPE(isolate, name)                                              \
   TimerScope vm_internal_timer_(true, &(isolate->timer_list().name()), isolate)
 
 #define PAUSETIMERSCOPE(isolate, name)                                         \
 PauseTimerScope vm_internal_timer_(true,                                       \
                                    &(isolate->timer_list().name()),            \
                                    isolate)
 
 }  // namespace dart
 
 #endif  // VM_TIMER_H_