Clean up PlatformThread.

webrtc/modules/video_render/mac/video_render_nsopengl.mm

 /*
  *  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 360 matching lines @@
 _fullScreenWindow( nil),
 _windowRect( ),
 _windowWidth( 0),
 _windowHeight( 0),
 _nsglChannels( ),
 _zOrderToChannel( ),
 _renderingIsPaused (FALSE),
 _windowRefSuperView(NULL),
 _windowRefSuperViewFrame(NSMakeRect(0,0,0,0))
 {
-  _screenUpdateThread = PlatformThread::CreateThread(
-      ScreenUpdateThreadProc, this, "ScreenUpdateNSOpenGL");
+  _screenUpdateThread.reset(new rtc::PlatformThread(
+      ScreenUpdateThreadProc, this, "ScreenUpdateNSOpenGL"));
 }
 
 int VideoRenderNSOpenGL::ChangeWindow(CocoaRenderView* newWindowRef)
 {
 
     LockAGLCntx();
 
     _windowRef = newWindowRef;
 
     if(CreateMixingContext() == -1)
@@ skipping 27 matching lines @@
 {
 
     LockAGLCntx();
 
     const unsigned int MONITOR_FREQ = 60;
     if(TRUE == _renderingIsPaused)
     {
         WEBRTC_TRACE(kTraceDebug, kTraceVideoRenderer, _id, "Restarting screenUpdateThread");
 
         // we already have the thread. Most likely StopRender() was called and they were paused
-        if(FALSE == _screenUpdateThread->Start() ||
-                FALSE == _screenUpdateEvent->StartTimer(true, 1000/MONITOR_FREQ))
-        {
+        _screenUpdateThread->Start();
+        if (FALSE ==
+            _screenUpdateEvent->StartTimer(true, 1000 / MONITOR_FREQ)) {
             WEBRTC_TRACE(kTraceError, kTraceVideoRenderer, _id, "Failed to restart screenUpdateThread or screenUpdateEvent");
             UnlockAGLCntx();
             return -1;
         }
 
-        _screenUpdateThread->SetPriority(kRealtimePriority);
+        _screenUpdateThread->SetPriority(rtc::kRealtimePriority);
 
         UnlockAGLCntx();
         return 0;
     }
 
 
     if (!_screenUpdateThread)
     {
         WEBRTC_TRACE(kTraceDebug, kTraceVideoRenderer, _id, "failed start screenUpdateThread");
         UnlockAGLCntx();
@@ skipping 15 matching lines @@
 
     // pause the update thread and the event timer
     if(!_screenUpdateThread || !_screenUpdateEvent)
     {
         _renderingIsPaused = TRUE;
 
         UnlockAGLCntx();
         return 0;
     }
 
-    if(FALSE == _screenUpdateThread->Stop() || FALSE == _screenUpdateEvent->StopTimer())
-    {
+    _screenUpdateThread->Stop();
+    if (FALSE == _screenUpdateEvent->StopTimer()) {
         _renderingIsPaused = FALSE;
 
         UnlockAGLCntx();
         return -1;
     }
 
     _renderingIsPaused = TRUE;
 
     UnlockAGLCntx();
     return 0;
@@ skipping 164 matching lines @@
               [_windowRef setFrame:_windowRefSuperViewFrame];
             }
 
             WEBRTC_TRACE(kTraceDebug, kTraceVideoRenderer, 0, "%s:%d Attempting to release fullscreen window", __FUNCTION__, __LINE__);
             [_fullScreenWindow releaseFullScreen];
 
         }
     }
 
     // Signal event to exit thread, then delete it
-    PlatformThread* tmpPtr = _screenUpdateThread.release();
+    rtc::PlatformThread* tmpPtr = _screenUpdateThread.release();
 
     if (tmpPtr)
     {
         _screenUpdateEvent->Set();
         _screenUpdateEvent->StopTimer();
 
-        if (tmpPtr->Stop())
-        {
-            delete tmpPtr;
-        }
+        tmpPtr->Stop();
+        delete tmpPtr;
         delete _screenUpdateEvent;
         _screenUpdateEvent = NULL;
     }
 
     if (_nsglContext != 0)
     {
         [_nsglContext makeCurrentContext];
         _nsglContext = nil;
     }
 
@@ skipping 28 matching lines @@
 {
 
     LockAGLCntx();
     if (!_screenUpdateThread)
     {
         UnlockAGLCntx();
         return -1;
     }
 
     _screenUpdateThread->Start();
-    _screenUpdateThread->SetPriority(kRealtimePriority);
+    _screenUpdateThread->SetPriority(rtc::kRealtimePriority);
 
     // Start the event triggering the render process
     unsigned int monitorFreq = 60;
     _screenUpdateEvent->StartTimer(true, 1000/monitorFreq);
 
     if (CreateMixingContext() == -1)
     {
         UnlockAGLCntx();
         return -1;
     }
@@ skipping 127 matching lines @@
 
         return -1;
     }
 
     return 0;
 }
 
 int VideoRenderNSOpenGL::StopThread()
 {
 
-    PlatformThread* tmpPtr = _screenUpdateThread.release();
+    rtc::PlatformThread* tmpPtr = _screenUpdateThread.release();
     WEBRTC_TRACE(kTraceInfo, kTraceVideoRenderer, _id,
                  "%s Stopping thread ", __FUNCTION__, tmpPtr);
 
     if (tmpPtr)
     {
         _screenUpdateEvent->Set();
-        if (tmpPtr->Stop())
-        {
-            delete tmpPtr;
-        }
+        tmpPtr->Stop();
+        delete tmpPtr;
     }
 
     delete _screenUpdateEvent;
     _screenUpdateEvent = NULL;
 
     return 0;
 }
 
 bool VideoRenderNSOpenGL::IsFullScreen()
 {
@@ skipping 354 matching lines @@
  }
 
 
 
  */
 
 
 }  // namespace webrtc
 
 #endif // COCOA_RENDERING