Remove syncproto.h

sync/engine/download_updates_command.cc

Index: sync/engine/download_updates_command.cc
diff --git a/sync/engine/download_updates_command.cc b/sync/engine/download_updates_command.cc
index 48e4acfebefde24711dfb5bad924dc10b7e6d80f..219d365a8bb407eb90a0ea826a629ffb3cefad61 100644
--- a/sync/engine/download_updates_command.cc
+++ b/sync/engine/download_updates_command.cc
@@ -9,7 +9,6 @@
 #include "base/command_line.h"
 #include "sync/engine/syncer.h"
 #include "sync/engine/syncer_proto_util.h"
-#include "sync/engine/syncproto.h"
 #include "sync/internal_api/public/base/model_type_payload_map.h"
 #include "sync/syncable/directory.h"
 
@@ -27,13 +26,15 @@ DownloadUpdatesCommand::DownloadUpdatesCommand(
 DownloadUpdatesCommand::~DownloadUpdatesCommand() {}
 
 SyncerError DownloadUpdatesCommand::ExecuteImpl(SyncSession* session) {
-  ClientToServerMessage client_to_server_message;
-  ClientToServerResponse update_response;
+  sync_pb::ClientToServerMessage client_to_server_message;
+  sync_pb::ClientToServerResponse update_response;
 
+  client_to_server_message.set_protocol_version(

[akalin, 2012/07/11 01:42:22]

hmm multiple calls?  Maybe put this call into PostClientToServerMessage?  I
guess that'd mean having to pass a pointer or copy the protobuf.

[rlarocque, 2012/07/11 19:22:16]

Done.  This has been converted to a function call and moved next to the
AddRequestBirthday() call, which performs a similar function: setting a required
field.

+      client_to_server_message.protocol_version());
   client_to_server_message.set_share(session->context()->account_name());
   client_to_server_message.set_message_contents(
-      ClientToServerMessage::GET_UPDATES);
-  GetUpdatesMessage* get_updates =
+      sync_pb::ClientToServerMessage::GET_UPDATES);
+  sync_pb::GetUpdatesMessage* get_updates =
       client_to_server_message.mutable_get_updates();
   get_updates->set_create_mobile_bookmarks_folder(
       create_mobile_bookmarks_folder_);