Sync performance tests: Cleanup TODOs and print in dashboard friendly format

chrome/test/live_sync/performance/sync_timing_helper.h

Index: chrome/test/live_sync/performance/sync_timing_helper.h
diff --git a/chrome/test/live_sync/performance/sync_timing_helper.h b/chrome/test/live_sync/performance/sync_timing_helper.h
index c45b3676fd5924d41ce0223e76b637f036292fad..bcad3f55394ea3c3257657c9cb1d7ecfa3e2b008 100644
--- a/chrome/test/live_sync/performance/sync_timing_helper.h
+++ b/chrome/test/live_sync/performance/sync_timing_helper.h
@@ -8,6 +8,7 @@
 
 #include "base/basictypes.h"
 
+#include <string>
 #include <vector>
 
 namespace base {
@@ -34,6 +35,13 @@ class SyncTimingHelper {
   static base::TimeDelta TimeUntilQuiescence(
       std::vector<ProfileSyncServiceHarness*>& clients);
 
+  // Print a timing measurement in a format appropriate for the chromium perf
+  // dashboard.  Simplified version of methods defined in
+  // chrome/test/ui/ui_perf_test.{h,cc}.
+  static void PrintResult(const std::string& measurement,
+                   const std::string& trace,
+                   const base::TimeDelta& dt);

[Raghu Simha, 2011/07/26 06:19:45]

nit: Style guide recommends aligning parameters, one in each line.

[braffert, 2011/07/26 17:46:59]

Done.  I added static to the first line and forgot to adjust the rest.

+
  private:
   DISALLOW_COPY_AND_ASSIGN(SyncTimingHelper);
 };