[Sync] Implementation of model association for passwords using sync API

chrome/browser/password_manager/password_syncable_service.h

 // Copyright 2013 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_PASSWORD_MANAGER_PASSWORD_SYNCABLE_SERVICE_H__
 #define CHROME_BROWSER_PASSWORD_MANAGER_PASSWORD_SYNCABLE_SERVICE_H__
 
 #include <string>
 #include <vector>
 
@@ skipping 42 matching lines @@
   // Returns the unique tag that will serve as the sync identifier for the
   // |password| entry.
   static std::string MakeTag(const autofill::PasswordForm& password);
   static std::string MakeTag(const sync_pb::PasswordSpecificsData& password);
   static std::string MakeTag(const std::string& origin_url,
                              const std::string& username_element,
                              const std::string& username_value,
                              const std::string& password_element,
                              const std::string& signon_realm);
 
+  // Converts the |PasswordForm| to |SyncData| suitable for syncing.

[Nicolas Zea, 2013/10/21 23:55:14]

nit: no need to mention suitable for syncing; that's implied by converting to
SyncData

[lipalani1, 2013/10/24 00:13:58]

Done.

+  static syncer::SyncData CreateSyncData(
+      const autofill::PasswordForm& password);

[Nicolas Zea, 2013/10/21 23:55:14]

nit: fix indent

[lipalani1, 2013/10/24 00:13:58]

Done.

+  static void ExtractPasswordFromSpecifics(
+     const sync_pb::PasswordSpecificsData& password,
+     autofill::PasswordForm* new_password);
+
  private:
   typedef std::vector<autofill::PasswordForm*> PasswordForms;
 
+  typedef std::map<std::string,

[Nicolas Zea, 2013/10/21 23:55:14]

nit: remove newline above

[lipalani1, 2013/10/24 00:13:58]

Done.

+                   std::pair<syncer::SyncChange::SyncChangeType,
+                             std::vector<autofill::PasswordForm*>::iterator > >
+      PasswordEntryMap;
+
   // Use the |PasswordStore| APIs to add and update entries.
   void WriteToPasswordStore(PasswordForms* new_entries,
                             PasswordForms* udpated_entries);
+  void CreateOrUpdateEntry(

[Nicolas Zea, 2013/10/21 23:55:14]

Comment about what function does

[lipalani1, 2013/10/24 00:13:58]

Done.

+      const syncer::SyncData& data,
+      PasswordEntryMap* loaded_data,
+      ScopedVector<autofill::PasswordForm>* new_entries,
+      ScopedVector<autofill::PasswordForm>* udpated_entries);

[Nicolas Zea, 2013/10/21 23:55:14]

udpated ->  updated

[lipalani1, 2013/10/24 00:13:58]

Done.

 
-  // Converts the |PasswordForm| to |SyncData| suitable for syncing.
-  syncer::SyncData CreateSyncData(const autofill::PasswordForm& password);
+  // Virtual so tests can override.
+  virtual void NotifyPasswordStore();
 
   // The factory that creates sync errors. |SyncError| has rich data
   // suitable for debugging.
   scoped_ptr<syncer::SyncErrorFactory> sync_error_factory_;
 
   // |SyncProcessor| will mirror the |PasswordStore| changes in the sync db.
   scoped_ptr<syncer::SyncChangeProcessor> sync_processor_;
 
   // The password store that adds/updates/deletes password entries.
   scoped_refptr<PasswordStore> password_store_;
 };
 
 #endif  // CHROME_BROWSER_PASSWORD_MANAGER_PASSWORD_SYNCABLE_SERVICE_H__