Don't auto-focus a "deemphasized" frame while debugging (microsoft#184962)

roblourens · web-flow · commit ca3c3466344e · 2023-06-12T16:55:45.000-07:00
Fix microsoft#184640
diff --git a/src/vs/workbench/contrib/debug/common/debugModel.ts b/src/vs/workbench/contrib/debug/common/debugModel.ts
@@ -528,7 +528,7 @@ export class Thread implements IThread {
 		const firstAvailableStackFrame = callStack.find(sf => !!(sf &&
 			((this.stoppedDetails?.reason === 'instruction breakpoint' || (this.stoppedDetails?.reason === 'step' && this.lastSteppingGranularity === 'instruction')) && sf.instructionPointerReference) ||
 			(sf.source && sf.source.available && sf.source.presentationHint !== 'deemphasize')));
-		return firstAvailableStackFrame || (callStack.length > 0 ? callStack[0] : undefined);
+		return firstAvailableStackFrame;
 	}
 
 	get stateLabel(): string {

Original file line number	Diff line number	Diff line change
`@@ -528,7 +528,7 @@ export class Thread implements IThread {`
`528`	`528`	`const firstAvailableStackFrame = callStack.find(sf => !!(sf &&`
`529`	`529`	`((this.stoppedDetails?.reason === 'instruction breakpoint' \|\| (this.stoppedDetails?.reason === 'step' && this.lastSteppingGranularity === 'instruction')) && sf.instructionPointerReference) \|\|`
`530`	`530`	`(sf.source && sf.source.available && sf.source.presentationHint !== 'deemphasize')));`
`531`		`- return firstAvailableStackFrame \|\| (callStack.length > 0 ? callStack[0] : undefined);`
	`531`	`+ return firstAvailableStackFrame;`
`532`	`532`	`}`
`533`	`533`
`534`	`534`	`get stateLabel(): string {`