New issue
Advanced search Search tips

Issue 855257 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: 2018-06-26
OS: Chrome
Pri: 3
Type: Bug
Team-Accessibility



Sign in to add a comment

A11y: Acapella voices with ChromeVox, pitch of message voice is too low

Project Member Reported by dsexton@chromium.org, Jun 21 2018

Issue description

Chrome: 69.0.3464.0 Canary

Steps to repro:
# Start Chromevox
# Open ChromeVox settings and set to use an Acapella tts voice
# Activate keyboard learn mode with search+o, search+k
# Press letter keys and number keys

Expected: Pitch of the keys would be slightly lower than the number keys

Actual: The pitch is much lower
 
Components: -UI>Accessibility UI>Accessibility>ChromeVox
Owner: dtseng@chromium.org
Status: Assigned (was: Untriaged)

Comment 2 by dtseng@chromium.org, Jun 26 2018

Labels: -Pri-2 Pri-3
NextAction: 2018-06-26
Owner: ----
Status: available (was: Assigned)
seems it happen with the builtin chromeos voices too
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 26 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/567cdc9a2e04b54a7ffe60f60d7029b60fe9265d

commit 567cdc9a2e04b54a7ffe60f60d7029b60fe9265d
Author: David Tseng <dtseng@chromium.org>
Date: Tue Jun 26 22:07:11 2018

Re-tune pitch in ChromeVox

- capital personality was tuned for Google tts some years ago. It is too high for the current engine
- re-do precedence logic. Do not set speech props if they were supplied by callers.

Bug:  855257 
Change-Id: I541021fa456c5283e943ecb71183e99c547a4f8a
Reviewed-on: https://chromium-review.googlesource.com/1115517
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570546}
[modify] https://crrev.com/567cdc9a2e04b54a7ffe60f60d7029b60fe9265d/chrome/browser/resources/chromeos/chromevox/chromevox/background/kbexplorer.js
[modify] https://crrev.com/567cdc9a2e04b54a7ffe60f60d7029b60fe9265d/chrome/browser/resources/chromeos/chromevox/host/interface/abstract_tts.js

Comment 5 by dtseng@chromium.org, Jun 26 2018

Status: fixed (was: Available)

Sign in to add a comment