Skip to content

HP Secure Browser support (#16377): Detect brchrome and flag it as not supporting IAccessible2#16392

Merged
seanbudd merged 1 commit into
nvaccess:masterfrom
tomaszw:hp-secure-browser-detect
Apr 16, 2024
Merged

HP Secure Browser support (#16377): Detect brchrome and flag it as not supporting IAccessible2#16392
seanbudd merged 1 commit into
nvaccess:masterfrom
tomaszw:hp-secure-browser-detect

Conversation

@tomaszw

@tomaszw tomaszw commented Apr 15, 2024

Copy link
Copy Markdown
Contributor

Link to issue number:

#16377

Summary of the issue:

HP Secure Browser is part of HP Wolf Security package. It is a chromium-based web browser where individual websites are isolated in separate virtual machines. To support screen readers, windows UIA trees are remoted between the virtual machines and the host. IAccessible2 interface is not remoted, hence HP Secure Browser is a rare case of chromium-browser which supports UIA but not IAccessible2.

Users have to explicitly configure in NVDA advanced options to "use UIA for Edge and other chromium based browsers", for NVDA to work with HP Secure Browser.

Description of user facing changes

Users won't have to explicitly configure UIA for all chromium-based browsers when they want to use HP Secure Browser

Description of development approach

Detect browser with a process name of "brchrome" which represents HP Secure Browser and flag it as not supporting IAccecssible2. In this scenario, UIA will be used by default for HP SB.

Testing strategy:

Manual testing

Known issues with pull request:

N/A

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.

…it as not supporting IAccessible2.

Summary of the issue:
Users have to explicitly configure in NVDA advanced options to "use UIA
for Edge and other chromium based browsers", for NVDA to work with HP Secure Browser.

Description of user facing changes:
Users won't have to explicitly configure UIA for all chromium-based browsers when
they want to use HP Secure Browser

Description of development approach:
Detect browser with a process name of "brchrome" which represents HP Secure Browser
and flag it as not supporting IAccecssible2. In this scenario, UIA will be
used by default for HP SB.
@tomaszw tomaszw mentioned this pull request Apr 15, 2024
5 tasks
@seanbudd seanbudd added the conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. label Apr 16, 2024
@tomaszw tomaszw marked this pull request as ready for review April 16, 2024 06:23
@tomaszw tomaszw requested a review from a team as a code owner April 16, 2024 06:23
@tomaszw tomaszw requested a review from gerald-hartig April 16, 2024 06:23
@seanbudd seanbudd merged commit f49d0ca into nvaccess:master Apr 16, 2024
@nvaccessAuto nvaccessAuto added this to the 2024.2 milestone Apr 16, 2024
tomaszw pushed a commit to tomaszw/nvda-hp that referenced this pull request Apr 17, 2024
…access#16395

Summary of the issue:
Changelog entries are missing for nvaccess#16392 nvaccess#16393 nvaccess#16394 nvaccess#16395

Description of user facing changes:
Added Changelog entries

Description of development approach:
N/A
tomaszw pushed a commit to tomaszw/nvda-hp that referenced this pull request Apr 17, 2024
…access#16395

Summary of the issue:
Changelog entries are missing for nvaccess#16392 nvaccess#16393 nvaccess#16394 nvaccess#16395

Description of user facing changes:
Added Changelog entries

Description of development approach:
N/A
tomaszw pushed a commit to tomaszw/nvda-hp that referenced this pull request Apr 17, 2024
…access#16395

Summary of the issue:
Changelog entries are missing for nvaccess#16392 nvaccess#16393 nvaccess#16394 nvaccess#16395

Description of user facing changes:
Added Changelog entries

Description of development approach:
N/A
seanbudd added a commit that referenced this pull request Apr 18, 2024
* Changelog entries for #16392 #16393 #16394 #16395

Summary of the issue:
Changelog entries are missing for #16392 #16393 #16394 #16395

Description of user facing changes:
Added Changelog entries

Description of development approach:
N/A

* Apply suggestions from code review

---------

Co-authored-by: Tomasz Wroblewski <tomasz.wroblewski@hp.com>
Co-authored-by: Sean Budd <seanbudd123@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants