Last pass using ?.() for method call (microsoft#152231)

Follow up on f17b33f

This handles sightly more complex cases, changing:

```ts
if (a) {
    a.b.c();
}
```

to:

```ts
a?.b.c();
```

Author: mjbvz

src/vs/base/browser/ui/actionbar/actionViewItems.ts

@@ -367,9 +367,7 @@ export class ActionViewItem extends BaseActionViewItem {
 
 			this.updateEnabled();
 		} else {
-			if (this.label) {
-				this.label.classList.remove('codicon');
-			}
+			this.label?.classList.remove('codicon');
 		}
 	}
 
@@ -380,18 +378,14 @@ export class ActionViewItem extends BaseActionViewItem {
 				this.label.classList.remove('disabled');
 			}
 
-			if (this.element) {
-				this.element.classList.remove('disabled');
-			}
+			this.element?.classList.remove('disabled');
 		} else {
 			if (this.label) {
 				this.label.setAttribute('aria-disabled', 'true');
 				this.label.classList.add('disabled');
 			}
 
-			if (this.element) {
-				this.element.classList.add('disabled');
-			}
+			this.element?.classList.add('disabled');
 		}
 	}
 

src/vs/base/browser/ui/dialog/dialog.ts

@@ -427,9 +427,7 @@ export class Dialog extends Disposable {
 			this.element.style.backgroundColor = bgColor?.toString() ?? '';
 			this.element.style.border = border;
 
-			if (this.buttonBar) {
-				this.buttonBar.buttons.forEach(button => button.style(style));
-			}
+			this.buttonBar?.buttons.forEach(button => button.style(style));
 
 			this.checkbox?.style(style);
 

src/vs/base/browser/ui/list/listView.ts

@@ -1110,19 +1110,15 @@ export class ListView<T> implements ISpliceable<T>, IDisposable {
 				const item = this.items[index]!;
 				item.dropTarget = true;
 
-				if (item.row) {
-					item.row.domNode.classList.add('drop-target');
-				}
+				item.row?.domNode.classList.add('drop-target');
 			}
 
 			this.currentDragFeedbackDisposable = toDisposable(() => {
 				for (const index of feedback) {
 					const item = this.items[index]!;
 					item.dropTarget = false;
 
-					if (item.row) {
-						item.row.domNode.classList.remove('drop-target');
-					}
+					item.row?.domNode.classList.remove('drop-target');
 				}
 			});
 		}

src/vs/base/browser/ui/menu/menubar.ts

@@ -956,9 +956,7 @@ export class MenuBar extends Disposable {
 			}
 
 			if (this.focusedMenu.holder) {
-				if (this.focusedMenu.holder.parentElement) {
-					this.focusedMenu.holder.parentElement.classList.remove('open');
-				}
+				this.focusedMenu.holder.parentElement?.classList.remove('open');
 
 				this.focusedMenu.holder.remove();
 			}

src/vs/base/browser/ui/splitview/paneview.ts

@@ -143,9 +143,7 @@ export abstract class Pane extends Disposable implements IView {
 			return false;
 		}
 
-		if (this.element) {
-			this.element.classList.toggle('expanded', expanded);
-		}
+		this.element?.classList.toggle('expanded', expanded);
 
 		this._expanded = !!expanded;
 		this.updateHeader();

src/vs/code/electron-sandbox/issue/issueReporterMain.ts

@@ -1204,12 +1204,8 @@ export class IssueReporter extends Disposable {
 // helper functions
 
 function hide(el: Element | undefined | null) {
-	if (el) {
-		el.classList.add('hidden');
-	}
+	el?.classList.add('hidden');
 }
 function show(el: Element | undefined | null) {
-	if (el) {
-		el.classList.remove('hidden');
-	}
+	el?.classList.remove('hidden');
 }

src/vs/editor/browser/widget/codeEditorWidget.ts

@@ -588,9 +588,7 @@ export class CodeEditorWidget extends Disposable implements editorBrowser.ICodeE
 	}
 
 	public setHiddenAreas(ranges: IRange[]): void {
-		if (this._modelData) {
-			this._modelData.viewModel.setHiddenAreas(ranges.map(r => Range.lift(r)));
-		}
+		this._modelData?.viewModel.setHiddenAreas(ranges.map(r => Range.lift(r)));
 	}
 
 	public getVisibleColumnFromPosition(rawPosition: IPosition): number {

src/vs/editor/contrib/documentSymbols/browser/outlineModel.ts

@@ -30,9 +30,7 @@ export abstract class TreeElement {
 	abstract parent: TreeElement | undefined;
 
 	remove(): void {
-		if (this.parent) {
-			this.parent.children.delete(this.id);
-		}
+		this.parent?.children.delete(this.id);
 	}
 
 	static findId(candidate: DocumentSymbol | string, container: TreeElement): string {

src/vs/editor/contrib/parameterHints/browser/parameterHintsWidget.ts

@@ -159,9 +159,7 @@ export class ParameterHintsWidget extends Disposable implements IContentWidget {
 		this.keyVisible.set(true);
 		this.visible = true;
 		setTimeout(() => {
-			if (this.domNodes) {
-				this.domNodes.element.classList.add('visible');
-			}
+			this.domNodes?.element.classList.add('visible');
 		}, 100);
 		this.editor.layoutContentWidget(this);
 	}
@@ -176,9 +174,7 @@ export class ParameterHintsWidget extends Disposable implements IContentWidget {
 		this.keyVisible.reset();
 		this.visible = false;
 		this.announcedLabel = null;
-		if (this.domNodes) {
-			this.domNodes.element.classList.remove('visible');
-		}
+		this.domNodes?.element.classList.remove('visible');
 		this.editor.layoutContentWidget(this);
 	}
 

src/vs/platform/issue/electron-main/issueMainService.ts

@@ -168,9 +168,7 @@ export class IssueMainService implements ICommonIssueService {
 					throw new Error(`Unexpected command source: ${from}`);
 			}
 
-			if (parentWindow) {
-				parentWindow.webContents.send('vscode:runAction', { id, from, args });
-			}
+			parentWindow?.webContents.send('vscode:runAction', { id, from, args });
 		});
 
 		validatedIpcMain.on('vscode:openExternal', (_: unknown, arg: string) => {