Minor cleanup

runtime/vm/thread.h

Index: runtime/vm/thread.h
diff --git a/runtime/vm/thread.h b/runtime/vm/thread.h
index 9bba418a5d6d8998bcfc8619c2256641a25085e6..34e3767e6d0e89f2b5d0c89919d83c631d395eed 100644
--- a/runtime/vm/thread.h
+++ b/runtime/vm/thread.h
@@ -91,25 +91,6 @@ class Zone;
   CACHED_VM_OBJECTS_LIST(V)                                                    \
   CACHED_ADDRESSES_LIST(V)                                                     \
 
-struct InterruptedThreadState {
-  uintptr_t pc;
-  uintptr_t csp;
-  uintptr_t dsp;
-  uintptr_t fp;
-  uintptr_t lr;
-};
-
-// When a thread is interrupted the thread specific interrupt callback will be
-// invoked. Each callback is given an InterruptedThreadState and the user data
-// pointer. When inside a thread interrupt callback doing any of the following
-// is forbidden:
-//   * Accessing TLS -- Because on Windows the callback will be running in a
-//                      different thread.
-//   * Allocating memory -- Because this takes locks which may already be held,
-//                          resulting in a dead lock.
-//   * Taking a lock -- See above.
-typedef void (*ThreadInterruptCallback)(Thread* thread,

[Cutch, 2015/11/06 20:10:15]

Please move the part of the comment about restrictions above
Profiler::SampleThread (profiler.cc) or next to the InterruptedThreadState
definition in thread_interrupter.h

[siva, 2015/11/06 21:59:32]

Done.

-                                        const InterruptedThreadState& state);
 
 // A VM thread; may be executing Dart code or performing helper tasks like
 // garbage collection or compilation. The Thread structure associated with