Add function support for Sleep and MessageLoop with TimeDelta input.

base/threading/platform_thread_posix.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/threading/platform_thread.h"
 
 #include <errno.h>
 #include <sched.h>
 
 #include "base/lazy_instance.h"
@@ skipping 147 matching lines @@
   sleep_time.tv_sec = duration_ms / 1000;
   duration_ms -= sleep_time.tv_sec * 1000;
 
   // Contains the portion of duration_ms < 1 sec.
   sleep_time.tv_nsec = duration_ms * 1000 * 1000;  // nanoseconds.
 
   while (nanosleep(&sleep_time, &remaining) == -1 && errno == EINTR)
     sleep_time = remaining;
 }
 
+// static
+void PlatformThread::Sleep(TimeDelta duration) {
+  // NOTE: This function is intended to supplant the other version of Sleep
+  // in the future.  See issue 108171 for more information.
+  Sleep(duration.InMillisecondsRoundedUp());

[brettw, 2011/12/28 20:10:27]

It seems like this is backwards. Sleep(ms) should be implemented in terms of
this function. Both TimeDelta and the timespec used in the existing sleep code
have a lot more resolution than just ms. While it arguably doesn't matter for
this case, it seems weird to throw it away.

+}
+
 #if defined(OS_LINUX)
 // static
 void PlatformThread::SetName(const char* name) {
   // have to cast away const because ThreadLocalPointer does not support const
   // void*
   current_thread_name.Pointer()->Set(const_cast<char*>(name));
   tracked_objects::ThreadData::InitializeThreadContext(name);
 
   // On linux we can get the thread names to show up in the debugger by setting
   // the process name for the LWP.  We don't want to do this for the main
@@ skipping 64 matching lines @@
 // Mac OS X uses lower-level mach APIs
 
 // static
 void PlatformThread::SetThreadPriority(PlatformThreadHandle, ThreadPriority) {
   // TODO(crogers): implement
   NOTIMPLEMENTED();
 }
 #endif
 
 }  // namespace base