Revert 67189 - Autofill work:

1. Added a command line swithc to facilitate incremental checking in of autofill code.
2. Autofill profile model associator.
3. Part of the upgrade code.(The part to take the latest changes from server on the legacy type)

Review URL: http://codereview.chromium.org/5126001

TBR=lipalani@chromium.org
Review URL: http://codereview.chromium.org/5315002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67199 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/sync/engine/syncapi.cc

@@ -83,7 +83,6 @@ using syncable::Directory;
 using syncable::DirectoryManager;
 using syncable::Entry;
 using syncable::SPECIFICS;
-using sync_pb::AutofillProfileSpecifics;
 
 typedef GoogleServiceAuthError AuthError;
 
@@ -282,11 +281,6 @@ const sync_pb::AutofillSpecifics& BaseNode::GetAutofillSpecifics() const {
   return GetEntry()->Get(SPECIFICS).GetExtension(sync_pb::autofill);
 }
 
-const AutofillProfileSpecifics& BaseNode::GetAutofillProfileSpecifics() const {
-  DCHECK_EQ(GetModelType(), syncable::AUTOFILL_PROFILE);
-  return GetEntry()->Get(SPECIFICS).GetExtension(sync_pb::autofill_profile);
-}
-
 const sync_pb::BookmarkSpecifics& BaseNode::GetBookmarkSpecifics() const {
   DCHECK(GetModelType() == syncable::BOOKMARKS);
   return GetEntry()->Get(SPECIFICS).GetExtension(sync_pb::bookmark);
@@ -763,11 +757,6 @@ ReadNode::ReadNode(const BaseTransaction* transaction)
   DCHECK(transaction);
 }
 
-ReadNode::ReadNode() {
-  entry_ = NULL;
-  transaction_ = NULL;
-}
-
 ReadNode::~ReadNode() {
   delete entry_;
 }

chrome/browser/sync/engine/syncapi.h

@@ -81,7 +81,6 @@ class WriteTransaction;
 namespace sync_pb {
 class AppSpecifics;
 class AutofillSpecifics;
-class AutofillProfileSpecifics;
 class BookmarkSpecifics;
 class EntitySpecifics;
 class ExtensionSpecifics;
@@ -150,7 +149,7 @@ class BaseNode {
   // metahandle).  These ids are strictly local handles.  They will persist
   // on this client, but the same object on a different client may have a
   // different ID value.
-  virtual int64 GetId() const;
+  int64 GetId() const;
 
   // Nodes are hierarchically arranged into a single-rooted tree.
   // InitByRootLookup on ReadNode allows access to the root. GetParentId is
@@ -196,9 +195,6 @@ class BaseNode {
   // data.  Can only be called if GetModelType() == AUTOFILL.
   const sync_pb::AutofillSpecifics& GetAutofillSpecifics() const;
 
-  virtual const sync_pb::AutofillProfileSpecifics&
-      GetAutofillProfileSpecifics() const;
-
   // Getter specific to the NIGORI datatype.  Returns protobuf
   // data.  Can only be called if GetModelType() == NIGORI.
   const sync_pb::NigoriSpecifics& GetNigoriSpecifics() const;
@@ -236,11 +232,11 @@ class BaseNode {
 
   // Return the ID of the node immediately after this in the sibling order.
   // For the last node in the ordering, return 0.
-  virtual int64 GetSuccessorId() const;
+  int64 GetSuccessorId() const;
 
   // Return the ID of the first child of this node.  If this node has no
   // children, return 0.
-  virtual int64 GetFirstChildId() const;
+  int64 GetFirstChildId() const;
 
   // These virtual accessors provide access to data members of derived classes.
   virtual const syncable::Entry* GetEntry() const = 0;
@@ -260,7 +256,8 @@ class BaseNode {
   bool DecryptIfNecessary(syncable::Entry* entry);
 
  private:
-  void* operator new(size_t size);  // Node is meant for stack use only.
+  // Node is meant for stack use only.
+  void* operator new(size_t size);
 
   // If this node represents a password, this field will hold the actual
   // decrypted password data.
@@ -460,9 +457,6 @@ class ReadNode : public BaseNode {
   virtual const syncable::Entry* GetEntry() const;
   virtual const BaseTransaction* GetTransaction() const;
 
- protected:
-  ReadNode();
-
  private:
   void* operator new(size_t size);  // Node is meant for stack use only.
 
@@ -498,8 +492,6 @@ class BaseTransaction {
   explicit BaseTransaction(UserShare* share);
   virtual ~BaseTransaction();
 
-  BaseTransaction() { lookup_= NULL; }
-
  private:
   // A syncable ScopedDirLookup, which is the parent of syncable transactions.
   syncable::ScopedDirLookup* lookup_;
@@ -545,12 +537,6 @@ class WriteTransaction : public BaseTransaction {
   virtual syncable::BaseTransaction* GetWrappedTrans() const;
   syncable::WriteTransaction* GetWrappedWriteTrans() { return transaction_; }
 
- protected:
-  WriteTransaction() {}
-
-  void SetTransaction(syncable::WriteTransaction* trans) {
-      transaction_ = trans;}
-
  private:
   void* operator new(size_t size);  // Transaction is meant for stack use only.