Migrate ProcessesEventRouter to the new task manager

chrome/common/extensions/api/processes.idl

Index: chrome/common/extensions/api/processes.idl
diff --git a/chrome/common/extensions/api/processes.idl b/chrome/common/extensions/api/processes.idl
index 556ba2418c3588d111224f9f1b1459a398099f3d..dae99cc702b15ee007677260dee0785042089859 100644
--- a/chrome/common/extensions/api/processes.idl
+++ b/chrome/common/extensions/api/processes.idl
@@ -18,6 +18,16 @@ namespace processes {
     gpu,
     other
   };
+
+  // An object that represents a Chrome task running on a process. Several tasks
+  // can share the same process.
+  dictionary TaskInfo {
+    // The title of the task.
+    DOMString title;
+    // Optional tab ID, if this task represents a tab running on a renderer
+    // process.
+    long? tabId;
+  };
   
   // The Cache object contains information about the size and utilization of a 
   // cache used by the browser.
@@ -34,8 +44,6 @@ namespace processes {
     long id;
     // The ID of the process, as provided by the OS.
     long osProcessId;
-    // The title of the process as seen in the task manager.
-    DOMString title;
     // The type of process.
     ProcessType type;
     // The profile which the process is associated with.
@@ -43,9 +51,8 @@ namespace processes {
     // The debugging port for Native Client processes. Zero for other process 
     // types and for NaCl processes that do not have debugging enabled.
     long naclDebugPort;
-    // Array of Tab IDs that have a page rendered by this process. The list will
-    // be non-empty for renderer processes only.
-    long[] tabs;
+    // Array of TaskInfos representing the tasks running on this process.
+    TaskInfo[] tasks;

[Devlin, 2016/03/03 23:12:43]

Breaking changes :(

Is there anything else we should update while we're here?

[afakhry, 2016/03/08 01:37:30]

No this is just it.

     // The most recent measurement of the process CPU usage, between 0 and 100.
     // Only available when receiving the object as part of a callback from 
     // onUpdated or onUpdatedWithMemory.
@@ -83,7 +90,7 @@ namespace processes {
     // or onUpdatedWithMemory.
     Cache? cssCache;
   };
-  
+
   // A callback to report the status of the termination. 
   // |didTerminate|: True if terminating the process was successful, and false
   // otherwise.
@@ -139,6 +146,7 @@ namespace processes {
     // included in each Process object. Note, collecting memory usage 
     // information incurs extra CPU usage and should only be listened for when
     // needed.
+

[Devlin, 2016/03/03 23:12:43]

Will this newline affect our doc parsing?

[afakhry, 2016/03/08 01:37:30]

Reverted this line.

     // |processes|: A dictionary of updated $(ref:Process) objects for each live
     // process in the browser, indexed by process ID.  Memory usage details will
     // be included in each Process object.