diff options
| author | joi@chromium.org <joi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-02 15:31:56 +0000 |
|---|---|---|
| committer | joi@chromium.org <joi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-02 15:31:56 +0000 |
| commit | d583c3a30db9474b0dcef5128ec6901c8ff23b98 (patch) | |
| tree | 227db6cd60ac59d5e4990d1afa9de091af5295db /base/memory | |
| parent | 4bacb82375e04d44967e71afc1fa6e1018f9810b (diff) | |
| download | chromium_src-d583c3a30db9474b0dcef5128ec6901c8ff23b98.zip chromium_src-d583c3a30db9474b0dcef5128ec6901c8ff23b98.tar.gz chromium_src-d583c3a30db9474b0dcef5128ec6901c8ff23b98.tar.bz2 | |
Thread::Stop() must be called before any subclass's destructor completes.
Update base::Thread documentation, fix all subclasses I could find
that had a problem, and remove no-longer-necessary suppressions.
BUG=102134
Review URL: http://codereview.chromium.org/8427007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108296 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/memory')
| -rw-r--r-- | base/memory/weak_ptr_unittest.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/base/memory/weak_ptr_unittest.cc b/base/memory/weak_ptr_unittest.cc index c244ec0..f9f8b3e 100644 --- a/base/memory/weak_ptr_unittest.cc +++ b/base/memory/weak_ptr_unittest.cc @@ -47,6 +47,10 @@ class BackgroundThread : public Thread { : Thread("owner_thread") { } + ~BackgroundThread() { + Stop(); + } + void CreateConsumerFromProducer(Consumer** consumer, Producer* producer) { WaitableEvent completion(true, false); message_loop()->PostTask( |