Adding logs to determine focus state of NativeEditContext (#260499)

Author: aiday-mar

src/vs/editor/browser/controller/editContext/native/nativeEditContext.ts

@@ -5,7 +5,7 @@
 
 import './nativeEditContext.css';
 import { isFirefox } from '../../../../../base/browser/browser.js';
-import { addDisposableListener, getWindow, getWindowId } from '../../../../../base/browser/dom.js';
+import { addDisposableListener, getActiveElement, getWindow, getWindowId } from '../../../../../base/browser/dom.js';
 import { FastDomNode } from '../../../../../base/browser/fastDomNode.js';
 import { StandardKeyboardEvent } from '../../../../../base/browser/keyboardEvent.js';
 import { KeyCode } from '../../../../../base/common/keyCodes.js';
@@ -100,7 +100,8 @@ export class NativeEditContext extends AbstractEditContext {
 		overflowGuardContainer.appendChild(this._imeTextArea);
 		this._parent = overflowGuardContainer.domNode;
 
-		this._focusTracker = this._register(new FocusTracker(this.domNode.domNode, (newFocusValue: boolean) => {
+		this._focusTracker = this._register(new FocusTracker(logService, this.domNode.domNode, (newFocusValue: boolean) => {
+			logService.trace('NativeEditContext#handleFocusChange : ', newFocusValue);
 			this._screenReaderSupport.handleFocusChange(newFocusValue);
 			this._context.viewModel.setHasFocus(newFocusValue);
 		}));
@@ -310,6 +311,11 @@ export class NativeEditContext extends AbstractEditContext {
 		this._screenReaderSupport.onWillPaste();
 	}
 
+	public onWillCopy(): void {
+		this.logService.trace('NativeEditContext#onWillCopy');
+		this.logService.trace('NativeEditContext#isFocused : ', this.domNode.domNode === getActiveElement());
+	}
+
 	public writeScreenReaderContent(): void {
 		this._screenReaderSupport.writeScreenReaderContent();
 	}

src/vs/editor/browser/controller/editContext/native/nativeEditContextUtils.ts

@@ -5,6 +5,7 @@
 
 import { addDisposableListener, getActiveElement, getShadowRoot } from '../../../../../base/browser/dom.js';
 import { IDisposable, Disposable } from '../../../../../base/common/lifecycle.js';
+import { ILogService } from '../../../../../platform/log/common/log.js';
 
 export interface ITypeData {
 	text: string;
@@ -18,11 +19,13 @@ export class FocusTracker extends Disposable {
 	private _isPaused: boolean = false;
 
 	constructor(
+		@ILogService _logService: ILogService,
 		private readonly _domNode: HTMLElement,
 		private readonly _onFocusChange: (newFocusValue: boolean) => void,
 	) {
 		super();
 		this._register(addDisposableListener(this._domNode, 'focus', () => {
+			_logService.trace('NativeEditContext.focus');
 			if (this._isPaused) {
 				return;
 			}
@@ -32,6 +35,7 @@ export class FocusTracker extends Disposable {
 			this.refreshFocusState();
 		}));
 		this._register(addDisposableListener(this._domNode, 'blur', () => {
+			_logService.trace('NativeEditContext.blur');
 			if (this._isPaused) {
 				return;
 			}

src/vs/editor/contrib/clipboard/browser/clipboard.ts

@@ -213,12 +213,14 @@ function registerExecCommandImpl(target: MultiCommand | undefined, browserComman
 			}
 			// TODO this is very ugly. The entire copy/paste/cut system needs a complete refactoring.
 			if (focusedEditor.getOption(EditorOption.effectiveEditContext) && browserCommand === 'cut') {
+				logCopyCommand(focusedEditor);
 				// execCommand(copy) works for edit context, but not execCommand(cut).
 				logService.trace('registerExecCommandImpl (before execCommand copy)');
 				focusedEditor.getContainerDomNode().ownerDocument.execCommand('copy');
 				focusedEditor.trigger(undefined, Handler.Cut, undefined);
 				logService.trace('registerExecCommandImpl (after execCommand copy)');
 			} else {
+				logCopyCommand(focusedEditor);
 				logService.trace('registerExecCommandImpl (before execCommand ' + browserCommand + ')');
 				focusedEditor.getContainerDomNode().ownerDocument.execCommand(browserCommand);
 				logService.trace('registerExecCommandImpl (after execCommand ' + browserCommand + ')');
@@ -239,6 +241,16 @@ function registerExecCommandImpl(target: MultiCommand | undefined, browserComman
 	});
 }
 
+function logCopyCommand(editor: ICodeEditor) {
+	const editContextEnabled = editor.getOption(EditorOption.effectiveEditContext);
+	if (editContextEnabled) {
+		const nativeEditContext = NativeEditContextRegistry.get(editor.getId());
+		if (nativeEditContext) {
+			nativeEditContext.onWillCopy();
+		}
+	}
+}
+
 registerExecCommandImpl(CutAction, 'cut');
 registerExecCommandImpl(CopyAction, 'copy');