Skip to content

Add input latency tracking to nativeEditContext#278209

Merged
Tyriar merged 2 commits intomainfrom
Tyriar-patch-1
Nov 19, 2025
Merged

Add input latency tracking to nativeEditContext#278209
Tyriar merged 2 commits intomainfrom
Tyriar-patch-1

Conversation

@Tyriar
Copy link
Member

@Tyriar Tyriar commented Nov 18, 2025

@Tyriar Tyriar added this to the November 2025 milestone Nov 18, 2025
@Tyriar Tyriar self-assigned this Nov 18, 2025
Copilot AI review requested due to automatic review settings November 18, 2025 20:59
@Tyriar Tyriar enabled auto-merge November 18, 2025 20:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds input latency tracking to the native EditContext implementation by integrating calls to the inputLatency performance monitoring utility at key interaction points.

  • Tracks user input events (keydown, keyup, beforeinput, textupdate, selectionchange) to measure input latency
  • Integrates the existing inputLatency performance monitoring utility from base/browser/performance.js

Comment on lines 173 to +174
this._register(editContextAddDisposableListener(this._editContext, 'textupdate', (e) => {
inputLatency.onInput();
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasn't totally sure about this one. Is there an equivalent to input for edit context?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Tyriar thanks for the ping. Just got round to looking at my notifications. There is no direct equivalent, the closest would be indeed textupdate

@Tyriar Tyriar merged commit 431ce3a into main Nov 19, 2025
28 checks passed
@Tyriar Tyriar deleted the Tyriar-patch-1 branch November 19, 2025 18:07
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Jan 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Report input latency when native edit context is enabled

4 participants