[Sync] Have bookmarks use a slower nudge delay to better handle automated writes

sync/internal_api/sync_manager_impl_unittest.cc

 // Copyright 2012 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.
 
 // Unit tests for the SyncApi. Note that a lot of the underlying
 // functionality is provided by the Syncable layer, which has its own
 // unit tests. We'll test SyncApi specific things in this harness.
 
 #include <cstddef>
 #include <map>
@@ skipping 1592 matching lines @@
   }
   {
     ReadTransaction trans(FROM_HERE, sync_manager_.GetUserShare());
     ReadNode password_node(&trans);
     EXPECT_EQ(BaseNode::INIT_FAILED_DECRYPT_IF_NECESSARY,
               password_node.InitByIdLookup(node_id));
   }
 }
 
 TEST_F(SyncManagerTest, NudgeDelayTest) {
-  EXPECT_EQ(sync_manager_.GetNudgeDelayTimeDelta(BOOKMARKS),
-      base::TimeDelta::FromMilliseconds(
-          SyncManagerImpl::GetDefaultNudgeDelay()));
+  EXPECT_EQ(
+      sync_manager_.GetNudgeDelayTimeDelta(BOOKMARKS),
+      base::TimeDelta::FromMilliseconds(SyncManagerImpl::GetSlowNudgeDelay()));
 
   EXPECT_EQ(sync_manager_.GetNudgeDelayTimeDelta(AUTOFILL),
-      base::TimeDelta::FromSeconds(
-          kDefaultShortPollIntervalSeconds));
+            base::TimeDelta::FromSeconds(kDefaultShortPollIntervalSeconds));
 
-  EXPECT_EQ(sync_manager_.GetNudgeDelayTimeDelta(PREFERENCES),
-      base::TimeDelta::FromMilliseconds(
-          SyncManagerImpl::GetPreferencesNudgeDelay()));
+  EXPECT_EQ(
+      sync_manager_.GetNudgeDelayTimeDelta(PREFERENCES),
+      base::TimeDelta::FromMilliseconds(SyncManagerImpl::GetSlowNudgeDelay()));
+
+  EXPECT_EQ(sync_manager_.GetNudgeDelayTimeDelta(EXTENSIONS),
+            base::TimeDelta::FromMilliseconds(
+                SyncManagerImpl::GetDefaultNudgeDelay()));
 }
 
 // Friended by WriteNode, so can't be in an anonymouse namespace.
 TEST_F(SyncManagerTest, EncryptBookmarksWithLegacyData) {
   EXPECT_TRUE(SetUpEncryption(WRITE_TO_NIGORI, DEFAULT_ENCRYPTION));
   std::string title;
   SyncAPINameToServerName("Google", &title);
   std::string url = "http://www.google.com";
   std::string raw_title2 = "..";  // An invalid cosmo title.
   std::string title2;
@@ skipping 1560 matching lines @@
 // SyncManagerInitInvalidStorageTest::GetFactory will return
 // DirectoryBackingStore that ensures that SyncManagerImpl::OpenDirectory fails.
 // SyncManagerImpl initialization is done in SyncManagerTest::SetUp. This test's
 // task is to ensure that SyncManagerImpl reported initialization failure in
 // OnInitializationComplete callback.
 TEST_F(SyncManagerInitInvalidStorageTest, FailToOpenDatabase) {
   EXPECT_FALSE(initialization_succeeded_);
 }
 
 }  // namespace syncer