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

chrome/test/sync/engine/mock_connection_manager.cc

 // 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 regression tests.
 
 #include "chrome/test/sync/engine/mock_connection_manager.h"
 
 #include "chrome/browser/sync/engine/syncer_proto_util.h"
 #include "chrome/browser/sync/protocol/bookmark_specifics.pb.h"
@@ skipping 404 matching lines @@
                             name, version, sync_ts);
 }
 
 SyncEntity* MockConnectionManager::AddUpdateBookmark(
     syncable::Id id, syncable::Id parent_id, string name, int64 version,
     int64 sync_ts) {
   return AddUpdateBookmark(id.GetServerId(), parent_id.GetServerId(),
                            name, version, sync_ts);
 }
 
-void MockConnectionManager::AddUpdateExtendedAttributes(SyncEntity* ent,
-    string* xattr_key, syncable::Blob* xattr_value, int xattr_count) {
-  sync_pb::ExtendedAttributes* mutable_extended_attributes =
-      ent->mutable_extended_attributes();
-  for (int i = 0; i < xattr_count; i++) {
-    sync_pb::ExtendedAttributes_ExtendedAttribute* extended_attribute =
-        mutable_extended_attributes->add_extendedattribute();
-    extended_attribute->set_key(xattr_key[i]);
-    SyncerProtoUtil::CopyBlobIntoProtoBytes(xattr_value[i],
-        extended_attribute->mutable_value());
-  }
-}
-
 SyncEntity* MockConnectionManager::GetMutableLastUpdate() {
   DCHECK(updates_.entries_size() > 0);
   return updates_.mutable_entries()->Mutable(updates_.entries_size() - 1);
 }
 
 const CommitMessage& MockConnectionManager::last_sent_commit() const {
   DCHECK(!commit_messages_.empty());
   return *commit_messages_.back();
 }
 
@@ skipping 24 matching lines @@
 bool MockConnectionManager::IsModelTypePresentInSpecifics(
     const sync_pb::EntitySpecifics& filter, syncable::ModelType value) {
   // This implementation is a little contorted; it's done this way
   // to avoid having to switch on the ModelType.  We're basically doing
   // the protobuf equivalent of ((value & filter) == filter).
   sync_pb::EntitySpecifics value_filter;
   syncable::AddDefaultExtensionValue(value, &value_filter);
   value_filter.MergeFrom(filter);
   return value_filter.SerializeAsString() == filter.SerializeAsString();
 }