Fix leak (microsoft#209588)

roblourens · web-flow · commit d6eba9b861e3 · 2024-04-04T20:52:04.000-04:00
Fix microsoft#208621
diff --git a/src/vs/workbench/contrib/chat/browser/chatListRenderer.ts b/src/vs/workbench/contrib/chat/browser/chatListRenderer.ts
@@ -1202,7 +1202,7 @@ class TreePool extends Disposable {
 	}
 
 	private treeFactory(): WorkbenchCompressibleAsyncDataTree<IChatResponseProgressFileTreeData, IChatResponseProgressFileTreeData, void> {
-		const resourceLabels = this.instantiationService.createInstance(ResourceLabels, { onDidChangeVisibility: this._onDidChangeVisibility });
+		const resourceLabels = this._register(this.instantiationService.createInstance(ResourceLabels, { onDidChangeVisibility: this._onDidChangeVisibility }));
 
 		const container = $('.interactive-response-progress-tree');
 		this._register(createFileIconThemableTreeContainerScope(container, this.themeService));
@@ -1262,7 +1262,7 @@ class ContentReferencesListPool extends Disposable {
 	}
 
 	private listFactory(): WorkbenchList<IChatContentReference> {
-		const resourceLabels = this.instantiationService.createInstance(ResourceLabels, { onDidChangeVisibility: this._onDidChangeVisibility });
+		const resourceLabels = this._register(this.instantiationService.createInstance(ResourceLabels, { onDidChangeVisibility: this._onDidChangeVisibility }));
 
 		const container = $('.chat-used-context-list');
 		this._register(createFileIconThemableTreeContainerScope(container, this.themeService));

Original file line number	Diff line number	Diff line change
`@@ -1202,7 +1202,7 @@ class TreePool extends Disposable {`
`1202`	`1202`	`}`
`1203`	`1203`
`1204`	`1204`	`private treeFactory(): WorkbenchCompressibleAsyncDataTree<IChatResponseProgressFileTreeData, IChatResponseProgressFileTreeData, void> {`
`1205`		`- const resourceLabels = this.instantiationService.createInstance(ResourceLabels, { onDidChangeVisibility: this._onDidChangeVisibility });`
	`1205`	`+ const resourceLabels = this._register(this.instantiationService.createInstance(ResourceLabels, { onDidChangeVisibility: this._onDidChangeVisibility }));`
`1206`	`1206`
`1207`	`1207`	`const container = $('.interactive-response-progress-tree');`
`1208`	`1208`	`this._register(createFileIconThemableTreeContainerScope(container, this.themeService));`
`@@ -1262,7 +1262,7 @@ class ContentReferencesListPool extends Disposable {`
`1262`	`1262`	`}`
`1263`	`1263`
`1264`	`1264`	`private listFactory(): WorkbenchList<IChatContentReference> {`
`1265`		`- const resourceLabels = this.instantiationService.createInstance(ResourceLabels, { onDidChangeVisibility: this._onDidChangeVisibility });`
	`1265`	`+ const resourceLabels = this._register(this.instantiationService.createInstance(ResourceLabels, { onDidChangeVisibility: this._onDidChangeVisibility }));`
`1266`	`1266`
`1267`	`1267`	`const container = $('.chat-used-context-list');`
`1268`	`1268`	`this._register(createFileIconThemableTreeContainerScope(container, this.themeService));`