fix(web): enhances integrated test stability

[jahorton]

This corresponds to an occasionally-seen test failure error from the build logs in which we'd get an error corresponding to this block of code:

keyman/web/src/app/browser/src/contextManager.ts

Lines 533 to 536 in f5b28aa

	if(lastElem && lastElem._kmwAttachment.keyboard != null) {
	lastElem._kmwAttachment.keyboard = keyboardID;
	lastElem._kmwAttachment.languageCode = langCode;
	} else {

The error would match line 533, stating an inability to read the keyboard field due to _kmwAttachment being null / undefined.

The actual trigger:

keyman/web/src/app/browser/src/context/pageIntegrationHandlers.ts

Lines 81 to 91 in f5b28aa

	/**
	* Reset context when entering or exiting the active element.
	* Will also trigger OSK shift state / layer reset.
	**/
	private pageFocusHandler: (e: FocusEvent) => boolean = () => {
	if(!this.focusAssistant.maintainingFocus && this.engine.osk?.vkbd) {
	this.engine.contextManager.deactivateCurrentTarget();
	this.engine.contextManager.resetContext();
	}
	return false;
	}

This event handler is called on page events, rather than events related to attached elements. But... on line 87, it calls a method that relates to attached elements... and so a niche case related to management of recently-attached elements had slipped through the cracks until now.

Discovered during work on #9657 and #9690. I've gone ahead and included its changes there, but this is easy enough to 🍒 out of the feature branch and make its own PR here.

@keymanapp-test-bot skip

[keymanapp-test-bot]

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

• Android
  • Keyman for Android apk
  • FirstVoices for Android 16.0 apk
  • FirstVoices Keyboards for Android apk
  • KeyboardHarness apk
  • Keyman for Android 16.0 apk
  • KMSample1 apk
  • KMSample2 apk
• Developer
  • Keyman Developer - build failure
  • Compiler Regression Tests - build failure
  • kmcomp.zip - build failure
• iOS
  • Keyman for iOS (simulator image)
  • FirstVoices Keyboards for iOS (simulator image)
  • TestFlight internal PR build version - 17.0.187 (0.9718.9380)
• Keyboards
  • Test Keyboards - build failure
• Web
  • KeymanWeb Test Home
• Windows
  • Keyman for Windows
  • FirstVoices Keyboards for Windows
  • Text Editor (32 bit)
  • Text Editor (64 bit)

[mcdurdin]

You need to update your master branch locally... iOS builds failing due to expired certificate.

[mcdurdin]

Can you add a link to a build which failed before this fix?

[jahorton]

Can you add a link to a build which failed before this fix?

That would take some hunting. It's something I've sporadically seen in the past, but I don't remember seeing it within the past week. If you want me to spend that time, I can... but I figure it's effort best spent elsewhere at this point.

[mcdurdin]

That would take some hunting. It's something I've sporadically seen in the past, but I don't remember seeing it within the past week. If you want me to spend that time, I can... but I figure it's effort best spent elsewhere at this point.

I'd appreciate it because right now I don't know what this is fixing.

[jahorton]

That would take some hunting. It's something I've sporadically seen in the past, but I don't remember seeing it within the past week. If you want me to spend that time, I can... but I figure it's effort best spent elsewhere at this point.

I'd appreciate it because right now I don't know what this is fixing.

I don't see anything recent enough in any of the listed TC logs. I know I've seen something about this in the past, but apparently it's too far back.

[mcdurdin]

LGTM? It's a little hard to be clear on the exact sequence of events and the complex web of event handlers but it's basically a race condition, yeah?

[keyman-server]

Changes in this pull request will be available for download in Keyman version 17.0.196-alpha