Why is bracket BracketPairsTextModelPart passed into TokenizationTextModelPart?

[Tyriar]

Part of #161622 - trying to defer bracket pairs evaluation so it doesn't impact input latency, that's harder with the current dependency graph.

It looks like it only does things when events fire:

vscode/src/vs/editor/common/model/textModel.ts

Lines 323 to 332 in e3a3693

	this._bracketPairs = this._register(new BracketPairsTextModelPart(this, this._languageConfigurationService));
	this._guidesTextModelPart = this._register(new GuidesTextModelPart(this, this._languageConfigurationService));
	this._decorationProvider = this._register(new ColorizedBracketPairsDecorationProvider(this));
	this._tokenizationTextModelPart = new TokenizationTextModelPart(
	this._languageService,
	this._languageConfigurationService,
	this,
	this._bracketPairs,
	languageId
	);

Instead of TokenizationTextModelPart using it, can BracketPairsTextModelPart use TokenizationTextModelPart? ie.

this._bracketPairs = this._register(new BracketPairsTextModelPart(this, this._tokenizationTextModelPart, this._languageConfigurationService));

I can make the change, just want some guidance.

[hediet]

See 43cf81a

I think this is because of event ordering issues (@alexdima).

[Tyriar]

After our chat it doesn't look like we'll be able to defer this to after keypress, so closing this off as this change was so I could experiment with doing that conveniently.

This may still be a valid debt item but I'll leave that up to you.