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

Unified Diff: chrome/test/live_sync/performance/sessions_sync_perf_test.cc

Issue 7605012: Sync performance tests: include datatype in trace label for performance graphs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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/test/live_sync/performance/sessions_sync_perf_test.cc
diff --git a/chrome/test/live_sync/performance/sessions_sync_perf_test.cc b/chrome/test/live_sync/performance/sessions_sync_perf_test.cc
index d68b3d5265b03ba24640f4cf70c435d3c75348e9..19ea0344f9e8c7b70399904b8af12ca3fe419ffb 100644
--- a/chrome/test/live_sync/performance/sessions_sync_perf_test.cc
+++ b/chrome/test/live_sync/performance/sessions_sync_perf_test.cc
@@ -107,18 +107,18 @@ IN_PROC_BROWSER_TEST_F(SessionsSyncPerfTest, P0) {
SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
ASSERT_EQ(kNumTabs, GetTabCount(0));
ASSERT_EQ(kNumTabs, GetTabCount(1));
- SyncTimingHelper::PrintResult("tabs", "add", dt);
+ SyncTimingHelper::PrintResult("tabs", "add_tabs", dt);
UpdateTabs(0);
dt = SyncTimingHelper::TimeMutualSyncCycle(GetClient(0), GetClient(1));
ASSERT_EQ(kNumTabs, GetTabCount(0));
ASSERT_EQ(kNumTabs, GetTabCount(1));
- SyncTimingHelper::PrintResult("tabs", "update", dt);
+ SyncTimingHelper::PrintResult("tabs", "update_tabs", dt);
RemoveTabs(0);
dt = SyncTimingHelper::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", dt);
+ SyncTimingHelper::PrintResult("tabs", "delete_tabs", dt);
}

Powered by Google App Engine
This is Rietveld 408576698