Refactor sync notifier out of sync api.

chrome/browser/sync/notifier/sync_notifier_observer.h

+// Copyright (c) 2011 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_SYNC_NOTIFIER_SYNC_NOTIFIER_OBSERVER_H_
+#define CHROME_BROWSER_SYNC_NOTIFIER_SYNC_NOTIFIER_OBSERVER_H_
+

[akalin, 2011/03/10 04:26:45]

#include <string>

[Agrawal, 2011/03/11 01:13:07]

Done.

+#include "chrome/browser/sync/sessions/session_state.h"
+
+namespace sync_notifier {
+
+class SyncNotifierObserver {
+ public:
+  SyncNotifierObserver() {}
+  virtual ~SyncNotifierObserver() {}
+
+  virtual void OnIncomingNotification(
+      const browser_sync::sessions::TypePayloadMap& type_payloads) = 0;
+  virtual void OnNotificationStateChange(bool notifications_enabled) = 0;
+
+  // TODO(nileshagrawal): Find a way to hide state handling inside the
+  // sync notifier implementation.
+  virtual void StoreCookie(const std::string& cookie) = 0;

[akalin, 2011/03/10 04:26:45]

Rename to WriteState or StoreState, matching existing terminology. (I don't
think changing to "cookie" really adds anything.)

[Agrawal, 2011/03/11 01:13:07]

Done.

+};
+}  // namespace sync_notifier
+#endif  // CHROME_BROWSER_SYNC_NOTIFIER_SYNC_NOTIFIER_OBSERVER_H_