[Extensions] Fix hiding browser actions without the toolbar redesign

chrome/browser/extensions/extension_context_menu_model.cc

Index: chrome/browser/extensions/extension_context_menu_model.cc
diff --git a/chrome/browser/extensions/extension_context_menu_model.cc b/chrome/browser/extensions/extension_context_menu_model.cc
index d97ee3abdf40b1ed6217780c38ac7122d4b3caac..27ef17bab7ebfb1796fdecb8ce0c35199442cf2f 100644
--- a/chrome/browser/extensions/extension_context_menu_model.cc
+++ b/chrome/browser/extensions/extension_context_menu_model.cc
@@ -256,6 +256,11 @@ void ExtensionContextMenuModel::ExecuteCommand(int command_id,
       break;
     case TOGGLE_VISIBILITY: {
       bool currently_visible = button_visibility_ == VISIBLE;
+      // Without the toolbar redesign turned on, action visibility refers to
+      // any action presence in the toolbar, independent of whether the action
+      // is visible or overflowed. So any action present is considered visible.
+      if (!FeatureSwitch::extension_action_redesign()->IsEnabled())
+        currently_visible = true;
       ToolbarActionsModel::Get(browser_->profile())
           ->SetActionVisibility(extension->id(), !currently_visible);
       break;