[ios] Do not call set_accept() for Confirm Infobars

Confirm Infobars do not have badges, so there is not need to update
the state of the Infobar. Moreover, after
ConfirmInfobarDelegate::Accept() the tab, and therefore the infobar,
is removed.

Bug: 1129762
Change-Id: I79d9c636b1df3e9de19c3f24f20193e9d0c2df88
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432584
Commit-Queue: Chris Lu <thegreenfrog@chromium.org>
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#811300}

ios/chrome/browser/infobars/overlays/browser_agent/interaction_handlers/confirm/confirm_infobar_banner_interaction_handler.mm

@@ -26,7 +26,8 @@ ConfirmInfobarBannerInteractionHandler::
 
 void ConfirmInfobarBannerInteractionHandler::MainButtonTapped(
     InfoBarIOS* infobar) {
-  infobar->set_accepted(GetInfobarDelegate(infobar)->Accept());
+  // Confirm Infobars don't need to update badge status.
+  GetInfobarDelegate(infobar)->Accept();
 }
 
 void ConfirmInfobarBannerInteractionHandler::BannerVisibilityChanged(

ios/chrome/browser/infobars/overlays/browser_agent/interaction_handlers/confirm/confirm_infobar_banner_interaction_handler_unittest.mm

@@ -52,13 +52,10 @@ class ConfirmInfobarBannerInteractionHandlerTest : public PlatformTest {
   InfoBarIOS* infobar_;
 };
 
-// Tests MainButtonTapped() calls Accept() on the mock delegate and resets
-// the infobar to be accepted.
+// Tests MainButtonTapped() calls Accept() on the mock delegate.
 TEST_F(ConfirmInfobarBannerInteractionHandlerTest, MainButton) {
-  ASSERT_FALSE(infobar_->accepted());
   EXPECT_CALL(mock_delegate(), Accept()).WillOnce(testing::Return(true));
   handler_.MainButtonTapped(infobar_);
-  EXPECT_TRUE(infobar_->accepted());
 }
 
 // Tests that BannerVisibilityChanged() InfobarDismissed() on the mock delegate.