[sidepanelpinning] fix potential crash for trying to hide an action t…

…hat doesn't exist.

I found this while fixing the SidePanelCoordinatorTests, which I'll send
out after this. If for some reason this gets called twice for a popped
out button, the button will first be removed then the second time a
button cannot be found which would cause a crash.

Bug: 1509504
Change-Id: I395a9ebfaeb96995d1e968e6b1062eeaec252f98
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5099962
Reviewed-by: Darryl James <dljames@chromium.org>
Commit-Queue: Caroline Rising <corising@chromium.org>
Code-Coverage: findit-for-me@appspot.gserviceaccount.com <findit-for-me@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1234632}

chrome/browser/ui/views/toolbar/pinned_toolbar_actions_container.cc

@@ -380,6 +380,12 @@ void PinnedToolbarActionsContainer::UpdateActionState(actions::ActionId id,
       button = AddPopOutButtonFor(id);
     }
   }
+  // If the button doesn't exist, do nothing. This could happen if |is_active|
+  // is false and there is no existing pinned out popped out button for the
+  // |id|.
+  if (!button) {
+    return;
+  }
 
   // Update button highlight and force visibility if the button is active.
   if (is_active) {