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

sync/internal_api/public/write_node.h

Index: sync/internal_api/public/write_node.h
diff --git a/sync/internal_api/public/write_node.h b/sync/internal_api/public/write_node.h
index 7844bb43bc70a2da53bb65d175b8f6bfb2627006..b2299f627426205d234279c98922cb3319ff7fd4 100644
--- a/sync/internal_api/public/write_node.h
+++ b/sync/internal_api/public/write_node.h
@@ -112,12 +112,6 @@ class WriteNode : public BaseNode {
   // 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);