Remove legacy GetURL(),SetURL(),GetFaviconBytes(),SetFaviconBytes()

sync/internal_api/public/write_node.h

 // Copyright (c) 2012 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 SYNC_INTERNAL_API_PUBLIC_WRITE_NODE_H_
 #define SYNC_INTERNAL_API_PUBLIC_WRITE_NODE_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 94 matching lines @@
 
   // Set a new parent and position.  Position is specified by |predecessor|; if
   // it is NULL, the node is moved to the first position.  |predecessor| must
   // be a child of |new_parent| or NULL.  Returns false on failure..
   bool SetPosition(const BaseNode& new_parent, const BaseNode* predecessor);
 
   // Set the bookmark specifics (url and favicon).
   // Should only be called if GetModelType() == BOOKMARK.
   void SetBookmarkSpecifics(const sync_pb::BookmarkSpecifics& specifics);
 
-  // Legacy, bookmark-specific setters that wrap SetBookmarkSpecifics() above.
-  // Should only be called if GetModelType() == BOOKMARK.
-  // TODO(ncarter): Remove these two datatype-specific accessors.
-  void SetURL(const GURL& url);
-  void SetFaviconBytes(const std::vector<unsigned char>& bytes);
-
   // Generic set specifics method. Will extract the model type from |specifics|.
   void SetEntitySpecifics(const sync_pb::EntitySpecifics& specifics);
 
   // Resets the EntitySpecifics for this node based on the unencrypted data.
   // Will encrypt if necessary.
   void ResetFromSpecifics();
 
   // TODO(sync): Remove the setters below when the corresponding data
   // types are ported to the new sync service API.
 
@@ skipping 67 matching lines @@
 
   // The sync API transaction that is the parent of this node.
   WriteTransaction* transaction_;
 
   DISALLOW_COPY_AND_ASSIGN(WriteNode);
 };
 
 }  // namespace syncer
 
 #endif  // SYNC_INTERNAL_API_PUBLIC_WRITE_NODE_H_