PlzNavigate: Properly set the pending entry in the NavigatorImpl unit test.

content/browser/frame_host/navigation_controller_impl.cc

Index: content/browser/frame_host/navigation_controller_impl.cc
diff --git a/content/browser/frame_host/navigation_controller_impl.cc b/content/browser/frame_host/navigation_controller_impl.cc
index 0b0f411a1a041f52893e84357af935f7fe81c0df..42a903a9f31b7427756cbcff33018a2e12c3284a 100644
--- a/content/browser/frame_host/navigation_controller_impl.cc
+++ b/content/browser/frame_host/navigation_controller_impl.cc
@@ -425,6 +425,7 @@ void NavigationControllerImpl::LoadEntry(NavigationEntryImpl* entry) {
 void NavigationControllerImpl::SetPendingEntry(NavigationEntryImpl* entry) {
   DiscardNonCommittedEntriesInternal();
   pending_entry_ = entry;
+  pending_entry_index_ = GetIndexOfEntry(entry);

[Charlie Reis, 2015/03/18 20:26:35]

Interesting that we didn't need this before, but I think it's because this was
only used for newly created NavigationEntries at first, when the index was
assumed to be -1.  I agree that this looks correct and more robust.

[Avi (use Gerrit), 2015/03/18 20:32:42]

Yep.

   NotificationService::current()->Notify(
       NOTIFICATION_NAV_ENTRY_PENDING,
       Source<NavigationController>(this),