Remove GetActiveEntry usage from content.

content/browser/web_contents/render_view_host_manager_unittest.cc

Index: content/browser/web_contents/render_view_host_manager_unittest.cc
diff --git a/content/browser/web_contents/render_view_host_manager_unittest.cc b/content/browser/web_contents/render_view_host_manager_unittest.cc
index 7d435fac5d6c5eb57747424ddfa010cfaf4532b4..5d56d4e794b77d0ab673b680cc323527d3450602 100644
--- a/content/browser/web_contents/render_view_host_manager_unittest.cc
+++ b/content/browser/web_contents/render_view_host_manager_unittest.cc
@@ -950,7 +950,8 @@ TEST_F(RenderViewHostManagerTest, PageDoesBackAndReload) {
   EXPECT_EQ(evil_rvh, contents()->GetRenderManagerForTesting()->current_host());
 
   // Also we should not have a pending navigation entry.
-  NavigationEntry* entry = contents()->GetController().GetActiveEntry();
+  EXPECT_TRUE(contents()->GetController().GetPendingEntry() == NULL);

[Charlie Reis, 2013/09/16 23:10:54]

nit: Just use EXPECT_FALSE and we won't need the "== NULL"

[nasko, 2013/09/18 17:03:18]

I kept it consistent with the surrounding code, which all uses EXPECT_TRUE with
explicit check inside. If you feel strongly about this, I'll put together
another CL and change all instances to consistently use the other pattern.

[Charlie Reis, 2013/09/18 18:36:05]

Ok.  Feel free to send a cleanup CL to me after this and I'll approve it.

+  NavigationEntry* entry = contents()->GetController().GetVisibleEntry();
   ASSERT_TRUE(entry != NULL);
   EXPECT_EQ(kUrl2, entry->GetURL());
 }