Chromium Code Reviews| Index: content/browser/site_per_process_browsertest.cc |
| diff --git a/content/browser/site_per_process_browsertest.cc b/content/browser/site_per_process_browsertest.cc |
| index f750a9d17b33eb503c0c3c54f209c01e70e5eb03..49d26ea78377ea770e32d3aaa4adf5de86d83ebf 100644 |
| --- a/content/browser/site_per_process_browsertest.cc |
| +++ b/content/browser/site_per_process_browsertest.cc |
| @@ -1031,8 +1031,8 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest, |
| GURL site_c_url(embedded_test_server()->GetURL("baz.com", "/title1.html")); |
| EXPECT_EQ(site_c_url, node4->current_url()); |
| - // |site_instance_c| is expected to go away once we kill |child_process_b| |
| - // below; refcount it to extend the lifetime. |
| + // |site_instance_c|'s frames are expected to go away once we kill |
| + // |child_process_b| below. |
| scoped_refptr<SiteInstanceImpl> site_instance_c = |
| node4->current_frame_host()->GetSiteInstance(); |
| @@ -1065,7 +1065,7 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest, |
| " B = http://bar.com/ (no process)", |
| DepictFrameTree(root)); |
| - EXPECT_TRUE(site_instance_c->HasOneRef()); |
| + EXPECT_EQ(0U, site_instance_c->active_frame_count()); |
|
Charlie Reis
2015/05/27 17:26:24
This is https://codereview.chromium.org/1155393003
|
| } |
| // Crash a subframe and ensures its children are cleared from the FrameTree. |