Most references to GetActiveEntry removed from chrome/browser/ui.

chrome/browser/ui/search/search_tab_helper.cc

Index: chrome/browser/ui/search/search_tab_helper.cc
diff --git a/chrome/browser/ui/search/search_tab_helper.cc b/chrome/browser/ui/search/search_tab_helper.cc
index f9dc05ab7aa66c532cda80c252c5574a2a9a96fd..25076b407bb163e4ddb3650ff22552eac17a6878 100644
--- a/chrome/browser/ui/search/search_tab_helper.cc
+++ b/chrome/browser/ui/search/search_tab_helper.cc
@@ -69,7 +69,7 @@ void RecordCacheableNTPLoadHistogram(bool succeeded) {
 
 bool IsCacheableNTP(const content::WebContents* contents) {
   const content::NavigationEntry* entry =
-      contents->GetController().GetActiveEntry();
+      contents->GetController().GetLastCommittedEntry();
   return chrome::ShouldUseCacheableNTP() &&
       chrome::NavEntryIsInstantNTP(contents, entry) &&
       entry->GetURL() != GURL(chrome::kChromeSearchLocalNtpUrl);
@@ -304,7 +304,7 @@ void SearchTabHelper::DidNavigateMainFrame(
   // also a race condition between this code and the page's SetTitle call which
   // this rule avoids.
   content::NavigationEntry* entry =
-      web_contents_->GetController().GetActiveEntry();
+      web_contents_->GetController().GetVisibleEntry();

[nasko, 2013/11/08 23:52:03]

DidNavigateMainFrame is the commit point. As such, it seems natural to use
LastCommitted here.

[jww, 2013/11/11 19:18:51]

Done.

   if (entry && entry->GetTitle().empty() &&
       (entry->GetVirtualURL() == GURL(chrome::kChromeUINewTabURL) ||
        chrome::NavEntryIsInstantNTP(web_contents_, entry))) {