Account reconcilor: Use cookie store cookie changed subscription.

chrome/browser/signin/signin_cookie_changed_subscription.h

Index: chrome/browser/signin/signin_cookie_changed_subscription.h
diff --git a/chrome/browser/signin/signin_cookie_changed_subscription.h b/chrome/browser/signin/signin_cookie_changed_subscription.h
new file mode 100644
index 0000000000000000000000000000000000000000..a084310c7fc0515b9009244763014ade5394240d
--- /dev/null
+++ b/chrome/browser/signin/signin_cookie_changed_subscription.h
@@ -0,0 +1,67 @@
+// 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"
+#include "net/url_request/url_request_context_getter.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:
+  // Creates a cookie changed subscription and registers for cookie changed
+  // events.
+  SigninCookieChangedSubscription(
+      scoped_refptr<net::URLRequestContextGetter> context_getter,
+      const GURL& url,
+      const std::string& name,
+      const net::CookieStore::CookieChangedCallback& callback);
+  ~SigninCookieChangedSubscription() override;
+
+  // Handler for cookie changed events.
+  void OnCookieChanged(const net::CanonicalCookie& cookie, bool removed);

[Roger Tawa OOO till Jul 10th, 2014/11/04 16:24:24]

Nit: any way to make this private?

[msarda, 2014/11/04 17:50:15]

Done.

+
+ private:
+  // Holder of a cookie store cookie changed subscription.
+  struct SubscriptionHolder {
+    scoped_ptr<net::CookieStore::CookieChangedSubscription> subscription;
+    SubscriptionHolder();
+    ~SubscriptionHolder();
+  };
+
+  // Adds a callback for cookie changed events. This method is called on the
+  // network thread, so it is safe to access the cookie store.
+  static void RegisterForCookieChangesOnIOThread(
+      scoped_refptr<net::URLRequestContextGetter> context_getter,
+      const GURL url,
+      const std::string name,
+      const net::CookieStore::CookieChangedCallback callback,
+      SigninCookieChangedSubscription::SubscriptionHolder*
+          out_subscription_holder);
+
+  void RegisterForCookieChangedNotifications(const GURL& url,
+                                             const std::string& name);
+
+  // The context getter.
+  scoped_refptr<net::URLRequestContextGetter> context_getter_;
+
+  // The holder of a cookie changed subscription. Must be destroyed on the
+  // network thread.
+  scoped_ptr<SubscriptionHolder> subscription_holder_io_;

[Roger Tawa OOO till Jul 10th, 2014/11/04 16:24:24]

Nit: comment says "netwotk thread", variable has "io" in name.  Maybe change
comment?

[msarda, 2014/11/04 17:50:15]

Not done as discussed off-line.

+
+  // Callback to be run on cookie changed events.
+  net::CookieStore::CookieChangedCallback callback_;
+
+  base::ThreadChecker thread_checker_;
+
+  DISALLOW_COPY_AND_ASSIGN(SigninCookieChangedSubscription);
+};
+
+#endif  // CHROME_BROWSER_SIGNIN_SIGNIN_COOKIE_CHANGED_SUBSCRIPTION_H_