Change the thread interface in runtime/platform and use it starting all threads

runtime/platform/thread.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 PLATFORM_THREAD_H_
 #define PLATFORM_THREAD_H_
 
 #include "platform/globals.h"
 
 // Declare the OS-specific types ahead of defining the generic classes.
 #if defined(TARGET_OS_LINUX)
 #include "platform/thread_linux.h"
 #elif defined(TARGET_OS_MACOS)
 #include "platform/thread_macos.h"
 #elif defined(TARGET_OS_WINDOWS)
 #include "platform/thread_win.h"
 #else
 #error Unknown target os.
 #endif
 
 namespace dart {
 
 class Thread {
  public:
-  // Function to be called on thread start.
+  static const int64_t kNoTimeout = 0;
+  static const ThreadHandle kInvalidThreadHandle;
+
   typedef void (*ThreadStartFunction) (uword parameter);
 
-  // TODO(iposva): Define the proper interface for spawning and killing threads.
-  Thread(ThreadStartFunction function, uword parameters);
-  ~Thread();
-
- private:
-  ThreadData data_;
-
-  DISALLOW_COPY_AND_ASSIGN(Thread);
+  // Start a thread running the specified function.
+  static ThreadHandle Start(ThreadStartFunction function, uword parameters);
+  // Wait for thread termination or timeout.
+  static bool Join(ThreadHandle thread, int64_t millis = kNoTimeout);

[Mads Ager (google), 2012/01/20 12:35:34]

Do you need the timeout version of this. I don't see anything in this change
that needs it. Since it is unimplemented on Mac in this change I think we should
leave it out.

[Søren Gjesse, 2012/01/20 13:25:45]

Good point, removed.

 };
 
 
 class Mutex {
  public:
   Mutex();
   ~Mutex();
 
   void Lock();
   bool TryLock();
@@ skipping 31 matching lines @@
  private:
   MonitorData data_;  // OS-specific data.
 
   DISALLOW_COPY_AND_ASSIGN(Monitor);
 };
 
 }  // namespace dart
 
 
 #endif  // PLATFORM_THREAD_H_