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

Unified Diff: chrome/browser/sync/test/integration/performance/sessions_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/sessions_sync_perf_test.cc
diff --git a/chrome/browser/sync/test/integration/performance/sessions_sync_perf_test.cc b/chrome/browser/sync/test/integration/performance/sessions_sync_perf_test.cc
index e2b8d47ad7d374857959e1cc61c348f09c7eded2..e56356372e7238f503214831c6a9c4afa3fbbd0d 100644
--- a/chrome/browser/sync/test/integration/performance/sessions_sync_perf_test.cc
+++ b/chrome/browser/sync/test/integration/performance/sessions_sync_perf_test.cc
@@ -19,6 +19,8 @@ using sessions_helper::OpenMultipleTabs;
using sessions_helper::SyncedSessionVector;
using sessions_helper::SessionWindowMap;
using sessions_helper::WaitForTabsToLoad;
+using sync_timing_helper::PrintResult;
+using sync_timing_helper::TimeMutualSyncCycle;
static const int kNumTabs = 150;
@@ -114,22 +116,21 @@ IN_PROC_BROWSER_TEST_F(SessionsSyncPerfTest, DISABLED_P0) {
ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
AddTabs(0, kNumTabs);
- base::TimeDelta dt =
- SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
+ base::TimeDelta dt = TimeMutualSyncCycle(GetClient(0), GetClient(1));
ASSERT_EQ(kNumTabs, GetTabCount(0));
ASSERT_EQ(kNumTabs, GetTabCount(1));
- SyncTimingHelper::PrintResult("tabs", "add_tabs", dt);
+ PrintResult("tabs", "add_tabs", dt);
UpdateTabs(0);
- dt = SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
+ dt = TimeMutualSyncCycle(GetClient(0), GetClient(1));
ASSERT_EQ(kNumTabs, GetTabCount(0));
ASSERT_EQ(kNumTabs, GetTabCount(1));
- SyncTimingHelper::PrintResult("tabs", "update_tabs", dt);
+ PrintResult("tabs", "update_tabs", dt);
RemoveTabs(0);
- dt = SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
+ dt = TimeMutualSyncCycle(GetClient(0), GetClient(1));
// New tab page remains open on profile 0 after closing all tabs.
ASSERT_EQ(1, GetTabCount(0));
ASSERT_EQ(0, GetTabCount(1));
- SyncTimingHelper::PrintResult("tabs", "delete_tabs", dt);
+ PrintResult("tabs", "delete_tabs", dt);
}

Powered by Google App Engine
This is Rietveld 408576698