Clean up PlatformThread.

webrtc/common_video/incoming_video_stream.cc

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 120 matching lines @@
 int32_t IncomingVideoStream::Start() {
   CriticalSectionScoped csS(stream_critsect_.get());
   if (running_) {
     return 0;
   }
 
   if (!disable_prerenderer_smoothing_) {
     CriticalSectionScoped csT(thread_critsect_.get());
     assert(incoming_render_thread_ == NULL);
 
-    incoming_render_thread_ = PlatformThread::CreateThread(
-        IncomingVideoStreamThreadFun, this, "IncomingVideoStreamThread");
+    incoming_render_thread_.reset(new rtc::PlatformThread(
+        IncomingVideoStreamThreadFun, this, "IncomingVideoStreamThread"));
     if (!incoming_render_thread_) {
       return -1;
     }
 
-    if (incoming_render_thread_->Start()) {
-    } else {
-      return -1;
-    }
-    incoming_render_thread_->SetPriority(kRealtimePriority);
+    incoming_render_thread_->Start();
+    incoming_render_thread_->SetPriority(rtc::kRealtimePriority);
     deliver_buffer_event_->StartTimer(false, kEventStartupTimeMs);
   }
+
   running_ = true;
   return 0;
 }
 
 int32_t IncomingVideoStream::Stop() {
   CriticalSectionScoped cs_stream(stream_critsect_.get());
 
   if (!running_) {
     return 0;
   }
 
-  PlatformThread* thread = NULL;
+  rtc::PlatformThread* thread = NULL;
   {
     CriticalSectionScoped cs_thread(thread_critsect_.get());
     if (incoming_render_thread_) {
       // Setting the incoming render thread to NULL marks that we're performing
       // a shutdown and will make IncomingVideoStreamProcess abort after wakeup.
       thread = incoming_render_thread_.release();
       deliver_buffer_event_->StopTimer();
       // Set the event to allow the thread to wake up and shut down without
       // waiting for a timeout.
       deliver_buffer_event_->Set();
     }
   }
   if (thread) {
-    if (thread->Stop()) {
-      delete thread;
-    } else {
-      assert(false);
-    }
+    thread->Stop();
+    delete thread;
   }
   running_ = false;
   return 0;
 }
 
 int32_t IncomingVideoStream::Reset() {
   CriticalSectionScoped cs_buffer(buffer_critsect_.get());
   render_buffers_->ReleaseAllFrames();
   return 0;
 }
@@ skipping 65 matching lines @@
     external_callback_->RenderFrame(stream_id_, video_frame);
   } else if (render_callback_) {
     render_callback_->RenderFrame(stream_id_, video_frame);
   }
 
   // We're done with this frame.
   last_render_time_ms_ = video_frame.render_time_ms();
 }
 
 }  // namespace webrtc