Sync performance tests: take timings with autofill keys in addition to profiles.

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) {}
+        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 bookmarks in the bookmark bar for |profile|.
+  // Removes all autofill profiles for |profile|.

[Raghu Simha, 2011/08/10 19:43:38]

nit: s/for/from/

[braffert, 2011/08/10 19:52:35]

Done.

   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 for |profile|.

[Raghu Simha, 2011/08/10 19:43:38]

nit: s/for/from/

[braffert, 2011/08/10 19:52:35]

Done.

+  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();
 
   // Returns a unique guid based on the input integer |n|.
   const std::string IntToGUID(int n);
 
   // Returns a new unused unique name.
   const std::string NextName();
 
   // Returns a unique name based on the input integer |n|.
   const std::string IntToName(int n);
 
+  // Returns a new unused unique value for autofill entries.
+  const std::string NextValue();
+
+  // Returnes a unique value based on the input integer |n|.
+  const std::string IntToValue(int n);
+
   int guid_number_;
   int name_number_;
+  int value_number_;
   DISALLOW_COPY_AND_ASSIGN(AutofillSyncPerfTest);
 };
 
 void AutofillSyncPerfTest::AddProfiles(int profile, int num_profiles) {
   const std::vector<AutofillProfile*>& all_profiles =
       GetAllProfiles(profile);
   std::vector<AutofillProfile> autofill_profiles;
   for (size_t i = 0; i < all_profiles.size(); ++i) {
     autofill_profiles.push_back(*all_profiles[i]);
   }
@@ skipping 13 matching lines @@
                                      UTF8ToUTF16(NextName()));
   }
   SetProfiles(profile, &autofill_profiles);
 }
 
 void AutofillSyncPerfTest::RemoveProfiles(int profile) {
   std::vector<AutofillProfile> empty;
   SetProfiles(profile, &empty);
 }
 
+void AutofillSyncPerfTest::AddKeys(int profile, int num_profiles) {

[Raghu Simha, 2011/08/10 19:43:38]

s/num_profiles/num_keys/

[braffert, 2011/08/10 19:52:35]

Done.

+  std::set<AutofillKey> keys;
+  for (int i = 0; i < num_profiles; ++i) {
+    keys.insert(NextAutofillKey());
+  };

[Raghu Simha, 2011/08/10 19:43:38]

Remove semicolon.

[braffert, 2011/08/10 19:52:35]

Done.  Don't know how that got in there.

+  autofill_helper::AddKeys(profile, keys);
+}
+
+void AutofillSyncPerfTest::RemoveKeys(int profile) {

[Raghu Simha, 2011/08/10 19:43:38]

Perhaps name this RemoveAllKeys, since that's what it's doing?

[braffert, 2011/08/10 19:52:35]

This is a good suggestion, but I will leave it as is for now so that it matches
the pattern established by the other datatypes.  Any change should be made
across the board (RemoveAllBookmarks, UpdateAllExtensions ...)

+  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());
+}
+
 const std::string AutofillSyncPerfTest::NextGUID() {
   return IntToGUID(guid_number_++);
 }
 
 const std::string AutofillSyncPerfTest::IntToGUID(int n) {
   return StringPrintf("00000000-0000-0000-0000-%012X", n);
 }
 
 const std::string AutofillSyncPerfTest::NextName() {
   return IntToName(name_number_++);
 }
 
 const std::string AutofillSyncPerfTest::IntToName(int n) {
-  return StringPrintf("Name%d" , n);
+  return StringPrintf("Name%d", n);
 }
 
-IN_PROC_BROWSER_TEST_F(AutofillSyncPerfTest, P0) {
+const std::string AutofillSyncPerfTest::NextValue() {
+  return IntToValue(value_number_++);
+}
+
+const std::string AutofillSyncPerfTest::IntToValue(int n) {
+  return StringPrintf("Value%d", n);
+}
+
+IN_PROC_BROWSER_TEST_F(AutofillSyncPerfTest, AutofillProfiles_P0) {
   ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
 
   // TCM ID - 7557873.
   AddProfiles(0, kNumProfiles);
   base::TimeDelta dt =
       SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
   ASSERT_EQ(kNumProfiles, GetProfileCount(1));
-  SyncTimingHelper::PrintResult("autofill", "add", dt);
+  SyncTimingHelper::PrintResult("autofill", "add_autofill_profiles", dt);
 
   // TCM ID - 7549835.
   UpdateProfiles(0);
   dt = SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
   ASSERT_EQ(kNumProfiles, GetProfileCount(1));
-  SyncTimingHelper::PrintResult("autofill", "update", dt);
+  SyncTimingHelper::PrintResult("autofill", "update_autofill_profiles", dt);
 
   // TCM ID - 7553678.
   RemoveProfiles(0);
   dt = SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
   ASSERT_EQ(0, GetProfileCount(1));
-  SyncTimingHelper::PrintResult("autofill", "delete", dt);
+  SyncTimingHelper::PrintResult("autofill", "delete_autofill_profiles", dt);
+}
+
+IN_PROC_BROWSER_TEST_F(AutofillSyncPerfTest, Autofill_P0) {

[Raghu Simha, 2011/08/10 19:43:38]

Perhaps name this test AutofillKeys_P0?

[braffert, 2011/08/10 19:52:35]

Anna suggested using only Autofill as the name here - I believe so that it
matches the naming used in TCM.

+  ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
+
+  AddKeys(0, kNumKeys);
+  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));
@@ skipping 14 matching lines @@
     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();
   }
 }