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

chrome/browser/ui/tabs/pinned_tab_codec.cc

Index: chrome/browser/ui/tabs/pinned_tab_codec.cc
diff --git a/chrome/browser/ui/tabs/pinned_tab_codec.cc b/chrome/browser/ui/tabs/pinned_tab_codec.cc
index e1932d069abeb51e5e1d518dd3651747e58b03c7..7a209e5e8f70e415e2ba9dac5419cda5f790a18f 100644
--- a/chrome/browser/ui/tabs/pinned_tab_codec.cc
+++ b/chrome/browser/ui/tabs/pinned_tab_codec.cc
@@ -67,7 +67,7 @@ static void EncodePinnedTab(TabStripModel* model,
         kURL, extensions::AppLaunchInfo::GetFullLaunchURL(extension).spec());
     values->Append(value.release());
   } else {
-    NavigationEntry* entry = web_contents->GetController().GetActiveEntry();
+    NavigationEntry* entry = web_contents->GetController().GetVisibleEntry();

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

Why not LastCommitted here?

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

In this case, my thinking was that a user is trying to pin the URL that they see
in the omnibox, not the content, but I suppose that's debatable. What do you
think? Probably needs confirmation from an owner either way on this one.

     if (!entry && web_contents->GetController().GetEntryCount())
       entry = web_contents->GetController().GetEntryAtIndex(0);
     if (entry) {