Sync performance test: Remove disabled benchmark tests.

chrome/test/live_sync/performance/autofill_sync_perf_test.cc

 // Copyright (c) 2011 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 "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/autofill/autofill_common_test.h"
 #include "chrome/browser/autofill/autofill_profile.h"
 #include "chrome/browser/sync/profile_sync_service_harness.h"
 #include "chrome/browser/webdata/autofill_entry.h"
 #include "chrome/test/live_sync/autofill_helper.h"
 #include "chrome/test/live_sync/live_sync_test.h"
 #include "chrome/test/live_sync/performance/sync_timing_helper.h"
 
 using autofill_helper::AllProfilesMatch;
 using autofill_helper::GetAllKeys;
 using autofill_helper::GetAllProfiles;
 using autofill_helper::GetKeyCount;
 using autofill_helper::GetProfileCount;
 using autofill_helper::RemoveKey;
 using autofill_helper::SetProfiles;
 
-// TODO(braffert): Move kNumBenchmarkPoints and kBenchmarkPoints for all
-// datatypes into a performance test base class, once it is possible to do so.
 static const int kNumKeys = 150;
 static const int kNumProfiles = 150;
-static const int kNumBenchmarkPoints = 18;
-static const int kBenchmarkPoints[] = {1, 10, 20, 30, 40, 50, 75, 100, 125,
-                                       150, 175, 200, 225, 250, 300, 350, 400,
-                                       500};
 
 class AutofillSyncPerfTest : public LiveSyncTest {
  public:
   AutofillSyncPerfTest()
       : LiveSyncTest(TWO_CLIENT),
         guid_number_(0),
         name_number_(0),
         value_number_(0) {}
 
   // Adds |num_profiles| new autofill profiles to the sync profile |profile|.
   void AddProfiles(int profile, int num_profiles);
 
   // Updates all autofill profiles for the sync profile |profile|.
   void UpdateProfiles(int profile);
 
   // Removes all autofill profiles from |profile|.
   void RemoveProfiles(int profile);
 
   // Adds |num_keys| new autofill keys to the sync profile |profile|.
   void AddKeys(int profile, int num_keys);
 
   // Removes all autofill keys from |profile|.
   void RemoveKeys(int profile);
 
-  // Removes all autofill profiles in all sync profiles.  Called between
-  // benchmark iterations.
-  void Cleanup();
-
  private:
   // Returns a new unique autofill profile.
   const AutofillProfile NextAutofillProfile();
 
   // Returns a new unique autofill key.
   const AutofillKey NextAutofillKey();
 
   // Returns an unused unique guid.
   const std::string NextGUID();
 
@@ skipping 57 matching lines @@
 }
 
 void AutofillSyncPerfTest::RemoveKeys(int profile) {
   std::set<AutofillEntry> keys = GetAllKeys(profile);
   for (std::set<AutofillEntry>::const_iterator it = keys.begin();
        it != keys.end(); ++it) {
     RemoveKey(profile, it->key());
   }
 }
 
-void AutofillSyncPerfTest::Cleanup() {
-  for (int i = 0; i < num_clients(); ++i) {
-    RemoveProfiles(i);
-  }
-  ASSERT_TRUE(AwaitQuiescence());
-}
-
 const AutofillProfile AutofillSyncPerfTest::NextAutofillProfile() {
   AutofillProfile profile;
   autofill_test::SetProfileInfoWithGuid(&profile, NextGUID().c_str(),
                                         NextName().c_str(), "", "", "", "", "",
                                         "", "", "", "", "", "", "");
   return profile;
 }
 
 const AutofillKey AutofillSyncPerfTest::NextAutofillKey() {
   return AutofillKey(NextName().c_str(), NextName().c_str());
@@ skipping 53 matching lines @@
   base::TimeDelta dt =
       SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
   ASSERT_EQ(kNumKeys, GetKeyCount(1));
   SyncTimingHelper::PrintResult("autofill", "add_autofill_keys", dt);
 
   RemoveKeys(0);
   dt = SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
   ASSERT_EQ(0, GetKeyCount(1));
   SyncTimingHelper::PrintResult("autofill", "delete_autofill_keys", dt);
 }
-
-IN_PROC_BROWSER_TEST_F(AutofillSyncPerfTest, DISABLED_Benchmark) {
-  ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
-
-  for (int i = 0; i < kNumBenchmarkPoints; ++i) {
-    int num_profiles = kBenchmarkPoints[i];
-    AddProfiles(0, num_profiles);
-    base::TimeDelta dt_add =
-        SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
-    ASSERT_EQ(num_profiles, GetProfileCount(0));
-    ASSERT_TRUE(AllProfilesMatch());
-    VLOG(0) << std::endl << "Add: " << num_profiles << " "
-            << dt_add.InSecondsF();
-
-    UpdateProfiles(0);
-    base::TimeDelta dt_update =
-        SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
-    ASSERT_EQ(num_profiles, GetProfileCount(0));
-    ASSERT_TRUE(AllProfilesMatch());
-    VLOG(0) << std::endl << "Update: " << num_profiles << " "
-            << dt_update.InSecondsF();
-
-    RemoveProfiles(0);
-    base::TimeDelta dt_delete =
-        SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
-    ASSERT_EQ(0, GetProfileCount(0));
-    ASSERT_TRUE(AllProfilesMatch());
-    VLOG(0) << std::endl << "Delete: " << num_profiles << " "
-            << dt_delete.InSecondsF();
-
-    Cleanup();
-  }
-}