Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2011 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 #include "base/command_line.h" | 5 #include "base/command_line.h" |
| 6 #include "base/file_path.h" | 6 #include "base/file_path.h" |
| 7 #include "base/scoped_ptr.h" | 7 #include "base/scoped_ptr.h" |
| 8 #include "base/string_number_conversions.h" | 8 #include "base/string_number_conversions.h" |
| 9 #include "base/test/test_timeouts.h" | 9 #include "base/test/test_timeouts.h" |
| 10 #include "chrome/app/chrome_command_ids.h" | 10 #include "chrome/app/chrome_command_ids.h" |
| (...skipping 432 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 443 | 443 |
| 444 ASSERT_EQ(url1_, GetActiveTabURL()); | 444 ASSERT_EQ(url1_, GetActiveTabURL()); |
| 445 } | 445 } |
| 446 | 446 |
| 447 // Make sure after a restore the number of processes matches that of the number | 447 // Make sure after a restore the number of processes matches that of the number |
| 448 // of processes running before the restore. This creates a new tab so that | 448 // of processes running before the restore. This creates a new tab so that |
| 449 // we should have two new tabs running. (This test will pass in both | 449 // we should have two new tabs running. (This test will pass in both |
| 450 // process-per-site and process-per-site-instance, because we treat the new tab | 450 // process-per-site and process-per-site-instance, because we treat the new tab |
| 451 // as a special case in process-per-site-instance so that it only ever uses one | 451 // as a special case in process-per-site-instance so that it only ever uses one |
| 452 // process.) | 452 // process.) |
| 453 // | 453 TEST_F(SessionRestoreUITest, ShareProcessesOnRestore) { |
|
Evan Martin
2011/02/23 00:17:32
Did you mean to include this?
sky
2011/02/23 00:38:57
Yes. All the changes I've done to ChromeURLDataMan
| |
| 454 // Flaky: http://code.google.com/p/chromium/issues/detail?id=52022 | |
| 455 TEST_F(SessionRestoreUITest, FLAKY_ShareProcessesOnRestore) { | |
| 456 if (ProxyLauncher::in_process_renderer()) { | 454 if (ProxyLauncher::in_process_renderer()) { |
| 457 // No point in running this test in single process mode. | 455 // No point in running this test in single process mode. |
| 458 return; | 456 return; |
| 459 } | 457 } |
| 460 | 458 |
| 461 scoped_refptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0)); | 459 scoped_refptr<BrowserProxy> browser_proxy(automation()->GetBrowserWindow(0)); |
| 462 ASSERT_TRUE(browser_proxy.get() != NULL); | 460 ASSERT_TRUE(browser_proxy.get() != NULL); |
| 463 int tab_count; | 461 int tab_count; |
| 464 ASSERT_TRUE(browser_proxy->GetTabCount(&tab_count)); | 462 ASSERT_TRUE(browser_proxy->GetTabCount(&tab_count)); |
| 465 | 463 |
| (...skipping 25 matching lines...) Expand all Loading... | |
| 491 ASSERT_TRUE(tab_proxy->WaitForTabToBeRestored( | 489 ASSERT_TRUE(tab_proxy->WaitForTabToBeRestored( |
| 492 TestTimeouts::action_max_timeout_ms())); | 490 TestTimeouts::action_max_timeout_ms())); |
| 493 } | 491 } |
| 494 | 492 |
| 495 int process_count = 0; | 493 int process_count = 0; |
| 496 ASSERT_TRUE(GetBrowserProcessCount(&process_count)); | 494 ASSERT_TRUE(GetBrowserProcessCount(&process_count)); |
| 497 ASSERT_EQ(expected_process_count, process_count); | 495 ASSERT_EQ(expected_process_count, process_count); |
| 498 } | 496 } |
| 499 | 497 |
| 500 } // namespace | 498 } // namespace |
| OLD | NEW |