[SyncFS] Introduce CallbackTracker to handle callback abortion

chrome/browser/sync_file_system/drive_backend/callback_tracker_unittest.cc

Index: chrome/browser/sync_file_system/drive_backend/callback_tracker_unittest.cc
diff --git a/chrome/browser/sync_file_system/drive_backend/callback_tracker_unittest.cc b/chrome/browser/sync_file_system/drive_backend/callback_tracker_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..60dc539ac8b4cc7e6e3816d09c8b4ea1bac7bac0
--- /dev/null
+++ b/chrome/browser/sync_file_system/drive_backend/callback_tracker_unittest.cc
@@ -0,0 +1,58 @@
+// Copyright 2014 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 "chrome/browser/sync_file_system/drive_backend/callback_tracker.h"
+
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace sync_file_system {
+namespace drive_backend {
+
+namespace {
+
+void Receiver(bool* called) {
+  DCHECK(called);
+  EXPECT_FALSE(*called);
+  *called = true;
+}
+
+}  // namespace
+
+TEST(CallbackTrackerTest, AbortAll) {
+  CallbackTracker tracker;
+
+  bool aborted = false;
+  bool invoked = false;
+  base::Closure callback = tracker.Register(base::Bind(&Receiver, &aborted),
+                                            base::Bind(&Receiver, &invoked));
+  tracker.AbortAll();
+  EXPECT_TRUE(aborted);
+  EXPECT_FALSE(invoked);
+
+  callback.Run();
+  EXPECT_TRUE(aborted);
+  EXPECT_FALSE(invoked);
+}
+
+TEST(CallbackTrackerTest, Invoke) {
+  CallbackTracker tracker;
+
+  bool aborted = false;
+  bool invoked = false;
+  base::Closure callback = tracker.Register(base::Bind(&Receiver, &aborted),
+                                            base::Bind(&Receiver, &invoked));
+  callback.Run();
+  EXPECT_FALSE(aborted);
+  EXPECT_TRUE(invoked);
+
+  // Second call should not do anything.
+  invoked = false;
+  callback.Run();
+  EXPECT_FALSE(invoked);
+
+  tracker.AbortAll();
+  EXPECT_FALSE(aborted);
+}

[nhiroki, 2014/06/04 09:16:47]

nit: can you add an empty line?

[tzik, 2014/06/04 09:45:24]

Done.

+}  // namespace drive_backend
+}  // namespace sync_file_system