Return NULL from base::Thread::message_loop_proxy() when thread is not started.

base/threading/thread.h

 // Copyright (c) 2011 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.
 
 #ifndef BASE_THREAD_H_
 #define BASE_THREAD_H_
 #pragma once
 
 #include <string>
 
@@ skipping 99 matching lines @@
   // NOTE: You must not call this MessageLoop's Quit method directly.  Use
   // the Thread's Stop method instead.
   //
   MessageLoop* message_loop() const { return message_loop_; }
 
   // Returns a MessageLoopProxy for this thread.  Use the MessageLoopProxy's
   // PostTask methods to execute code on the thread.  This only returns
   // non-NULL after a successful call to Start. After Stop has been called,
   // this will return NULL. Callers can hold on to this even after the thread
   // is gone.
   // TODO(sanjeevr): Look into merging MessageLoop and MessageLoopProxy.

[willchan no longer on Chromium, 2012/05/04 19:00:57]

Can you delete this comment? They aren't gonna be merged.

[Do not use (sergeyu), 2012/05/04 21:01:36]

Done.

   scoped_refptr<MessageLoopProxy> message_loop_proxy() const {
-    return message_loop_->message_loop_proxy();
+    return message_loop_ ? message_loop_->message_loop_proxy() : NULL;
   }
 
   // Returns the name of this thread (for display in debugger too).
   const std::string &thread_name() { return name_; }
 
   // The native thread handle.
   PlatformThreadHandle thread_handle() { return thread_; }
 
   // The thread ID.
   PlatformThreadId thread_id() const { return thread_id_; }
@@ skipping 50 matching lines @@
   std::string name_;
 
   friend void ThreadQuitHelper();
 
   DISALLOW_COPY_AND_ASSIGN(Thread);
 };
 
 }  // namespace base
 
 #endif  // BASE_THREAD_H_