Always set the navigation pending item for page reload.

ios/web/web_state/navigation_callbacks_inttest.mm

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <memory>
 #include <string>
 
 #include "base/memory/ptr_util.h"
 #include "base/strings/stringprintf.h"
 #import "ios/web/public/navigation_item.h"
@@ skipping 69 matching lines @@
 ACTION_P3(VerifyPostStartedContext, web_state, url, context) {
   *context = arg0;
   ASSERT_TRUE(*context);
   EXPECT_EQ(web_state, (*context)->GetWebState());
   EXPECT_EQ(url, (*context)->GetUrl());
   EXPECT_FALSE((*context)->IsSameDocument());
   EXPECT_TRUE((*context)->IsPost());
   EXPECT_FALSE((*context)->GetError());
   ASSERT_FALSE((*context)->GetResponseHeaders());
   ASSERT_TRUE(web_state->IsLoading());
-  // TODO(crbug.com/676129): Reload does not create a pending item. Remove this
-  // workaround once the bug is fixed.
-  if (!ui::PageTransitionTypeIncludingQualifiersIs(
-          ui::PageTransition::PAGE_TRANSITION_RELOAD,
-          (*context)->GetPageTransition())) {
-    NavigationManager* navigation_manager = web_state->GetNavigationManager();
-    NavigationItem* item = navigation_manager->GetPendingItem();
-    EXPECT_EQ(url, item->GetURL());
-  }
+  NavigationManager* navigation_manager = web_state->GetNavigationManager();
+  NavigationItem* item = navigation_manager->GetPendingItem();
+  EXPECT_EQ(url, item->GetURL());
 }
 
 // Verifies correctness of |NavigationContext| (|arg0|) for navigations via POST
 // HTTP methods passed to |DidFinishNavigation|. Stores |NavigationContext| in
 // |context| pointer.
 ACTION_P3(VerifyPostFinishedContext, web_state, url, context) {
   ASSERT_EQ(*context, arg0);
   EXPECT_EQ(web_state, (*context)->GetWebState());
   ASSERT_TRUE((*context));
   EXPECT_EQ(web_state, (*context)->GetWebState());
@@ skipping 101 matching lines @@
   *context = arg0;
   ASSERT_TRUE(*context);
   EXPECT_EQ(web_state, (*context)->GetWebState());
   EXPECT_EQ(url, (*context)->GetUrl());
   EXPECT_TRUE(
       PageTransitionCoreTypeIs(ui::PageTransition::PAGE_TRANSITION_RELOAD,
                                (*context)->GetPageTransition()));
   EXPECT_FALSE((*context)->IsSameDocument());
   EXPECT_FALSE((*context)->GetError());
   EXPECT_FALSE((*context)->GetResponseHeaders());
-  // TODO(crbug.com/676129): Reload does not create a pending item. Check
-  // pending item once the bug is fixed.
-  EXPECT_FALSE(web_state->GetNavigationManager()->GetPendingItem());
+  EXPECT_EQ(web_state->GetNavigationManager()->GetPendingItem(),
+            web_state->GetNavigationManager()->GetLastCommittedItem());
 }
 
 // Verifies correctness of |NavigationContext| (|arg0|) for reload navigation
 // passed to |DidFinishNavigation|. Asserts that |NavigationContext| the same as
 // |context|.
 ACTION_P4(VerifyReloadFinishedContext, web_state, url, context, is_web_page) {
   ASSERT_EQ(*context, arg0);
   ASSERT_TRUE(*context);
   EXPECT_EQ(web_state, (*context)->GetWebState());
   EXPECT_EQ(url, (*context)->GetUrl());
@@ skipping 410 matching lines @@
   // delegate does not run callback for ShowRepostFormWarningDialog. Clearing
   // the delegate will allow form resubmission. Remove this workaround (clearing
   // the delegate, once |check_for_repost| is supported).
   web_state()->SetDelegate(nullptr);
   ExecuteBlockAndWaitForLoad(action, ^{
     navigation_manager()->GoForward();
   });
 }
 
 }  // namespace web