[Sync] Don't set old style bookmarks anymore.

chrome/browser/sync/engine/build_commit_command.cc

 // Copyright (c) 2010 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.
 
 #include "chrome/browser/sync/engine/build_commit_command.h"
 
 #include <set>
 #include <string>
 #include <vector>
 
@@ skipping 159 matching lines @@
     // Deletion is final on the server, let's move things and then delete them.
     if (meta_entry.Get(IS_DEL)) {
       sync_entry->set_deleted(true);
     } else {
       if (meta_entry.Get(SPECIFICS).HasExtension(sync_pb::bookmark)) {
         // Common data in both new and old protocol.
         const Id& prev_id = meta_entry.Get(syncable::PREV_ID);
         string prev_id_string =
             prev_id.IsRoot() ? string() : prev_id.GetServerId();
         sync_entry->set_insert_after_item_id(prev_id_string);
-
-        // TODO(ncarter): In practice we won't want to send this data twice
-        // over the wire; instead, when deployed servers are able to accept
-        // the new-style scheme, we should abandon the old way.
-        SetOldStyleBookmarkData(&meta_entry, sync_entry);
       }
       SetEntrySpecifics(&meta_entry, sync_entry);
     }
   }
   session->status_controller()->mutable_commit_message()->CopyFrom(message);
 }
 
 }  // namespace browser_sync