[Sync] Add timestamp to TrafficRecorder records

sync/test/engine/fake_traffic_recorder.h

Index: sync/test/engine/fake_traffic_recorder.h
diff --git a/sync/test/engine/fake_traffic_recorder.h b/sync/test/engine/fake_traffic_recorder.h
new file mode 100644
index 0000000000000000000000000000000000000000..bd962a8bf55ed8031a270b3d915ecbdc2005119b
--- /dev/null
+++ b/sync/test/engine/fake_traffic_recorder.h
@@ -0,0 +1,30 @@
+// Copyright (c) 2012 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 SYNC_TEST_ENGINE_FAKE_TRAFFIC_RECORDER_H_
+#define SYNC_TEST_ENGINE_FAKE_TRAFFIC_RECORDER_H_
+
+#include "sync/engine/traffic_recorder.h"
+#include "base/time.h"
+
+namespace syncer {
+
+// Fake implementation of TrafficRecorder
+class FakeTrafficRecorder : public TrafficRecorder {

[rlarocque, 2013/01/31 19:13:10]

I'm not a big fan of this sort of inheritance.  Normally we would prefer
dependency-injection, ie. have a 'TimeGetter' class that we pass in to the
TrafficRecorder at construction time. 

However, in this case, the class being tested is so trivial and so unlikely to
become more complex that I can't see any harm coming from this.  Feel free to
leave it as is.  

+ public:
+  FakeTrafficRecorder(unsigned int max_messages, unsigned int max_message_size);
+  virtual ~FakeTrafficRecorder();
+
+  virtual base::Time GetTime() OVERRIDE;
+  void SetTime(int64 time);
+  void SetTime(base::Time time);
+
+ private:
+  base::Time time_;
+};
+
+}  // namespace syncer
+
+#endif  // SYNC_TEST_ENGINE_FAKE_TRAFFIC_RECORDER_H_
+