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

Unified Diff: sync/engine/traffic_recorder.cc

Issue 12088080: [Sync] Add timestamp to TrafficRecorder records (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix comment Created 7 years, 11 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: sync/engine/traffic_recorder.cc
diff --git a/sync/engine/traffic_recorder.cc b/sync/engine/traffic_recorder.cc
index 379c488db8aaf41644172cc1f439802fe8c4645e..0e19aaaea1c5dcebb947a093bb7b173b86ec3edd 100644
--- a/sync/engine/traffic_recorder.cc
+++ b/sync/engine/traffic_recorder.cc
@@ -11,15 +11,23 @@
#include "sync/protocol/proto_value_conversions.h"
#include "sync/protocol/sync.pb.h"
#include "sync/sessions/sync_session.h"
+#include "sync/util/time.h"
namespace syncer {
+// Return current time.
+base::Time TrafficRecorder::GetTime() {
+ return base::Time::Now();
+}
+
TrafficRecorder::TrafficRecord::TrafficRecord(const std::string& message,
TrafficMessageType message_type,
- bool truncated) :
+ bool truncated,
+ base::Time time) :
message(message),
message_type(message_type),
- truncated(truncated) {
+ truncated(truncated),
+ timestamp(time) {
}
TrafficRecorder::TrafficRecord::TrafficRecord()
@@ -76,6 +84,8 @@ DictionaryValue* TrafficRecorder::TrafficRecord::ToValue() const {
NOTREACHED();
}
+ value->SetString("timestamp", GetTimeDebugString(timestamp));
+
return value.release();
}
@@ -115,7 +125,7 @@ void TrafficRecorder::StoreProtoInQueue(
msg.SerializeToString(&message);
}
- TrafficRecord record(message, type, truncated);
+ TrafficRecord record(message, type, truncated, GetTime());
AddTrafficToQueue(&record);
}

Powered by Google App Engine
This is Rietveld 408576698