Performance tests for bookmark sync.

chrome/test/live_sync/live_sync_timing_helper.h

Index: chrome/test/live_sync/live_sync_timing_helper.h
diff --git a/chrome/test/live_sync/live_sync_timing_helper.h b/chrome/test/live_sync/live_sync_timing_helper.h
new file mode 100644
index 0000000000000000000000000000000000000000..0cfe4ac1efa12f76561a285614beaab25992203d
--- /dev/null
+++ b/chrome/test/live_sync/live_sync_timing_helper.h
@@ -0,0 +1,44 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_TEST_LIVE_SYNC_LIVE_SYNC_TIMING_HELPER_H_
+#define CHROME_TEST_LIVE_SYNC_LIVE_SYNC_TIMING_HELPER_H_
+#pragma once
+
+#include "base/basictypes.h"
+
+namespace base {
+  class TimeDelta;

[Raghu Simha, 2011/06/16 22:42:55]

indent -= 2.

[braffert, 2011/06/17 18:19:46]

Done.

+}
+
+class LiveSyncTest;
+class ProfileSyncServiceHarness;
+
+class LiveSyncTimingHelper {
+ public:
+  LiveSyncTimingHelper();
+  ~LiveSyncTimingHelper();
+
+  // Associates the timing helper object with a test.
+  void Setup(LiveSyncTest* test);

[Raghu Simha, 2011/06/16 22:42:55]

With the change I've suggested to TimeUntilQuiescence, we won't need this
method.

[braffert, 2011/06/17 18:19:46]

Done.

+
+  // Returns the time taken for |client| to complete a single sync cycle.
+  base::TimeDelta TimeSyncCycle(ProfileSyncServiceHarness* client);
+
+  // Returns the time taken for both |client| and partner to complete a sync

[Raghu Simha, 2011/06/16 22:42:55]

nit: partner -> |partner|.

[braffert, 2011/06/17 18:19:46]

Done.

+  // cycle.
+  base::TimeDelta TimeMutualSyncCycle(ProfileSyncServiceHarness* client,
+                                      ProfileSyncServiceHarness* partner);
+
+  // Returns the time taken for all clients to complete their respective sync
+  // cycles.
+  base::TimeDelta TimeUntilQuiescence();

[Raghu Simha, 2011/06/16 22:42:55]

I suggest making this method take a std::vector<ProfileSyncServiceHarness*>&
called |clients|, and then time the call to
ProfileSyncServiceHarness::AwaitQuiescence(clients) instead of
LiveSyncTest::AwaitQuiescence().

You can then make all the methods in this class static and the performance tests
can use this class as a helper class without having to each maintain a member
variable of type LiveSyncTimingHelper.

[braffert, 2011/06/17 18:19:46]

Done.

+
+ private:
+  LiveSyncTest* test_;

[Raghu Simha, 2011/06/16 22:42:55]

With the change I've suggested to TimeUntilQuiescence, we won't need this
member.

[braffert, 2011/06/17 18:19:46]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(LiveSyncTimingHelper);
+};
+
+#endif  // CHROME_TEST_LIVE_SYNC_LIVE_SYNC_TIMING_HELPER_H_