Style fixes for cast_unittest to build

media/cast/cast_thread.cc

Index: media/cast/cast_thread.cc
diff --git a/media/cast/cast_thread.cc b/media/cast/cast_thread.cc
index e47446870a362da531ad526c6a802f5a7ff4c4e7..5745fb3b871486771c9b45502f3457d7336286b7 100644
--- a/media/cast/cast_thread.cc
+++ b/media/cast/cast_thread.cc
@@ -25,6 +25,8 @@ CastThread::CastThread(
   DCHECK(main_thread_proxy) << "Main thread required";
 }
 
+CastThread::~CastThread() {}
+
 bool CastThread::PostTask(ThreadId identifier,
                           const tracked_objects::Location& from_here,
                           const base::Closure& task) {
@@ -57,6 +59,9 @@ scoped_refptr<TaskRunner> CastThread::GetMessageTaskRunnerForThread(
       return video_encode_thread_proxy_;
     case CastThread::VIDEO_DECODER:
       return video_decode_thread_proxy_;
+    default:
+      CHECK(false) << "Invalid Thread ID.";

[hubbe, 2013/10/02 19:44:52]

In google3, LOG(FATAL) is preferred over CHECK(false)
Is that not the case in chrome?
But perhaps NOTREACHED() would be more appropriate?

[Alpha Left Google, 2013/10/02 19:56:23]

Updated it to NOTREACHED().

+      return NULL;
   }
 }
 
@@ -73,7 +78,8 @@ bool CastThread::CurrentlyOn(ThreadId identifier) {
     case CastThread::VIDEO_DECODER:
       return video_decode_thread_proxy_->RunsTasksOnCurrentThread();
     default:
-      DCHECK(false) << "Wrong thread identifier";
+      CHECK(false) << "Wrong thread identifier";
+      return false;
   }
 }