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

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

Issue 1792453004: Disable newTabPagesShareProcess in ProcessesApiTest.Processes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Processes API test for Chrome. 5 // Processes API test for Chrome.
6 // browser_tests.exe --gtest_filter=ExtensionApiTest.Processes 6 // browser_tests.exe --gtest_filter=ExtensionApiTest.Processes
7 7
8 var pass = chrome.test.callbackPass; 8 var pass = chrome.test.callbackPass;
9 var fail = chrome.test.callbackFail; 9 var fail = chrome.test.callbackFail;
10 var assertEq = chrome.test.assertEq; 10 var assertEq = chrome.test.assertEq;
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
162 162
163 function newTabPageInOwnProcess() { 163 function newTabPageInOwnProcess() {
164 getProcessId(tabs[0].id, pass(function(pid0) { 164 getProcessId(tabs[0].id, pass(function(pid0) {
165 getProcessId(tabs[3].id, pass(function(pid3) { 165 getProcessId(tabs[3].id, pass(function(pid3) {
166 // NTP should not share a process with current tabs 166 // NTP should not share a process with current tabs
167 assertTrue(pid0 != pid3); 167 assertTrue(pid0 != pid3);
168 })); 168 }));
169 })); 169 }));
170 }, 170 },
171 171
172 // DISABLED: crbug.com/594318
173 // Fails in --site-per-process
174 /*
172 function newTabPagesShareProcess() { 175 function newTabPagesShareProcess() {
173 getProcessId(tabs[3].id, pass(function(pid3) { 176 getProcessId(tabs[3].id, pass(function(pid3) {
174 getProcessId(tabs[4].id, pass(function(pid4) { 177 getProcessId(tabs[4].id, pass(function(pid4) {
175 // Multiple NTPs should share a process 178 // Multiple NTPs should share a process
176 assertEq(pid3, pid4); 179 assertEq(pid3, pid4);
177 })); 180 }));
178 })); 181 }));
179 }, 182 },
183 */
180 184
181 function idsInUpdateEvent() { 185 function idsInUpdateEvent() {
182 listenOnce(chrome.processes.onUpdated, function(processes) { 186 listenOnce(chrome.processes.onUpdated, function(processes) {
183 // onUpdated should return a valid dictionary of processes, 187 // onUpdated should return a valid dictionary of processes,
184 // indexed by process ID. 188 // indexed by process ID.
185 var pids = Object.keys(processes); 189 var pids = Object.keys(processes);
186 // There should be at least 5 processes: 1 browser, 1 extension, and 3 190 // There should be at least 5 processes: 1 browser, 1 extension, and 3
187 // renderers (for the 5 tabs). 191 // renderers (for the 5 tabs).
188 assertTrue(pids.length >= 5, "Unexpected size of pids"); 192 assertTrue(pids.length >= 5, "Unexpected size of pids");
189 193
(...skipping 143 matching lines...) Expand 10 before | Expand all | Expand 10 after
333 }); 337 });
334 }); 338 });
335 chrome.tabs.create({"url": "chrome://hang" }, function(tab) { 339 chrome.tabs.create({"url": "chrome://hang" }, function(tab) {
336 getProcessId(tab.id, function(pid0) { 340 getProcessId(tab.id, function(pid0) {
337 hangingTabProcess = pid0; 341 hangingTabProcess = pid0;
338 }); 342 });
339 chrome.tabs.update(tab.id, { "url": "chrome://flags" }); 343 chrome.tabs.update(tab.id, { "url": "chrome://flags" });
340 }); 344 });
341 } 345 }
342 ]); 346 ]);
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698