Fixed member use after destruction in BarrierClosure.

base/barrier_closure.cc

Index: base/barrier_closure.cc
diff --git a/base/barrier_closure.cc b/base/barrier_closure.cc
index 1b77429ae3f3041294f89d77ffb00a76075e1355..d870fcf28be7cbc437fbf90f7e4191e9b4803fac 100644
--- a/base/barrier_closure.cc
+++ b/base/barrier_closure.cc
@@ -28,8 +28,8 @@ BarrierInfo::BarrierInfo(int num_callbacks, const base::Closure& done_closure)
 void BarrierInfo::Run() {
   DCHECK(!base::AtomicRefCountIsZero(&num_callbacks_left_));
   if (!base::AtomicRefCountDec(&num_callbacks_left_)) {
-    done_closure_.Run();
-    done_closure_.Reset();
+    base::Closure done_closure = done_closure_;

[danakj, 2015/04/30 17:32:55]

Why no Reset still? Was it important before? There appears to be no test
coverage for that Reset.

[dzhioev (left Google), 2015/04/30 20:00:54]

I missed that part. Returned Reset back and added a test case.

+    done_closure.Run();
   }
 }