commit d0c95d91992b4adc968de877554af2528beec5cd
author Louis Romero <lpromero@google.com> Mon Nov 14 15:43:09 2022
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Nov 14 15:43:09 2022
tree 79f4e459835c3cd8fb638dfadc913ce018e6753b
parent 43b7aedb61468b4bea1fcb2cc719d00f24f858cc

Remove extraneous early returns

webStateList is a prerequisite to KeyCommandsProvider already, so no
need to be so defensive.

Bug: 1379148
Change-Id: I06686253ad289b8f8cb2b079b0c7e3b3f892fb3d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4023376
Commit-Queue: Louis Romero <lpromero@google.com>
Reviewed-by: Aliona Dangla <alionadangla@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1071025}

ios/chrome/browser/ui/browser_view/key_commands_provider.mm

diff --git a/ios/chrome/browser/ui/browser_view/key_commands_provider.mm b/ios/chrome/browser/ui/browser_view/key_commands_provider.mm
index 140cb7a..bce73e1 100644
--- a/ios/chrome/browser/ui/browser_view/key_commands_provider.mm
+++ b/ios/chrome/browser/ui/browser_view/key_commands_provider.mm
@@ -317,9 +317,6 @@
 - (void)keyCommand_showNextTab {
   RecordAction(UserMetricsAction("MobileKeyCommandShowNextTab"));
   WebStateList* webStateList = self.browser->GetWebStateList();
-  if (!webStateList)
-    return;
-
   int activeIndex = webStateList->active_index();
   if (activeIndex == WebStateList::kInvalidIndex)
     return;
@@ -337,9 +334,6 @@
 - (void)keyCommand_showPreviousTab {
   RecordAction(UserMetricsAction("MobileKeyCommandShowPreviousTab"));
   WebStateList* webStateList = self.browser->GetWebStateList();
-  if (!webStateList)
-    return;
-
   int activeIndex = webStateList->active_index();
   if (activeIndex == WebStateList::kInvalidIndex)
     return;