Thread::Stop() must be called before any subclass's destructor completes.

base/memory/weak_ptr_unittest.cc

 // 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.
 
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "base/message_loop.h"
 #include "base/synchronization/waitable_event.h"
 #include "base/threading/thread.h"
@@ skipping 29 matching lines @@
 struct Consumer { WeakPtr<Producer> producer; };
 
 // Helper class to create and destroy weak pointer copies
 // and delete objects on a background thread.
 class BackgroundThread : public Thread {
  public:
   BackgroundThread()
       : Thread("owner_thread") {
   }
 
+  ~BackgroundThread() {
+    Stop();
+  }
+
   void CreateConsumerFromProducer(Consumer** consumer, Producer* producer) {
     WaitableEvent completion(true, false);
     message_loop()->PostTask(
         FROM_HERE,
         NewRunnableFunction(&BackgroundThread::DoCreateFromProducer,
                             consumer,
                             producer,
                             &completion));
     completion.Wait();
   }
@@ skipping 275 matching lines @@
     Producer producer;
     Consumer consumer;
     consumer.producer = producer.AsWeakPtr();
     thread.CreateConsumerFromConsumer(&consumer_copy, &consumer);
   }
   EXPECT_TRUE(consumer_copy->producer == NULL);
   thread.DeleteConsumer(consumer_copy);
 }
 
 }  // namespace base