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

Unified Diff: chrome/browser/sync/internal_api/debug_info_event_listener.h

Issue 8189003: Send important client side event information to the server. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: committing. Created 9 years, 2 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: chrome/browser/sync/internal_api/debug_info_event_listener.h
diff --git a/chrome/browser/sync/internal_api/debug_info_event_listener.h b/chrome/browser/sync/internal_api/debug_info_event_listener.h
new file mode 100644
index 0000000000000000000000000000000000000000..6a1a2f1094b871633eb8241cb2745fb8d2cb5a6b
--- /dev/null
+++ b/chrome/browser/sync/internal_api/debug_info_event_listener.h
@@ -0,0 +1,71 @@
+// 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_BROWSER_SYNC_INTERNAL_API_DEBUG_INFO_EVENT_LISTENER_H_
+#define CHROME_BROWSER_SYNC_INTERNAL_API_DEBUG_INFO_EVENT_LISTENER_H_
+
+#include <string>
+
+#include "chrome/browser/sync/internal_api/sync_manager.h"
+#include "chrome/browser/sync/js/js_backend.h"
+#include "chrome/browser/sync/protocol/sync.pb.h"
+#include "chrome/browser/sync/sessions/debug_info_getter.h"
+#include "chrome/browser/sync/sessions/session_state.h"
+#include "chrome/browser/sync/util/weak_handle.h"
+#include "chrome/common/net/gaia/google_service_auth_error.h"
+
+namespace sync_api {
+
+const unsigned int kMaxEntries = 6;
+
+// Listens to events and records them in a queue. And passes the events to
+// syncer when requested.
+class DebugInfoEventListener : public sync_api::SyncManager::Observer,
+ public browser_sync::sessions::DebugInfoGetter {
+ public:
+ DebugInfoEventListener();
+ virtual ~DebugInfoEventListener();
+
+ // SyncManager::Observer implementation.
+ virtual void OnSyncCycleCompleted(
+ const browser_sync::sessions::SyncSessionSnapshot* snapshot) OVERRIDE;
+ virtual void OnInitializationComplete(
+ const browser_sync::WeakHandle<browser_sync::JsBackend>& js_backend,
+ bool success) OVERRIDE;
+ virtual void OnAuthError(
+ const GoogleServiceAuthError& auth_error) OVERRIDE;
+ virtual void OnPassphraseRequired(
+ sync_api::PassphraseRequiredReason reason) OVERRIDE;
+ virtual void OnPassphraseAccepted(
+ const std::string& bootstrap_token) OVERRIDE;
+ virtual void OnStopSyncingPermanently() OVERRIDE;
+ virtual void OnUpdatedToken(const std::string& token) OVERRIDE;
+ virtual void OnClearServerDataFailed() OVERRIDE;
+ virtual void OnClearServerDataSucceeded() OVERRIDE;
+ virtual void OnEncryptionComplete(
+ const syncable::ModelTypeSet& encrypted_types) OVERRIDE;
+ virtual void OnActionableError(
+ const browser_sync::SyncProtocolError& sync_error) OVERRIDE;
+
+ // DebugInfoGetter Implementation.
+ virtual void GetAndClearDebugInfo(sync_pb::DebugInfo* debug_info) OVERRIDE;
+
+ private:
+ FRIEND_TEST_ALL_PREFIXES(DebugInfoEventListenerTest, VerifyEventsAdded);
+ FRIEND_TEST_ALL_PREFIXES(DebugInfoEventListenerTest, VerifyQueueSize);
+ FRIEND_TEST_ALL_PREFIXES(DebugInfoEventListenerTest, VerifyGetAndClearEvents);
+
+ void AddEventToQueue(const sync_pb::DebugEventInfo& event_info);
+ void CreateAndAddEvent(sync_pb::DebugEventInfo::EventType type);
+ std::queue<sync_pb::DebugEventInfo> events_;
+
+ // True indicates we had to drop one or more events to keep our limit of
+ // |kMaxEntries|.
+ bool events_dropped_;
+
+ DISALLOW_COPY_AND_ASSIGN(DebugInfoEventListener);
+};
+
+} // namespace sync_api
+#endif // CHROME_BROWSER_SYNC_INTERNAL_API_DEBUG_INFO_EVENT_LISTENER_H_
« no previous file with comments | « chrome/browser/sync/engine/syncer_unittest.cc ('k') | chrome/browser/sync/internal_api/debug_info_event_listener.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698