Merge pull request microsoft#164571 from microsoft/tyriar/revert

Revert "Merge pull request microsoft#163958 from microsoft/tyriar/161622_brack…

Author: Tyriar

src/vs/editor/common/model/bracketPairsTextModelPart/bracketPairsTree/bracketPairsTree.ts

@@ -40,8 +40,6 @@ export class BracketPairsTree extends Disposable {
 	private readonly denseKeyProvider = new DenseKeyProvider<string>();
 	private readonly brackets = new LanguageAgnosticBracketTokens(this.denseKeyProvider, this.getLanguageConfiguration);
 
-	private readonly parseQueue: TextEditInfo[][] = [];
-
 	public didLanguageChange(languageId: string): boolean {
 		return this.brackets.didLanguageChange(languageId);
 	}
@@ -92,7 +90,10 @@ export class BracketPairsTree extends Disposable {
 				toLength(r.toLineNumber - r.fromLineNumber + 1, 0)
 			)
 		);
-		this.queueParsing(edits);
+		this.astWithTokens = this.parseDocumentFromTextBuffer(edits, this.astWithTokens, false);
+		if (!this.initialAstWithoutTokens) {
+			this.didChangeEmitter.fire();
+		}
 	}
 
 	public handleContentChanged(change: IModelContentChangedEvent) {
@@ -104,23 +105,11 @@ export class BracketPairsTree extends Disposable {
 				lengthOfString(c.text)
 			);
 		}).reverse();
-		this.queueParsing(edits);
-	}
 
-	private queueParsing(edits: TextEditInfo[]): void {
-		// Queues parsing of the document so that it only runs before the state is actually
-		// requested, this helps reduce editor input latency by doing parsing lazily instead of eagerly.
-		this.parseQueue.push(edits);
-	}
-
-	private ensureState() {
-		for (const edits of this.parseQueue) {
-			this.astWithTokens = this.parseDocumentFromTextBuffer(edits, this.astWithTokens, false);
-			if (this.initialAstWithoutTokens) {
-				this.initialAstWithoutTokens = this.parseDocumentFromTextBuffer(edits, this.initialAstWithoutTokens, false);
-			}
+		this.astWithTokens = this.parseDocumentFromTextBuffer(edits, this.astWithTokens, false);
+		if (this.initialAstWithoutTokens) {
+			this.initialAstWithoutTokens = this.parseDocumentFromTextBuffer(edits, this.initialAstWithoutTokens, false);
 		}
-		this.parseQueue.length = 0;
 	}
 
 	//#endregion
@@ -138,7 +127,6 @@ export class BracketPairsTree extends Disposable {
 	}
 
 	public getBracketsInRange(range: Range): CallbackIterable<BracketInfo> {
-		this.ensureState();
 		const startOffset = toLength(range.startLineNumber - 1, range.startColumn - 1);
 		const endOffset = toLength(range.endLineNumber - 1, range.endColumn - 1);
 		return new CallbackIterable(cb => {
@@ -148,7 +136,6 @@ export class BracketPairsTree extends Disposable {
 	}
 
 	public getBracketPairsInRange(range: Range, includeMinIndentation: boolean): CallbackIterable<BracketPairWithMinIndentationInfo> {
-		this.ensureState();
 		const startLength = positionToLength(range.getStartPosition());
 		const endLength = positionToLength(range.getEndPosition());
 
@@ -160,13 +147,11 @@ export class BracketPairsTree extends Disposable {
 	}
 
 	public getFirstBracketAfter(position: Position): IFoundBracket | null {
-		this.ensureState();
 		const node = this.initialAstWithoutTokens || this.astWithTokens!;
 		return getFirstBracketAfter(node, lengthZero, node.length, positionToLength(position));
 	}
 
 	public getFirstBracketBefore(position: Position): IFoundBracket | null {
-		this.ensureState();
 		const node = this.initialAstWithoutTokens || this.astWithTokens!;
 		return getFirstBracketBefore(node, lengthZero, node.length, positionToLength(position));
 	}