Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from Apr 16, 2024

Conversation

tomaszw
Copy link
Contributor

@tomaszw tomaszw commented Apr 15, 2024

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
@seanbudd seanbudd merged commit f49d0ca into nvaccess:master Apr 16, 2024
1 check passed
@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.

None yet

3 participants