Remove profiler signal handler on shutdown

runtime/vm/thread_interrupter.cc

 // Copyright (c) 2013, 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.
 
 #include "vm/thread_interrupter.h"
 
 #include "vm/flags.h"
 #include "vm/lockers.h"
 #include "vm/os.h"
 #include "vm/simulator.h"
@@ skipping 296 matching lines @@
         // No isolates were profiled. In order to reduce unnecessary CPU
         // load, we will wait until we are notified before attempting to
         // interrupt again.
         current_wait_time_ = Monitor::kNoTimeout;
         continue;
       }
 
       ASSERT(current_wait_time_ != Monitor::kNoTimeout);
     }
   }
+  RemoveSignalHandler();

[siva, 2015/04/07 18:18:07]

Instead of a per OS RemoveSignalHandler() implementation why not just call
SignalHandler::Remove() here?

[Cutch, 2015/04/07 18:35:55]

SignalHandler::Remove() on Windows is UNREACHABLE().

   if (FLAG_trace_thread_interrupter) {
     OS::Print("ThreadInterrupter thread exiting.\n");
   }
   {
     // Signal to main thread we are exiting.
     MonitorLocker shutdown_ml(monitor_);
     thread_running_ = false;
     shutdown_ml.Notify();
   }
 }
 
 }  // namespace dart