Save the oauth client id used in App Notification setup

chrome/browser/extensions/extension_sync_data.h

Index: chrome/browser/extensions/extension_sync_data.h
diff --git a/chrome/browser/extensions/extension_sync_data.h b/chrome/browser/extensions/extension_sync_data.h
index 3386adcdf614da0c1564fc8f8d1bd93645594ff3..f072de643ea90ad593c1f2f02cbf49cb077ff3f7 100644
--- a/chrome/browser/extensions/extension_sync_data.h
+++ b/chrome/browser/extensions/extension_sync_data.h
@@ -29,7 +29,7 @@ class ExtensionSyncData {
   ExtensionSyncData(const Extension& extension,
                     bool enabled,
                     bool incognito_enabled,
-                    bool notifications_initial_setup_done,
+                    const std::string& notifications_client_id,
                     bool notifications_disabled);
   ~ExtensionSyncData();
 
@@ -56,8 +56,8 @@ class ExtensionSyncData {
   // Used only for debugging.
   const std::string& name() const { return name_; }
 
-  bool notifications_initial_setup_done() const {

[Munjal (Google), 2011/11/29 21:55:34]

May be keep this method also in addition to the new method to check if initial
setup is done? Name it IsInitialSetupDone?

[asargent_no_longer_on_chrome, 2011/11/30 06:06:20]

I thought the idea was that the initial_setup_done field in the sync protobuffer
was deprecated and we were going to stop using it (that's what the comment in
app_specifics.proto that you added says :) ). 

-    return notifications_initial_setup_done_;
+  const std::string& notifications_client_id() const {
+    return notifications_client_id_;
   }
 
   bool notifications_disabled() const {
@@ -77,7 +77,7 @@ class ExtensionSyncData {
   Version version_;
   GURL update_url_;
   std::string name_;
-  bool notifications_initial_setup_done_;
+  std::string notifications_client_id_;
   bool notifications_disabled_;
 };