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

Unified Diff: content/public/browser/child_process_data.h

Issue 972083002: Report utility process JS memory in task manager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@v8-pac-oop
Patch Set: Clean up. Created 5 years, 8 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/public/browser/child_process_data.h
diff --git a/content/public/browser/child_process_data.h b/content/public/browser/child_process_data.h
index 395c4876b8d736e4de46b11a71970bac95a2e818..c796895fb831a4d01dd44271a0d042f5f5ef6a1a 100644
--- a/content/public/browser/child_process_data.h
+++ b/content/public/browser/child_process_data.h
@@ -5,14 +5,16 @@
#ifndef CONTENT_PUBLIC_BROWSER_CHILD_PROCESS_DATA_H_
#define CONTENT_PUBLIC_BROWSER_CHILD_PROCESS_DATA_H_
+#include "base/memory/ref_counted.h"
#include "base/process/process.h"
#include "base/strings/string16.h"
#include "content/common/content_export.h"
+#include "content/public/browser/process_resource_usage.h"
namespace content {
// Holds information about a child process.
-struct ChildProcessData {
+struct CONTENT_EXPORT ChildProcessData {
// The type of the process.
int process_type;
@@ -30,9 +32,12 @@ struct ChildProcessData {
// current process.
base::ProcessHandle handle;
- explicit ChildProcessData(int process_type)
- : process_type(process_type), id(0), handle(base::kNullProcessHandle) {
- }
+ // Resource usage reporter to obtain certain resource stats such as JS memory
+ // usage. May be null.
+ scoped_refptr<ProcessResourceUsage> process_resource_usage;
ncarter (slow) 2015/05/01 21:16:52 As implemented, this modification does plumb the p
Anand Mistry (off Chromium) 2015/05/06 02:44:18 Done. It requires more plumbing, but isn't as bad
+
+ explicit ChildProcessData(int process_type);
+ ~ChildProcessData();
};
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698