| Index: chrome/browser/tab_contents/render_view_host_manager_unittest.cc
|
| ===================================================================
|
| --- chrome/browser/tab_contents/render_view_host_manager_unittest.cc (revision 70813)
|
| +++ chrome/browser/tab_contents/render_view_host_manager_unittest.cc (working copy)
|
| @@ -253,8 +253,6 @@
|
| EXPECT_TRUE(host);
|
| EXPECT_TRUE(host == manager.current_host());
|
| EXPECT_FALSE(manager.pending_render_view_host());
|
| - EXPECT_TRUE(manager.pending_dom_ui());
|
| - EXPECT_FALSE(manager.dom_ui());
|
|
|
| // It's important that the site instance get set on the DOM UI page as soon
|
| // as the navigation starts, rather than lazily after it commits, so we don't
|
| @@ -263,11 +261,13 @@
|
| EXPECT_TRUE(host->site_instance()->has_site());
|
| EXPECT_EQ(url, host->site_instance()->site());
|
|
|
| + // The DOM UI is committed immediately because the RenderViewHost has not been
|
| + // used yet. UpdateRendererStateForNavigate() took the short cut path.
|
| + EXPECT_FALSE(manager.pending_dom_ui());
|
| + EXPECT_TRUE(manager.dom_ui());
|
| +
|
| // Commit.
|
| manager.DidNavigateMainFrame(host);
|
| -
|
| - EXPECT_FALSE(manager.pending_dom_ui());
|
| - EXPECT_TRUE(manager.dom_ui());
|
| }
|
|
|
| // Tests that chrome: URLs that are not DOM UI pages do not get grouped into
|
|
|