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 |