Rename BrowserThread::IsWellKnownThread to IsThreadInitialized.

content/browser/browser_thread_impl.cc

Index: content/browser/browser_thread_impl.cc
diff --git a/content/browser/browser_thread_impl.cc b/content/browser/browser_thread_impl.cc
index 90f660a55af6163b85290457b165efffd397c44c..fdfc62144007a2a8644fdbfc0c5cc36f67588405 100644
--- a/content/browser/browser_thread_impl.cc
+++ b/content/browser/browser_thread_impl.cc
@@ -339,14 +339,14 @@ base::SequencedWorkerPool* BrowserThread::GetBlockingPool() {
 }
 
 // static
-bool BrowserThread::IsWellKnownThread(ID identifier) {
+bool BrowserThread::IsThreadInitialized(ID identifier) {
   if (g_globals == NULL)
     return false;
 
   BrowserThreadGlobals& globals = g_globals.Get();
   base::AutoLock lock(globals.lock);
-  return (identifier >= 0 && identifier < ID_COUNT &&
-          globals.threads[identifier]);
+  DCHECK(identifier >= 0 && identifier < ID_COUNT);

[Matt Giuca, 2013/08/05 01:31:16]

I changed this because if |identifier| is not a valid enum value, that is a
programming error which should DCHECK, not merely a false condition.

All call sites pass a literal enum value, so this should not change the current
behaviour at all.

+  return globals.threads[identifier];
 }
 
 // static