Fix memory leak in SequencedWorkerPool

Fix memory leak in SequencedWorkerPool

The leak was due to a cyclic reference from SequencedWorkerPool::Worker.

BUG=115987
TEST=
TBR=brettw

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=124097

[akalin, 2012-02-28 01:45:34]

+brettw for review

[brettw, 2012-02-28 04:38:45]

https://chromiumcodereview.appspot.com/9480016/diff/1/base/threading/sequence...
File base/threading/sequenced_worker_pool.cc (right):

https://chromiumcodereview.appspot.com/9480016/diff/1/base/threading/sequence...
base/threading/sequenced_worker_pool.cc:49: // around as long as we are running.
 Note that this (deliberately)
Is the leaks on shutdown comment still valid? If so, it needs more explanation
since it doesn't make sense to me.

[akalin, 2012-02-28 22:10:33]

PTAL

http://codereview.chromium.org/9480016/diff/1/base/threading/sequenced_worker...
File base/threading/sequenced_worker_pool.cc (right):

http://codereview.chromium.org/9480016/diff/1/base/threading/sequenced_worker...
base/threading/sequenced_worker_pool.cc:49: // around as long as we are running.
 Note that this (deliberately)
On 2012/02/28 04:38:46, brettw wrote:
> Is the leaks on shutdown comment still valid? If so, it needs more explanation
> since it doesn't make sense to me.

Done.

[akalin, 2012-02-29 02:30:53]

I'm just going to TBR this, to save memory sheriffs the trouble of widening the
suppression, etc.

[akalin, 2012-02-29 02:31:08]

if you have further comments, i can address them in another CL