Account reconcilor: Use cookie store cookie changed subscription.

chrome/browser/signin/signin_cookie_changed_subscription.h

+// 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.
+
+#ifndef CHROME_BROWSER_SIGNIN_SIGNIN_COOKIE_CHANGED_SUBSCRIPTION_H_
+#define CHROME_BROWSER_SIGNIN_SIGNIN_COOKIE_CHANGED_SUBSCRIPTION_H_
+
+#include "base/memory/weak_ptr.h"
+#include "base/threading/thread_checker.h"
+#include "components/signin/core/browser/signin_client.h"
+
+// The subscription for a cookie changed events. This class lives on the
+// main thread.
+class SigninCookieChangedSubscription
+    : public SigninClient::CookieChangedSubscription,
+      public base::SupportsWeakPtr<SigninCookieChangedSubscription> {
+ public:
+  // Holder of a cookie store cookie changed subscription.
+  struct SubscriptionHolder {

[droger, 2014/11/04 10:31:36]

Should this be private?
(To make it private you may need RegisterForCookieChangesOnIOThread() and
RunAsyncOnCookieChanged() to be static class methods on
SigninCookieChangedSubscription.)

[msarda, 2014/11/04 12:06:06]

Done.

+    scoped_ptr<net::CookieStore::CookieChangedSubscription> subscription;
+    SubscriptionHolder();
+    ~SubscriptionHolder();
+  };
+
+  explicit SigninCookieChangedSubscription(
+      const net::CookieStore::CookieChangedCallback& callback);
+  ~SigninCookieChangedSubscription() override;
+
+  // Handler for cookie changed events.
+  void OnCookieChanged(const net::CanonicalCookie& cookie, bool removed);
+
+  // Registers for cookie changed events.
+  void RegisterForCookieChangedNotifications(
+      scoped_refptr<net::URLRequestContextGetter> context_getter,
+      const GURL& url,
+      const std::string& name);
+
+ private:
+  // Callback to be run on cookie changed events.
+  net::CookieStore::CookieChangedCallback callback_;
+
+  // The holder of a cookie changed subscription. Must be destroyed on the IO
+  // thread.
+  scoped_ptr<SubscriptionHolder> subscription_holder_io_;
+
+  base::ThreadChecker thread_checker_;
+
+  DISALLOW_COPY_AND_ASSIGN(SigninCookieChangedSubscription);
+};
+
+#endif  // CHROME_BROWSER_SIGNIN_SIGNIN_COOKIE_CHANGED_SUBSCRIPTION_H_