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

Unified Diff: content/browser/profiler_message_filter.h

Issue 9702014: [UMA] Use proper C++ objects to serialize tracked_objects across process boundaries. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Make sure to initialize all primitive fields Created 8 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
Index: content/browser/profiler_message_filter.h
diff --git a/content/browser/profiler_message_filter.h b/content/browser/profiler_message_filter.h
index 79e6e942fef4097ff9e112fce80bc8b62c90241b..3477aaa00a1931de3826102dca5462bd299c7133 100644
--- a/content/browser/profiler_message_filter.h
+++ b/content/browser/profiler_message_filter.h
@@ -8,15 +8,18 @@
#include <string>
#include "content/public/browser/browser_message_filter.h"
+#include "content/public/common/process_type.h"
-namespace base {
-class DictionaryValue;
+namespace tracked_objects {
+struct SerializedProcessData;
}
+namespace content {
+
// This class sends and receives profiler messages in the browser process.
-class ProfilerMessageFilter : public content::BrowserMessageFilter {
+class ProfilerMessageFilter : public BrowserMessageFilter {
public:
- ProfilerMessageFilter();
+ explicit ProfilerMessageFilter(ProcessType process_type);
virtual ~ProfilerMessageFilter();
// content::BrowserMessageFilter implementation.
@@ -28,11 +31,15 @@ class ProfilerMessageFilter : public content::BrowserMessageFilter {
private:
// Message handlers.
- void OnChildProfilerData(int sequence_number,
- const base::DictionaryValue& profiler_data);
+ void OnChildProfilerData(
+ int sequence_number,
+ const tracked_objects::SerializedProcessData& profiler_data);
+
+ ProcessType process_type_;
DISALLOW_COPY_AND_ASSIGN(ProfilerMessageFilter);
};
-#endif // CONTENT_BROWSER_PROFILER_MESSAGE_FILTER_H_
+} // namespace content
+#endif // CONTENT_BROWSER_PROFILER_MESSAGE_FILTER_H_

Powered by Google App Engine
This is Rietveld 408576698