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

Side by Side Diff: chrome/browser/sync/test/integration/performance/sync_timing_helper.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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/sync/test/integration/performance/sync_timing_helper.h" 5 #include "chrome/browser/sync/test/integration/performance/sync_timing_helper.h"
6 6
7 #include "base/strings/string_number_conversions.h" 7 #include "base/strings/string_number_conversions.h"
8 #include "base/time/time.h" 8 #include "base/time/time.h"
9 #include "chrome/browser/sync/test/integration/profile_sync_service_harness.h" 9 #include "chrome/browser/sync/test/integration/profile_sync_service_harness.h"
10 #include "chrome/browser/sync/test/integration/sync_integration_test_util.h" 10 #include "chrome/browser/sync/test/integration/updated_progress_marker_checker.h "
11 #include "testing/gtest/include/gtest/gtest.h" 11 #include "testing/gtest/include/gtest/gtest.h"
12 12
13 using sync_integration_test_util::AwaitCommitActivityCompletion; 13 namespace sync_timing_helper {
14 14
15 SyncTimingHelper::SyncTimingHelper() {} 15 base::TimeDelta TimeSyncCycle(ProfileSyncServiceHarness* client) {
16
17 SyncTimingHelper::~SyncTimingHelper() {}
18
19 // static
20 base::TimeDelta SyncTimingHelper::TimeSyncCycle(
21 ProfileSyncServiceHarness* client) {
22 base::Time start = base::Time::Now(); 16 base::Time start = base::Time::Now();
23 EXPECT_TRUE(AwaitCommitActivityCompletion(client->service())); 17 EXPECT_TRUE(UpdatedProgressMarkerChecker(client->service()).Wait());
24 return base::Time::Now() - start; 18 return base::Time::Now() - start;
25 } 19 }
26 20
27 // static 21 base::TimeDelta TimeMutualSyncCycle(ProfileSyncServiceHarness* client,
28 base::TimeDelta SyncTimingHelper::TimeMutualSyncCycle( 22 ProfileSyncServiceHarness* partner) {
29 ProfileSyncServiceHarness* client, ProfileSyncServiceHarness* partner) {
30 base::Time start = base::Time::Now(); 23 base::Time start = base::Time::Now();
31 EXPECT_TRUE(client->AwaitMutualSyncCycleCompletion(partner)); 24 EXPECT_TRUE(client->AwaitMutualSyncCycleCompletion(partner));
32 return base::Time::Now() - start; 25 return base::Time::Now() - start;
33 } 26 }
34 27
35 // static 28 base::TimeDelta TimeUntilQuiescence(
36 base::TimeDelta SyncTimingHelper::TimeUntilQuiescence( 29 const std::vector<ProfileSyncServiceHarness*>& clients) {
37 std::vector<ProfileSyncServiceHarness*>& clients) {
38 base::Time start = base::Time::Now(); 30 base::Time start = base::Time::Now();
39 EXPECT_TRUE(ProfileSyncServiceHarness::AwaitQuiescence(clients)); 31 EXPECT_TRUE(ProfileSyncServiceHarness::AwaitQuiescence(clients));
40 return base::Time::Now() - start; 32 return base::Time::Now() - start;
41 } 33 }
42 34
43 // static 35 void PrintResult(const std::string& measurement,
44 void SyncTimingHelper::PrintResult(const std::string& measurement, 36 const std::string& trace,
45 const std::string& trace, 37 const base::TimeDelta& dt) {
46 const base::TimeDelta& dt) {
47 printf("*RESULT %s: %s= %s ms\n", measurement.c_str(), trace.c_str(), 38 printf("*RESULT %s: %s= %s ms\n", measurement.c_str(), trace.c_str(),
48 base::Int64ToString(dt.InMilliseconds()).c_str()); 39 base::Int64ToString(dt.InMilliseconds()).c_str());
49 } 40 }
41
42 } // namespace sync_timing_helper
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698