Be able to build cast_unittest and related targets in Chrome tree

media/cast/cast_thread.cc

 // Copyright 2013 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/cast/cast_thread.h"
 
 #include "base/logging.h"
 
 using base::TaskRunner;
 
 namespace media {
 namespace cast {
 
 CastThread::CastThread(
     scoped_refptr<TaskRunner> main_thread_proxy,
     scoped_refptr<TaskRunner> audio_encode_thread_proxy,
     scoped_refptr<TaskRunner> audio_decode_thread_proxy,
     scoped_refptr<TaskRunner> video_encode_thread_proxy,
     scoped_refptr<TaskRunner> video_decode_thread_proxy)
     : main_thread_proxy_(main_thread_proxy),
       audio_encode_thread_proxy_(audio_encode_thread_proxy),
       audio_decode_thread_proxy_(audio_decode_thread_proxy),
       video_encode_thread_proxy_(video_encode_thread_proxy),
       video_decode_thread_proxy_(video_decode_thread_proxy) {
   DCHECK(main_thread_proxy) << "Main thread required";
 }
 
+CastThread::~CastThread() {

[pwestin, 2013/09/26 23:20:23]

nit: {} on same line

[Alpha Left Google, 2013/09/27 00:45:13]

Done.

+}
+
 bool CastThread::PostTask(ThreadId identifier,
                           const tracked_objects::Location& from_here,
                           const base::Closure& task) {
   scoped_refptr<TaskRunner> task_runner =
       GetMessageTaskRunnerForThread(identifier);
 
   return task_runner->PostTask(from_here, task);
 }
 
 bool CastThread::PostDelayedTask(ThreadId identifier,
@@ skipping 12 matching lines @@
     case CastThread::MAIN:
       return main_thread_proxy_;
     case CastThread::AUDIO_ENCODER:
       return audio_encode_thread_proxy_;
     case CastThread::AUDIO_DECODER:
       return audio_decode_thread_proxy_;
     case CastThread::VIDEO_ENCODER:
       return video_encode_thread_proxy_;
     case CastThread::VIDEO_DECODER:
       return video_decode_thread_proxy_;
+    default:
+      CHECK(false) << "Invalid Thread ID.";
+      return NULL;

[pwestin, 2013/09/26 23:20:23]

Why add this? it's much better to get a compilation error than a run-time error 

[Alpha Left Google, 2013/09/27 00:45:13]

VC++ complains without this case.

   }
 }
 
 }  // namespace cast
 }  // namespace media