No more focus event issue after reloading appModules

[CyrilleB79]

Link to issue number:

Fixes #17323

Summary of the issue:

After a reload of plugins (NVDA+control+f3), focus events do not report the new focus as expected.

During focus event, NVDA tests which tree interceptors are alive or not to clean up those which are dead. In this test, for MSHTML, NVDA looks at root.appModule.appName, root being the root object associated to the virtual MSHTML buffer.

However, root.appModule may be None after plugins are reloaded.

Description of user facing changes

After plugins are reloaded while a browseable message is opened, the focus will no longer fail to be reported.

Description of development approach

Fixed the logic for MSHTML

Testing strategy:

Tested STR in #17323.

Known issues with pull request:

None

Code Review Checklist:

• Documentation:
  • Change log entry
  • User Documentation
  • Developer / Technical Documentation
  • Context sensitive help for GUI changes
• Testing:
  • Unit tests
  • System (end to end) tests
  • Manual testing
• UX of all users considered:
  • Speech
  • Braille
  • Low Vision
  • Different web browsers
  • Localization in other languages / culture than English
• API is compatible with existing add-ons.
• Security precautions taken.

@coderabbitai summary

[seanbudd]

I merged #17334 from @LeonarddeR. can you resolve conflicts?

[CyrilleB79]

@seanbudd, I have upmerged from latest master.

The changes that I had made in source/keyboardHandler.py have been taken by @LeonarddeR in #17334 so this file is not part of this PR anymore.

[CyrilleB79]

Also updated the initial description consequently.

[seanbudd]

Thanks @CyrilleB79