Replace delay loop by sleep() call.

media/audio/audio_output_controller.cc

 // Copyright (c) 2012 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 "media/audio/audio_output_controller.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/message_loop.h"
 #include "base/synchronization/waitable_event.h"
@@ skipping 191 matching lines @@
     message_loop_->PostDelayedTask(
         FROM_HERE,
         base::Bind(&AudioOutputController::PollAndStartIfDataReady,
         weak_this_.GetWeakPtr()),
         TimeDelta::FromMilliseconds(kPollPauseInMilliseconds));
   }
 }
 
 void AudioOutputController::StartStream() {
   DCHECK(message_loop_->BelongsToCurrentThread());
+
 #if defined(OS_MACOSX)
   // HACK: workaround for crbug.com/128128.
   // Mac OS crashes if we start playback too soon after previous ended.
   // Audio mixer contains better fix, it keeps physical stream opened for
   // some time after logical one is closed, so sequence of play / pause / play /
   // pause / ... would reuse the same stream, but we need fix for M20.
   // TODO(enal): Remove after turning on mixer by default.
-  while ((Time::Now() - previous_stop_time_).InMilliseconds() <
-         kMacWorkaroundInMilliseconds) {
-    base::PlatformThread::YieldCurrentThread();
+  int milliseconds_since_stop =
+      (Time::Now() - previous_stop_time_).InMilliseconds();
+  if (milliseconds_since_stop < kMacWorkaroundInMilliseconds) {

[DaleCurtis, 2012/05/16 20:41:45]

Add milliseconds_since_stop >= 0 && for safety.

+    base::PlatformThread::Sleep(
+        TimeDelta::FromMilliseconds(kMacWorkaroundInMilliseconds -
+                                    milliseconds_since_stop));
   }
 #endif
 
   state_ = kPlaying;
 
   // We start the AudioOutputStream lazily.
   stream_->Start(this);
 
   // Tell the event handler that we are now playing.
   handler_->OnPlaying(this);
@@ skipping 121 matching lines @@
     stream_ = NULL;
     weak_this_.InvalidateWeakPtrs();
   }
 
   // Should be last in the method, do not touch "this" from here on.
   if (done != NULL)
     done->Signal();
 }
 
 }  // namespace media