Does this issue occur when all extensions are disabled?: Yes

• VS Code Version: 1.76.0
• OS Version: macOS

Steps to Reproduce:

1. Click on a line gutter icon for an expanded comment thread.
2. ReviewZoneWidget.toggleExpand() sets the comment thread's collapsibleState to Collapsed:

   vscode/src/vs/workbench/contrib/comments/browser/commentThreadZoneWidget.ts

   Line 278 in e282742

   this._commentThread.collapsibleState = languages.CommentThreadCollapsibleState.Collapsed;

3. This causes the widget's listener on the thread's collapsible state to fire synchronously and call hide():

   vscode/src/vs/workbench/contrib/comments/browser/commentThreadZoneWidget.ts

   Line 377 in e282742

   this.hide();

4. toggleExpand() follows this up with an explicit, duplicate call to hide():

   vscode/src/vs/workbench/contrib/comments/browser/commentThreadZoneWidget.ts

   Line 279 in e282742

   this.hide();

For the other direction, the problem is the same and show() is called twice with identical parameters on every change.

I am not sure if these duplicate calls cause unnecessary rendering or similar elsewhere in the stack, but they seem redundant. I think relying on the listener for onDidChangeCollapsibleState is probably sufficient, but I am not that familiar enough with the code base to have a confident opinion here.