Add IsMutatorThread to the Thread class and use it instead of MutatorThreadIsCurrentThread

runtime/vm/debugger.h

 // Copyright (c) 2012, 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_DEBUGGER_H_
 #define VM_DEBUGGER_H_
 
 #include "include/dart_tools_api.h"
 
 #include "vm/object.h"
@@ skipping 633 matching lines @@
                             const String& prefix,
                             bool include_private_fields);
 
   // Handles any events which pause vm execution.  Breakpoints,
   // interrupts, etc.
   void Pause(DebuggerEvent* event);
 
   void HandleSteppingRequest(DebuggerStackTrace* stack_trace);
 
   Zone* zone() const {
-    ASSERT(isolate_->MutatorThreadIsCurrentThread());

[srdjan, 2015/10/26 19:21:41]

Why did you remove this assert? We do not want to access wrong zone. Maybe:
ASSERT(Thread::Current()->IsMutator...

[siva, 2015/10/26 19:56:21]

current_zone has the same assert
(Thread::Current() == mutator_thread_);

     return isolate_->current_zone();
   }
 
   Isolate* isolate_;
   Dart_Port isolate_id_;  // A unique ID for the isolate in the debugger.
   bool initialized_;
 
   // ID number generator.
   intptr_t next_id_;
 
@@ skipping 32 matching lines @@
 
   friend class Isolate;
   friend class BreakpointLocation;
   DISALLOW_COPY_AND_ASSIGN(Debugger);
 };
 
 
 }  // namespace dart
 
 #endif  // VM_DEBUGGER_H_