[Sync] Remove --create-mobile-bookmarks-folder

chrome/browser/sync/engine/download_updates_command.h

Index: chrome/browser/sync/engine/download_updates_command.h
diff --git a/chrome/browser/sync/engine/download_updates_command.h b/chrome/browser/sync/engine/download_updates_command.h
index 3b12c714aa7fff42df0f1376dd6873eaa3f1169a..6494ecb39bb6f2f8622bafcb719f02d3e1a13e85 100644
--- a/chrome/browser/sync/engine/download_updates_command.h
+++ b/chrome/browser/sync/engine/download_updates_command.h
@@ -40,7 +40,10 @@ namespace browser_sync {
 // is encountered.
 class DownloadUpdatesCommand : public SyncerCommand {
  public:
-  DownloadUpdatesCommand();
+  // |create_mobile_bookmarks_folder| controls whether or not to
+  // create the mobile bookmarks folder if it's not already created.

[Nicolas Zea, 2012/02/27 22:35:23]

Since the folder is actually called "synced bookmarks", how about renaming to
that and having the description be something like
"controls whether or not to tell the server to create the synced bookmarks
permanent item if it's not already created". 

[akalin, 2012/02/27 22:56:44]

actually, the folder is called "Mobile Bookmarks" (I checked on my phone).  The
proto had 'synced bookmarks' only because it was checked in before Chrome on
Android was public.

+  // Should be set to true only by mobile clients.
+  explicit DownloadUpdatesCommand(bool create_mobile_bookmarks_folder);
   virtual ~DownloadUpdatesCommand();
 
   // SyncerCommand implementation.
@@ -50,6 +53,9 @@ class DownloadUpdatesCommand : public SyncerCommand {
   FRIEND_TEST_ALL_PREFIXES(DownloadUpdatesCommandTest, VerifyAppendDebugInfo);
   void AppendClientDebugInfoIfNeeded(sessions::SyncSession* session,
       sync_pb::DebugInfo* debug_info);
+
+  const bool create_mobile_bookmarks_folder_;
+
   DISALLOW_COPY_AND_ASSIGN(DownloadUpdatesCommand);
 };