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

Unified Diff: components/gcm_driver/gcm_stats_recorder_android.h

Issue 1515153003: Enable chrome://gcm-internals on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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: components/gcm_driver/gcm_stats_recorder_android.h
diff --git a/components/gcm_driver/gcm_stats_recorder_android.h b/components/gcm_driver/gcm_stats_recorder_android.h
new file mode 100644
index 0000000000000000000000000000000000000000..bfcb6723fbf855d560779a5181dfce37701ed463
--- /dev/null
+++ b/components/gcm_driver/gcm_stats_recorder_android.h
@@ -0,0 +1,75 @@
+// Copyright 2015 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 COMPONENTS_GCM_DRIVER_GCM_STATS_RECORDER_ANDROID_H_
+#define COMPONENTS_GCM_DRIVER_GCM_STATS_RECORDER_ANDROID_H_
+
+#include <deque>
+#include <string>
+
+#include "base/macros.h"
+#include "components/gcm_driver/gcm_activity.h"
+
+namespace gcm {
+
+class RecordedActivities;
+
+// Stats recorder for Android, used for recording stats and activities on the
+// GCM Driver level for debugging purposes. Based on the GCMStatsRecorder, as
+// defined in the GCM Engine, which does not exist on Android.
+class GCMStatsRecorderAndroid {
+ public:
+ // A delegate interface that allows the GCMStatsRecorderAndroid instance to
+ // interact with its container.
+ class Delegate {
+ public:
+ // Called when the GCMStatsRecorderAndroid is recording activities and a new
+ // activity has just been recorded.
+ virtual void OnActivityRecorded() = 0;
+ };
+
+ explicit GCMStatsRecorderAndroid(Delegate* delegate);
+ ~GCMStatsRecorderAndroid();
+
+ void SetRecording(bool recording) { is_recording_ = recording; }
Nicolas Zea 2015/12/16 22:24:32 nit: Generally if you're going to inline this meth
Peter Beverloo 2015/12/17 17:08:24 Done.
+
+ bool is_recording() const { return is_recording_; }
+
+ void Clear();
+
+ // Collect all recorded activities into |*recorder_activities|.
+ void CollectActivities(RecordedActivities* recorder_activities) const;
+
+ void RecordRegistrationSent(const std::string& app_id);
Nicolas Zea 2015/12/16 22:24:32 nit: would be good to have comments for these meth
Peter Beverloo 2015/12/17 17:08:24 Done.
+ void RecordRegistrationResponse(const std::string& app_id, bool success);
+
+ void RecordUnregistrationSent(const std::string& app_id);
+ void RecordUnregistrationResponse(const std::string& app_id, bool success);
+
+ void RecordDataMessageReceived(const std::string& app_id,
+ int message_byte_size);
+
+ private:
+ void RecordRegistration(const std::string& app_id,
+ const std::string& event,
+ const std::string& details);
+
+ // Delegate made available by the container. May be a nullptr.
+ Delegate* delegate_;
+
+ // Toggle determining whether the recorder is recording.
+ bool is_recording_ = false;
+
+ // Recorded registration activities (which includes unregistrations).
+ std::deque<RegistrationActivity> registration_activities_;
+
+ // Recorded received message activities.
+ std::deque<ReceivingActivity> receiving_activities_;
+
+ DISALLOW_COPY_AND_ASSIGN(GCMStatsRecorderAndroid);
+};
+
+} // namespace gcm
+
+#endif // COMPONENTS_GCM_DRIVER_GCM_STATS_RECORDER_ANDROID_H_

Powered by Google App Engine
This is Rietveld 408576698