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

Unified Diff: chrome/test/data/extensions/api_test/processes/api/test.js

Issue 165153003: Add process title and nacl debug port fields to the processes api. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 6 years, 10 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
« no previous file with comments | « chrome/common/extensions/api/processes.json ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/extensions/api_test/processes/api/test.js
diff --git a/chrome/test/data/extensions/api_test/processes/api/test.js b/chrome/test/data/extensions/api_test/processes/api/test.js
index 32f8804f7a48a01b6fef14d93d667ff17ea9d6f0..abeb0ba49b3cf13ae5e2876e5219f12f202b54e2 100644
--- a/chrome/test/data/extensions/api_test/processes/api/test.js
+++ b/chrome/test/data/extensions/api_test/processes/api/test.js
@@ -29,6 +29,7 @@ function pageUrl(letter) {
function dumpProcess(process) {
console.log("id " + process.id);
+ console.log("title " + process.title);
console.log("osProcId " + process.osProcessId);
console.log("type " + process.type);
console.log("profile " + process.profile);
@@ -40,6 +41,7 @@ function dumpProcess(process) {
console.log("jsMemUsed " + process.jsMemoryUsed);
console.log("sqliteMem " + process.sqliteMemory);
console.log("fps " + process.fps);
+ console.log("naclDebugPort " + process.naclDebugPort);
if ("imageCache" in process) {
console.log("imageCache.size " + process.imageCache.size);
console.log("imageCache.liveSize " + process.imageCache.liveSize);
@@ -57,6 +59,8 @@ function dumpProcess(process) {
function validateProcessProperties(process, updating, memory_included) {
// Always present.
assertTrue("id" in process);
+ assertTrue("title" in process);
+ assertTrue("naclDebugPort" in process);
assertTrue("osProcessId" in process);
assertTrue("type" in process);
assertTrue("profile" in process);
@@ -73,7 +77,7 @@ function validateProcessProperties(process, updating, memory_included) {
// sqliteMemory is only reported for the browser process
if (process.type == "browser") {
assertEq(("sqliteMemory" in process), updating);
- } else {
+ } else if (process.type == "renderer") {
// The rest are not present in the browser process
assertEq(("jsMemoryAllocated" in process), updating);
assertEq(("jsMemoryUsed" in process), updating);
@@ -264,6 +268,9 @@ chrome.test.runTests([
createTab(5, "chrome://newtab/");
},
+ // DISABLED: crbug.com/345411
+ // Hangs consistently.
+ /*
function testOnExited() {
listenOnce(chrome.processes.onExited,
function(processId, type, code) {
@@ -273,6 +280,7 @@ chrome.test.runTests([
chrome.tabs.remove(tab.id);
}));
},
+ */
function testGetProcessInfoList() {
getProcessId(tabs[0].id, pass(function(pidTab0) {
« no previous file with comments | « chrome/common/extensions/api/processes.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698