Remove nbsp replacing from buffer content tracker

Fixes microsoft#181834

Author: Tyriar

src/vs/workbench/contrib/terminalContrib/accessibility/browser/bufferContentTracker.ts

@@ -82,9 +82,8 @@ export class BufferContentTracker {
 			const isWrapped = buffer.getLine(i + 1)?.isWrapped;
 			currentLine += line.translateToString(!isWrapped);
 			if (currentLine && !isWrapped || i === (buffer.baseY + this._xterm.raw.rows - 1)) {
-				const line = replaceWithNonBreakingSpaces(currentLine);
 				if (line.length) {
-					cachedLines.push(line);
+					cachedLines.push(currentLine);
 					currentLine = '';
 				}
 			}
@@ -122,18 +121,13 @@ export class BufferContentTracker {
 			const isWrapped = buffer.getLine(i + 1)?.isWrapped;
 			currentLine += line.translateToString(!isWrapped);
 			if (currentLine && !isWrapped || i === (buffer.baseY + this._xterm.raw.rows - 1)) {
-				const line = replaceWithNonBreakingSpaces(currentLine);
 				if (line.length) {
 					this._priorEditorViewportLineCount++;
-					this._lines.push(line);
+					this._lines.push(currentLine);
 					currentLine = '';
 				}
 			}
 		}
 		this._logService.debug('Viewport content update complete, ', this._lines.length, ' lines in the viewport');
 	}
 }
-
-export function replaceWithNonBreakingSpaces(s: string): string {
-	return s.replace(new RegExp('  ', 'g'), ' \xA0');
-}

src/vs/workbench/contrib/terminalContrib/accessibility/test/browser/bufferContentTracker.test.ts

@@ -20,7 +20,7 @@ import { TerminalConfigHelper } from 'vs/workbench/contrib/terminal/browser/term
 import { writeP } from 'vs/workbench/contrib/terminal/browser/terminalTestHelpers';
 import { XtermTerminal } from 'vs/workbench/contrib/terminal/browser/xterm/xtermTerminal';
 import { ITerminalConfiguration } from 'vs/workbench/contrib/terminal/common/terminal';
-import { BufferContentTracker, replaceWithNonBreakingSpaces } from 'vs/workbench/contrib/terminalContrib/accessibility/browser/bufferContentTracker';
+import { BufferContentTracker } from 'vs/workbench/contrib/terminalContrib/accessibility/browser/bufferContentTracker';
 import { ILifecycleService } from 'vs/workbench/services/lifecycle/common/lifecycle';
 import { TestLifecycleService } from 'vs/workbench/test/browser/workbenchTestServices';
 import { Terminal } from 'xterm';
@@ -99,10 +99,10 @@ suite('Buffer Content Tracker', () => {
 		await writeAndAssertBufferState(promptPlusData, 6, xterm.raw, bufferTracker);
 		await writeP(xterm.raw, '\x1b[3Ainserteddata');
 		await bufferTracker.update();
-		assert.deepStrictEqual(bufferTracker.lines, [promptPlusData, promptPlusData, `${promptPlusData}inserteddata`, promptPlusData, promptPlusData, promptPlusData].map(s => replaceWithNonBreakingSpaces(s)));
+		assert.deepStrictEqual(bufferTracker.lines, [promptPlusData, promptPlusData, `${promptPlusData}inserteddata`, promptPlusData, promptPlusData, promptPlusData]);
 	});
 	test('should refresh viewport with full scrollback', async () => {
-		const content = replaceWithNonBreakingSpaces(`${prompt}\r\n`.repeat(1030).trimEnd());
+		const content = `${prompt}\r\n`.repeat(1030).trimEnd();
 		await writeP(xterm.raw, content);
 		await bufferTracker.update();
 		await writeP(xterm.raw, '\x1b[4Ainsertion');
@@ -115,7 +115,7 @@ suite('Buffer Content Tracker', () => {
 		const content = `${prompt}\r\n`.repeat(1036).trimEnd();
 		await writeP(xterm.raw, content);
 		await bufferTracker.update();
-		const expected = content.split('\r\n').map(s => replaceWithNonBreakingSpaces(s));
+		const expected = content.split('\r\n');
 		// delete the 6 lines that should be trimmed
 		for (let i = 0; i < 6; i++) {
 			expected.pop();
@@ -134,6 +134,6 @@ async function writeAndAssertBufferState(data: string, rows: number, terminal: T
 	await writeP(terminal, content);
 	await bufferTracker.update();
 	assert.strictEqual(bufferTracker.lines.length, rows);
-	assert.deepStrictEqual(bufferTracker.lines, content.split('\r\n').map(s => replaceWithNonBreakingSpaces(s)));
+	assert.deepStrictEqual(bufferTracker.lines, content.split('\r\n'));
 }