| Index: chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc
|
| diff --git a/chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc b/chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc
|
| index 3e3bf57280b81a8ec49b1cf496dae8f60f81a59c..08e400dc822a59b4afcde487cf34285d58cd89a9 100644
|
| --- a/chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc
|
| +++ b/chrome/test/live_sync/performance/bookmarks_sync_perf_test.cc
|
| @@ -15,13 +15,7 @@ using bookmarks_helper::IndexedURLTitle;
|
| using bookmarks_helper::Remove;
|
| using bookmarks_helper::SetURL;
|
|
|
| -// 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 kNumBookmarks = 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 BookmarksSyncPerfTest : public LiveSyncTest {
|
| public:
|
| @@ -42,10 +36,6 @@ class BookmarksSyncPerfTest : public LiveSyncTest {
|
| // Returns the number of bookmarks stored in the bookmark bar for |profile|.
|
| int GetURLCount(int profile);
|
|
|
| - // Remvoes all bookmarks in the bookmark bars for all profiles. Called
|
| - // between benchmark iterations.
|
| - void Cleanup();
|
| -
|
| private:
|
| // Returns a new unique bookmark URL.
|
| std::string NextIndexedURL();
|
| @@ -85,15 +75,6 @@ int BookmarksSyncPerfTest::GetURLCount(int profile) {
|
| return GetBookmarkBarNode(profile)->child_count();
|
| }
|
|
|
| -void BookmarksSyncPerfTest::Cleanup() {
|
| - for (int i = 0; i < num_clients(); ++i) {
|
| - RemoveURLs(i);
|
| - }
|
| - ASSERT_TRUE(AwaitQuiescence());
|
| - ASSERT_EQ(0, GetBookmarkBarNode(0)->child_count());
|
| - ASSERT_TRUE(AllModelsMatch());
|
| -}
|
| -
|
| std::string BookmarksSyncPerfTest::NextIndexedURL() {
|
| return IndexedURL(url_number_++);
|
| }
|
| @@ -124,37 +105,3 @@ IN_PROC_BROWSER_TEST_F(BookmarksSyncPerfTest, P0) {
|
| ASSERT_EQ(0, GetURLCount(1));
|
| SyncTimingHelper::PrintResult("bookmarks", "delete_bookmarks", dt);
|
| }
|
| -
|
| -IN_PROC_BROWSER_TEST_F(BookmarksSyncPerfTest, DISABLED_Benchmark) {
|
| - ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
|
| - DisableVerifier();
|
| -
|
| - for (int i = 0; i < kNumBenchmarkPoints; ++i) {
|
| - int num_bookmarks = kBenchmarkPoints[i];
|
| - AddURLs(0, num_bookmarks);
|
| - base::TimeDelta dt_add =
|
| - SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
|
| - ASSERT_EQ(num_bookmarks, GetBookmarkBarNode(0)->child_count());
|
| - ASSERT_TRUE(AllModelsMatch());
|
| - VLOG(0) << std::endl << "Add: " << num_bookmarks << " "
|
| - << dt_add.InSecondsF();
|
| -
|
| - UpdateURLs(0);
|
| - base::TimeDelta dt_update =
|
| - SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
|
| - ASSERT_EQ(num_bookmarks, GetBookmarkBarNode(0)->child_count());
|
| - ASSERT_TRUE(AllModelsMatch());
|
| - VLOG(0) << std::endl << "Update: " << num_bookmarks << " "
|
| - << dt_update.InSecondsF();
|
| -
|
| - RemoveURLs(0);
|
| - base::TimeDelta dt_delete =
|
| - SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
|
| - ASSERT_EQ(0, GetBookmarkBarNode(0)->child_count());
|
| - ASSERT_TRUE(AllModelsMatch());
|
| - VLOG(0) << std::endl << "Delete: " << num_bookmarks << " "
|
| - << dt_delete.InSecondsF();
|
| -
|
| - Cleanup();
|
| - }
|
| -}
|
|
|