Remove extended attributes. The lame broken codepath in the DBS was causing

chrome/test/sync/engine/mock_connection_manager.h

 // Copyright (c) 2009 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.
 //
 // Mock ServerConnectionManager class for use in client unit tests.
 
 #ifndef CHROME_TEST_SYNC_ENGINE_MOCK_CONNECTION_MANAGER_H_
 #define CHROME_TEST_SYNC_ENGINE_MOCK_CONNECTION_MANAGER_H_
 
 #include <bitset>
@@ skipping 43 matching lines @@
   void SetCommitTimeRename(string prepend);
 
   // Control of get updates response. All updates set will only be returned
   // once. This mock object doesn't simulate a changelist, it simulates server
   // responses.
   void ResetUpdates();
   // Generic versions of AddUpdate functions. Tests using these function should
   // compile for both the int64 and string id based versions of the server.
   // The SyncEntity returned is only valid until the Sync is completed
   // (e.g. with SyncShare.) It allows to add further entity properties before
-  // sync, using AddUpdateExtendedAttributes.
+  // sync, using SetLastXXX() methods and/or GetMutableLastUpdate().
   sync_pb::SyncEntity* AddUpdateDirectory(syncable::Id id,
                                           syncable::Id parent_id,
                                           string name,
                                           int64 version,
                                           int64 sync_ts);
   sync_pb::SyncEntity* AddUpdateBookmark(syncable::Id id,
                                          syncable::Id parent_id,
                                          string name,
                                          int64 version,
                                          int64 sync_ts);
@@ skipping 12 matching lines @@
   sync_pb::SyncEntity* AddUpdateDirectory(string id,
                                           string parent_id,
                                           string name,
                                           int64 version,
                                           int64 sync_ts);
   sync_pb::SyncEntity* AddUpdateBookmark(string id,
                                          string parent_id,
                                          string name,
                                          int64 version,
                                          int64 sync_ts);
-  void AddUpdateExtendedAttributes(sync_pb::SyncEntity* ent,
-                                   std::string* xattr_key,
-                                   syncable::Blob* xattr_value,
-                                   int xattr_count);
-
   void SetLastUpdateDeleted();
   void SetLastUpdateServerTag(const string& tag);
   void SetLastUpdateClientTag(const string& tag);
   void SetLastUpdateOriginatorFields(const string& client_id,
                                      const string& entry_id);
   void SetLastUpdatePosition(int64 position_in_parent);
   void SetNewTimestamp(int64 ts);
   void SetChangesRemaining(int64 timestamp);
 
   // For AUTHENTICATE responses.
@@ skipping 169 matching lines @@
   std::bitset<syncable::MODEL_TYPE_COUNT> expected_filter_;
 
   int num_get_updates_requests_;
 
   sync_pb::ClientToServerMessage last_request_;
 
   DISALLOW_COPY_AND_ASSIGN(MockConnectionManager);
 };
 
 #endif  // CHROME_TEST_SYNC_ENGINE_MOCK_CONNECTION_MANAGER_H_