New issue
Advanced search Search tips

Issue 826529 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Team-Accessibility

Blocked on:
issue 874295

Blocking:
issue 594887



Sign in to add a comment

Support select-to-speak in multi-process mash

Project Member Reported by jamescook@chromium.org, Mar 27 2018

Issue description

Right now chrome/browser calls into ash for this feature, which won't work on mustash:

chromeos/chrome_browser_main_chromeos.cc:#include "ash/root_window_controller.h"
chromeos/chrome_browser_main_chromeos.cc:#include "ash/shell.h"
 
Labels: -Proj-Mustash-Mash
Components: UI>Accessibility>SelectToSpeak
Labels: Proj-Mash-SingleProcess
Status: Available (was: Untriaged)
Summary: mash: Support select-to-speak (was: mash: Support select-to-speak (spoken feedback))
Blockedon: 874295
Looks like the Automation API also doesn't work well under mash:
* hit testing doesn't work
* chrome.automation.getFocus doesn't return
Labels: Pri-2
Owner: katie@chromium.org
Status: Fixed (was: Available)
I think select-to-speak should actually work under Mash (and for window service windows after Issue 888145 is fixed)
The pre-target EventHandler is installed on the Ash side:
 https://cs.chromium.org/chromium/src/ash/events/select_to_speak_event_handler.cc?rcl=eeb9122621630cdc1cd3600de32ff1f17f044f75&l=27
And that class uses a mojom::SelectToSpeakEventHandlerDelegate interface to dispatch events to the extension:
 https://cs.chromium.org/chromium/src/ash/public/interfaces/accessibility_controller.mojom?rcl=eeb9122621630cdc1cd3600de32ff1f17f044f75&l=105
I'm going to mark this piece fixed (per katie@'s work on  Issue 874295 ), thanks!
Select-to-speak won't fully work under Mash until hit-testing and chrome.automation.getFocus work. Should we leave this open until the feature actually works in Mash -- and the integration tests can run in Mash?
Cc: jamescook@chromium.org
Labels: -Proj-Mash-SingleProcess Proj-Mash-MultiProcess
Owner: ----
Status: Available (was: Fixed)
Summary: Support select-to-speak in multi-process mash (was: mash: Support select-to-speak)
We can use this for multi-process mash. As katie says, there's work to do.

SingleProcessMash select-to-speak is  issue 889093 

Sign in to add a comment