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

chrome/browser/sync/engine/syncer.cc

Index: chrome/browser/sync/engine/syncer.cc
diff --git a/chrome/browser/sync/engine/syncer.cc b/chrome/browser/sync/engine/syncer.cc
index f64b030767d740777a4dc2c65eedeb92fce8404c..80c0d08b2d6ea4309406de82147c410fb1a4e473 100644
--- a/chrome/browser/sync/engine/syncer.cc
+++ b/chrome/browser/sync/engine/syncer.cc
@@ -9,6 +9,7 @@
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/time.h"
+#include "build/build_config.h"
 #include "chrome/browser/sync/engine/apply_updates_command.h"
 #include "chrome/browser/sync/engine/build_commit_command.h"
 #include "chrome/browser/sync/engine/cleanup_disabled_types_command.h"
@@ -141,7 +142,14 @@ void Syncer::SyncShare(sessions::SyncSession* session,
         break;
       }
       case DOWNLOAD_UPDATES: {
-        DownloadUpdatesCommand download_updates;
+        // TODO(akalin): We may want to propagate this switch up
+        // eventually.
+#if defined(OS_ANDROID)
+        const bool kIncludeSyncableBookmarks = true;

[Yaron, 2012/02/27 22:17:01]

You could start renaming this to kCreateMobileBookmarksFolder and propagating
that throughout. I assume it's also safe to rename the proto-field name, right?
(i.e. there's a unique id and the actual name isn't important)

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

Done.

+#else
+        const bool kIncludeSyncableBookmarks = false;
+#endif
+        DownloadUpdatesCommand download_updates(kIncludeSyncableBookmarks);
         session->mutable_status_controller()->set_last_download_updates_result(
             download_updates.Execute(session));
         next_step = PROCESS_CLIENT_COMMAND;