[vm] Revert changes related to triggering the 1st GC

runtime/vm/isolate.h

Index: runtime/vm/isolate.h
diff --git a/runtime/vm/isolate.h b/runtime/vm/isolate.h
index 62572410ee3a6bd156e7fd0fd7705c346fbb7e55..6a8c5feb389a142da075db119da86d065a6eb31b 100644
--- a/runtime/vm/isolate.h
+++ b/runtime/vm/isolate.h
@@ -227,9 +227,10 @@ class Isolate : public BaseIsolate {
 #if !defined(PRODUCT)
   const char* debugger_name() const { return debugger_name_; }
   void set_debugger_name(const char* name);
-  int64_t UptimeMicros() const;
 #endif  // !defined(PRODUCT)
 
+  int64_t UptimeMicros() const;
+
   Dart_Port main_port() const { return main_port_; }
   void set_main_port(Dart_Port port) {
     ASSERT(main_port_ == 0);  // Only set main port once.
@@ -780,6 +781,7 @@ class Isolate : public BaseIsolate {
   bool compilation_allowed_;
   bool all_classes_finalized_;
   bool remapping_cids_;
+  int64_t start_time_micros_;

[zra, 2017/08/04 20:08:28]

In non-product builds, this will break up the bool fields. Could you move this
instead down to where I've noted below?

[alexmarkov, 2017/08/04 20:28:06]

Done.

 
 // Fields that aren't needed in a product build go here with boolean flags at
 // the top.
@@ -790,7 +792,6 @@ class Isolate : public BaseIsolate {
   bool should_pause_post_service_request_;
 
   char* debugger_name_;
-  int64_t start_time_micros_;
   Debugger* debugger_;
   int64_t last_resume_timestamp_;