Avoid error in PowerPoint slideshows when focus highlighting is on

[LeonarddeR]

Link to issue number:

None

Summary of the issue:

NVDA raises the following error when the highlighter is on in a PowerPoint slide show

Error running handler <bound method NVDAHighlighter.handleBrowseModeMove of <visionEnhancementProviders.NVDAHighlighter.NVDAHighlighter object at 0x088D25D0>> for <extensionPoints.Action object at 0x0640C350>
Traceback (most recent call last):
  File "extensionPoints\__init__.pyc", line 56, in notify
  File "extensionPoints\util.pyc", line 216, in callWithSupportedKwargs
  File "visionEnhancementProviders\NVDAHighlighter.pyc", line 497, in handleBrowseModeMove
  File "visionEnhancementProviders\NVDAHighlighter.pyc", line 481, in updateContextRect
  File "vision\util.pyc", line 79, in getContextRect
  File "vision\util.pyc", line 48, in getCaretRect
  File "vision\util.pyc", line 94, in getRectFromTextInfo
  File "baseObject.pyc", line 39, in __get__
  File "NVDAObjects\__init__.pyc", line 72, in _get_boundingRects
AttributeError: 'ReviewableSlideshowTreeInterceptor' object has no attribute 'hasIrrelevantLocation'

Description of user facing changes

No longer an error in the log

Description of development approach

Added an override of boundingRects on SlideShowTreeInterceptorTextInfo

Testing strategy:

Ensured that the error no longer occurs.

Known issues with pull request:

I didn't add a changelog error as this was only revealed by an error sound, and the release versions of NVDA have their error sound disabled, so this is an invisible bug.

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

[AppVeyorBot]

• PASS: Translation comments check.
• PASS: License check.
• PASS: Unit tests.
• PASS: Lint check.
• Build execution time has reached the maximum allowed time for your plan (60 minutes).

See test results for failed build of commit a6b0ea416b

[hwf1324]

Related: #15461

[LeonarddeR]

This pr doesn't fix #15461. Browse mode in powerpoint is simply based on presenting the text of a slide in browse mode, therefore information about the position of the caret is not available.

[hwf1324]

This pr doesn't fix #15461. Browse mode in powerpoint is simply based on presenting the text of a slide in browse mode, therefore information about the position of the caret is not available.

That means this PR just eliminates the error in the log?

[AppVeyorBot]

• PASS: Translation comments check.
• PASS: License check.
• PASS: Unit tests.
• FAIL: Lint check. See test results and lint artifacts for more information.
• PASS: System tests (tags: installer NVDA).
• Build (for testing PR): https://ci.appveyor.com/api/buildjobs/adue7pxvuvs86ovq/artifacts/output/nvda_snapshot_pr17724-35450,70e823ce.exe
• CI timing (mins):
  INIT 0.0,
  INSTALL_START 1.2,
  INSTALL_END 1.0,
  BUILD_START 0.0,
  BUILD_END 18.6,
  TESTSETUP_START 0.0,
  TESTSETUP_END 0.4,
  TEST_START 0.0,
  TEST_END 19.1,
  FINISH_END 0.1

See test results for failed build of commit 70e823ced0