Collect profiler stats from browser child processes.

content/public/browser/profiler_subscriber.h

Index: content/public/browser/profiler_subscriber.h
===================================================================
--- content/public/browser/profiler_subscriber.h	(revision 0)
+++ content/public/browser/profiler_subscriber.h	(revision 0)
@@ -0,0 +1,29 @@
+// 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 CONTENT_PUBLIC_BROWSER_PROFILER_SUBSCRIBER_H_
+#define CONTENT_PUBLIC_BROWSER_PROFILER_SUBSCRIBER_H_
+
+#include "content/common/content_export.h"
+
+namespace base {
+class DictionaryValue;
+}
+
+// Objects interested in receiving profiler data derive from ProfilerSubscriber.
+// See also: profiler_message_filter.h
+// See also: child_profiler_message_filter.h

[jam, 2011/11/28 15:17:34]

nit: these two comments aren't necessary, someone can search quickly to get the
same information

[ramant (doing other things), 2011/11/29 01:32:20]

Done.

+class CONTENT_EXPORT ProfilerSubscriber {

[jam, 2011/11/28 15:17:34]

all classes in content/public have to be in the content namespace

[ramant (doing other things), 2011/11/29 01:32:20]

Done.

+ public:
+  virtual ~ProfilerSubscriber() {}
+
+  virtual void OnPendingProcesses(int sequence_number,

[jam, 2011/11/28 15:17:34]

nit: here and below, please add comments for the methods

[ramant (doing other things), 2011/11/29 01:32:20]

Done.

+                                  int pending_processes) = 0;
+
+  virtual void OnProfilerDataCollected(
+      int sequence_number,
+      const base::DictionaryValue& profiler_data) = 0;
+};
+
+#endif  // CONTENT_PUBLIC_BROWSER_PROFILER_SUBSCRIBER_H_
Property changes on: content\public\browser\profiler_subscriber.h
___________________________________________________________________
Added: svn:executable
   + *