Call known TLS destructors on Windows

Call known TLS destructors on Windows

Partially fixes #23474 by freeing the MonitorWaitData and Thread objects on Windows when a thread exits.

Machinery to run a function at thread exit on Windows taken from Chromium's base.

R=asiva@google.com

Committed: https://github.com/dart-lang/sdk/commit/1ec9d78b89303a7a45dc99db8e886be77005bf3b

[Cutch, 2015-10-13 17:39:51]

johnmccutchan@google.com changed reviewers:
+ asiva@google.com

[siva, 2015-10-13 21:30:54]

lgtm with one comment about moving Thread::Cleanup entirely into
os_thread_win.cc and avoiding the ugly #ifdef OS_WINDOWS in the common file.

https://codereview.chromium.org/1391013006/diff/1/runtime/vm/thread.cc
File runtime/vm/thread.cc (right):

https://codereview.chromium.org/1391013006/diff/1/runtime/vm/thread.cc#newcod...
runtime/vm/thread.cc:536: }
Ditto comment.

https://codereview.chromium.org/1391013006/diff/1/runtime/vm/thread.h
File runtime/vm/thread.h (right):

https://codereview.chromium.org/1391013006/diff/1/runtime/vm/thread.h#newcode518
runtime/vm/thread.h:518: #endif
Not sure why this needs to be here in this common file as opposed to
os_thread_win.h

Same comment for Thread::Cleanup it is not needed here can be done entirely in
os_thread_win.cc

[Cutch, 2015-10-14 14:43:18]

https://codereview.chromium.org/1391013006/diff/1/runtime/vm/thread.cc
File runtime/vm/thread.cc (right):

https://codereview.chromium.org/1391013006/diff/1/runtime/vm/thread.cc#newcod...
runtime/vm/thread.cc:536: }
On 2015/10/13 21:30:54, siva wrote:
> Ditto comment.

Acknowledged.

https://codereview.chromium.org/1391013006/diff/1/runtime/vm/thread.h
File runtime/vm/thread.h (right):

https://codereview.chromium.org/1391013006/diff/1/runtime/vm/thread.h#newcode518
runtime/vm/thread.h:518: #endif
On 2015/10/13 21:30:54, siva wrote:
> Not sure why this needs to be here in this common file as opposed to
> os_thread_win.h
> 
> Same comment for Thread::Cleanup it is not needed here can be done entirely in
> os_thread_win.cc

This is structured like this because the Thread::Cleanup code needs the Thread
class and its destructor. This isn't available in os_thread_win.*

I've added lots of comments.

[Cutch, 2015-10-14 14:59:38]

Committed patchset #2 (id:20001) manually as
1ec9d78b89303a7a45dc99db8e886be77005bf3b (presubmit successful).