Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(211)

Unified Diff: chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc

Issue 2379433002: [Sync] Refactoring of sync integration test checkers to remove boilerplate await methods. (Closed)
Patch Set: Rebase Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc
diff --git a/chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc b/chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc
index bc7acc10ea5ba2d009df60b7c85cd5bec2b9b3e6..5e8be39cd31d44cbb6bad8a2f1760e29d32b8e03 100644
--- a/chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc
+++ b/chrome/browser/sync/test/integration/performance/typed_urls_sync_perf_test.cc
@@ -13,7 +13,8 @@
using typed_urls_helper::AddUrlToHistory;
using typed_urls_helper::DeleteUrlsFromHistory;
using typed_urls_helper::GetTypedUrlsFromClient;
-
+using sync_timing_helper::PrintResult;
+using sync_timing_helper::TimeMutualSyncCycle;
// This number should be as far away from a multiple of
// kDefaultMaxCommitBatchSize as possible, so that sync cycle counts
// for batch operations stay the same even if some batches end up not
@@ -98,20 +99,19 @@ IN_PROC_BROWSER_TEST_F(TypedUrlsSyncPerfTest, P0) {
// TCM ID - 7985716.
AddURLs(0, kNumUrls);
- base::TimeDelta dt =
- SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
+ base::TimeDelta dt = TimeMutualSyncCycle(GetClient(0), GetClient(1));
ASSERT_EQ(kNumUrls, GetURLCount(1));
- SyncTimingHelper::PrintResult("typed_urls", "add_typed_urls", dt);
+ PrintResult("typed_urls", "add_typed_urls", dt);
// TCM ID - 7981755.
UpdateURLs(0);
- dt = SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
+ dt = TimeMutualSyncCycle(GetClient(0), GetClient(1));
ASSERT_EQ(kNumUrls, GetURLCount(1));
- SyncTimingHelper::PrintResult("typed_urls", "update_typed_urls", dt);
+ PrintResult("typed_urls", "update_typed_urls", dt);
// TCM ID - 7651271.
RemoveURLs(0);
- dt = SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
+ dt = TimeMutualSyncCycle(GetClient(0), GetClient(1));
ASSERT_EQ(0, GetURLCount(1));
- SyncTimingHelper::PrintResult("typed_urls", "delete_typed_urls", dt);
+ PrintResult("typed_urls", "delete_typed_urls", dt);
}

Powered by Google App Engine
This is Rietveld 408576698