[BackgroundSync] Add browser side support for SyncRegistration.done

content/browser/background_sync/background_sync_registration.cc

 // Copyright 2015 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 "content/browser/background_sync/background_sync_registration.h"
 
+#include "base/bind.h"
+#include "base/location.h"
+#include "base/logging.h"
+#include "base/single_thread_task_runner.h"
+#include "base/thread_task_runner_handle.h"
+
 namespace content {
 
 const BackgroundSyncRegistration::RegistrationId
     BackgroundSyncRegistration::kInvalidRegistrationId = -1;
 
 const BackgroundSyncRegistration::RegistrationId
     BackgroundSyncRegistration::kInitialId = 0;
 
+BackgroundSyncRegistration::BackgroundSyncRegistration() = default;
+BackgroundSyncRegistration::~BackgroundSyncRegistration() = default;
+
 bool BackgroundSyncRegistration::Equals(
     const BackgroundSyncRegistration& other) const {
   return options_.Equals(other.options_);
 }
 
 bool BackgroundSyncRegistration::IsValid() const {
   return id_ != kInvalidRegistrationId;
 }
 
+void BackgroundSyncRegistration::AddDoneCallback(
+    const StateCallback& callback) {
+  DCHECK_NE(BACKGROUND_SYNC_STATE_SUCCESS, sync_state_);
+  DCHECK_NE(BACKGROUND_SYNC_STATE_FAILED, sync_state_);
+  DCHECK_NE(BACKGROUND_SYNC_STATE_UNREGISTERED, sync_state_);

[iclelland, 2015/09/17 13:48:09]

This set of negative tests complements the positive tests in NotifyWhenDoneImpl,
but they could easily get out of sync. Could we switch these around, or add a
comment that adding a state means updating one or the other method?

[jkarlin, 2015/09/17 17:40:25]

Nice catch. I've added HasCompleted() which uses a switch so we should be good
on future enum changes.

+  notify_done_callbacks_.push_back(callback);
+}
+
+void BackgroundSyncRegistration::RunDoneCallbacks() {
+  DCHECK_NE(BACKGROUND_SYNC_STATE_PENDING, sync_state_);
+  DCHECK_NE(BACKGROUND_SYNC_STATE_FIRING, sync_state_);
+
+  for (auto& callback : notify_done_callbacks_) {
+    base::ThreadTaskRunnerHandle::Get()->PostTask(
+        FROM_HERE, base::Bind(callback, sync_state_));
+  }
+
+  notify_done_callbacks_.clear();
+}
+
 }  // namespace content