Fix newTabPageShareProces() in chrome.processes API browsertests

chrome/test/data/extensions/api_test/processes/api/test.js

 // Copyright (c) 2012 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.
 
 // Processes API test for Chrome.
 // browser_tests.exe --gtest_filter=ExtensionApiTest.Processes
 
 var pass = chrome.test.callbackPass;
 var fail = chrome.test.callbackFail;
 var assertEq = chrome.test.assertEq;
@@ skipping 89 matching lines @@
     // Wait for all loads to complete.
     var completedCount = 0;
     var onUpdatedCompleted = chrome.test.listenForever(
       chrome.tabs.onUpdated,
       function(changedTabId, changeInfo, changedTab) {
         if (changedTab.status == "complete") {
           completedCount++;
 
           // Once the NTP finishes loading, create another one.  This ensures
           // both NTPs end up in the same process.
-          if (changedTabId == tabs[3].id) {
+          if (changedTabId == tabs[3].id)
             createTab(4, "chrome://newtab/");
-          }
         }
 
         // Once all tabs are done loading, continue with the next test.
-        if (completedCount == 4) {
+        if (completedCount == 5)
           onUpdatedCompleted();
-        }
       }
     );
 
   },
 
   function extensionPageInOwnProcess() {
     getProcessId(tabs[0].id, pass(function(pid0) {
       getProcessId(tabs[1].id, pass(function(pid1) {
         // about:blank and extension page should not share a process
         assertTrue(pid0 != pid1);
@@ skipping 33 matching lines @@
 
   function newTabPageInOwnProcess() {
     getProcessId(tabs[0].id, pass(function(pid0) {
       getProcessId(tabs[3].id, pass(function(pid3) {
         // NTP should not share a process with current tabs
         assertTrue(pid0 != pid3);
       }));
     }));
   },
 
-  // DISABLED: crbug.com/594318
-  // Fails in --site-per-process
-  /*
   function newTabPagesShareProcess() {
     getProcessId(tabs[3].id, pass(function(pid3) {
       getProcessId(tabs[4].id, pass(function(pid4) {
         // Multiple NTPs should share a process
         assertEq(pid3, pid4);
       }));
     }));
   },
-  */
 
   function idsInUpdateEvent() {
     listenOnce(chrome.processes.onUpdated, function(processes) {
       // onUpdated should return a valid dictionary of processes,
       // indexed by process ID.
       var pids = Object.keys(processes);
       // There should be at least 5 processes: 1 browser, 1 extension, and 3
       // renderers (for the 5 tabs).
       assertTrue(pids.length >= 5, "Unexpected size of pids");
 
@@ skipping 143 matching lines @@
       });
     });
     chrome.tabs.create({"url": "chrome://hang" }, function(tab) {
       getProcessId(tab.id, function(pid0) {
         hangingTabProcess = pid0;
       });
       chrome.tabs.update(tab.id, { "url": "chrome://flags" });
     });
   }
 ]);