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

Unified Diff: sync/engine/traffic_recorder.h

Issue 217633003: sync: Remove TrafficRecorder (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 9 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
« no previous file with comments | « sync/engine/syncer_unittest.cc ('k') | sync/engine/traffic_recorder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/engine/traffic_recorder.h
diff --git a/sync/engine/traffic_recorder.h b/sync/engine/traffic_recorder.h
deleted file mode 100644
index 55ee0b3ed7094927936454a54d1aa3037c724e4f..0000000000000000000000000000000000000000
--- a/sync/engine/traffic_recorder.h
+++ /dev/null
@@ -1,86 +0,0 @@
-// Copyright 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 CHROME_BROWSER_SYNC_ENGINE_TRAFFIC_RECORDER_H_
-#define CHROME_BROWSER_SYNC_ENGINE_TRAFFIC_RECORDER_H_
-
-#include <deque>
-#include <string>
-
-#include "base/basictypes.h"
-#include "base/gtest_prod_util.h"
-#include "base/time/time.h"
-#include "base/values.h"
-#include "sync/base/sync_export.h"
-#include "sync/protocol/sync.pb.h"
-
-namespace sync_pb {
-class ClientToServerResponse;
-class ClientToServerMessage;
-}
-
-namespace syncer {
-
-class SYNC_EXPORT_PRIVATE TrafficRecorder {
- public:
- enum TrafficMessageType {
- CLIENT_TO_SERVER_MESSAGE,
- CLIENT_TO_SERVER_RESPONSE,
- UNKNOWN_MESSAGE_TYPE
- };
-
- struct SYNC_EXPORT_PRIVATE TrafficRecord {
- // The serialized message.
- std::string message;
- TrafficMessageType message_type;
- // If the message is too big to be kept in memory then it should be
- // truncated. For now the entire message is omitted if it is too big.
- // TODO(lipalani): Truncate the specifics to fit within size.
- bool truncated;
-
- TrafficRecord(const std::string& message,
- TrafficMessageType message_type,
- bool truncated,
- base::Time time);
- TrafficRecord();
- ~TrafficRecord();
- base::DictionaryValue* ToValue() const;
-
- // Time of record creation.
- base::Time timestamp;
- };
-
- TrafficRecorder(unsigned int max_messages, unsigned int max_message_size);
- virtual ~TrafficRecorder();
-
- void RecordClientToServerMessage(const sync_pb::ClientToServerMessage& msg);
- void RecordClientToServerResponse(
- const sync_pb::ClientToServerResponse& response);
- base::ListValue* ToValue() const;
-
- const std::deque<TrafficRecord>& records() {
- return records_;
- }
-
- private:
- void AddTrafficToQueue(TrafficRecord* record);
- void StoreProtoInQueue(const ::google::protobuf::MessageLite& msg,
- TrafficMessageType type);
-
- // Method to get record creation time.
- virtual base::Time GetTime();
-
- // Maximum number of messages stored in the queue.
- unsigned int max_messages_;
-
- // Maximum size of each message.
- unsigned int max_message_size_;
- std::deque<TrafficRecord> records_;
- DISALLOW_COPY_AND_ASSIGN(TrafficRecorder);
-};
-
-} // namespace syncer
-
-#endif // CHROME_BROWSER_SYNC_ENGINE_TRAFFIC_RECORDER_H_
-
« no previous file with comments | « sync/engine/syncer_unittest.cc ('k') | sync/engine/traffic_recorder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698