New issue
Advanced search Search tips

Issue 706044 link

Starred by 9 users

Issue metadata

Status: Verified
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: 2017-03-28
OS: Chrome
Pri: 1
Type: Bug
Team-Accessibility



Sign in to add a comment

Tts stops responding and goes silent on some common strings

Project Member Reported by dtseng@chromium.org, Mar 28 2017

Issue description

Execute chrome.tts.speak('hello    world')


result:
sbusequent calls to tts result in no output

This has been traced to an upstream bug.
 

Comment 1 by dtseng@chromium.org, Mar 28 2017

Labels: OS-Chrome
Labels: ReleaseBlock-Stable
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/assets/+/6e3b4c46834f5ee80b50c7728ac7682bdf99fdfc

commit 6e3b4c46834f5ee80b50c7728ac7682bdf99fdfc
Author: David Tseng <dtseng@chromium.org>
Date: Tue Mar 28 23:43:20 2017

Merge tts fixes from upstream

BUG= chromium:706044 
TEST=unpack and use tts_service_x86-64.nexe on a linux Chrome OS build. Verify common usage (navigation through complex sites) works and that reproducing strings when passed to chrome.tts.speak don't result in silence.
On an arm device, build and deploy patched tts binaries. Verify the same as above.

Change-Id: I9eb7ff21787dedbedb5584fcb46e68a8f7880719
Reviewed-on: https://chromium-review.googlesource.com/462000
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Tested-by: David Tseng <dtseng@chromium.org>

[modify] https://crrev.com/6e3b4c46834f5ee80b50c7728ac7682bdf99fdfc/speech_synthesis/patts/tts_service_x86-64.nexe.zip
[modify] https://crrev.com/6e3b4c46834f5ee80b50c7728ac7682bdf99fdfc/speech_synthesis/patts/tts_service_x86-32.nexe.zip
[modify] https://crrev.com/6e3b4c46834f5ee80b50c7728ac7682bdf99fdfc/speech_synthesis/patts/tts_service_arm.nexe.zip

Comment 4 by dtseng@chromium.org, Mar 28 2017

Labels: Merge-Request-58
Project Member

Comment 5 by sheriffbot@chromium.org, Mar 29 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 30 2017

Labels: merge-merged-release-R58-9334.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/assets/+/93ef941bd0fca02e851c7cd241665f95f8771c29

commit 93ef941bd0fca02e851c7cd241665f95f8771c29
Author: David Tseng <dtseng@chromium.org>
Date: Thu Mar 30 23:23:32 2017

Merge to m58: Merge tts fixes from upstream

BUG= chromium:706044 
TEST=unpack and use tts_service_x86-64.nexe on a linux Chrome OS build. Verify common usage (navigation through complex sites) works and that reproducing strings when passed to chrome.tts.speak don't result in silence.
On an arm device, build and deploy patched tts binaries. Verify the same as above.

Change-Id: I9eb7ff21787dedbedb5584fcb46e68a8f7880719
Reviewed-on: https://chromium-review.googlesource.com/462000
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Tested-by: David Tseng <dtseng@chromium.org>
(cherry picked from commit 6e3b4c46834f5ee80b50c7728ac7682bdf99fdfc)
Reviewed-on: https://chromium-review.googlesource.com/464070
Reviewed-by: David Tseng <dtseng@chromium.org>
Commit-Queue: David Tseng <dtseng@chromium.org>
Trybot-Ready: David Tseng <dtseng@chromium.org>

[modify] https://crrev.com/93ef941bd0fca02e851c7cd241665f95f8771c29/speech_synthesis/patts/tts_service_x86-64.nexe.zip
[modify] https://crrev.com/93ef941bd0fca02e851c7cd241665f95f8771c29/speech_synthesis/patts/tts_service_x86-32.nexe.zip
[modify] https://crrev.com/93ef941bd0fca02e851c7cd241665f95f8771c29/speech_synthesis/patts/tts_service_arm.nexe.zip

Project Member

Comment 7 by bugdroid1@chromium.org, Mar 30 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/assets/+/93ef941bd0fca02e851c7cd241665f95f8771c29

commit 93ef941bd0fca02e851c7cd241665f95f8771c29
Author: David Tseng <dtseng@chromium.org>
Date: Thu Mar 30 23:23:32 2017

Merge to m58: Merge tts fixes from upstream

BUG= chromium:706044 
TEST=unpack and use tts_service_x86-64.nexe on a linux Chrome OS build. Verify common usage (navigation through complex sites) works and that reproducing strings when passed to chrome.tts.speak don't result in silence.
On an arm device, build and deploy patched tts binaries. Verify the same as above.

Change-Id: I9eb7ff21787dedbedb5584fcb46e68a8f7880719
Reviewed-on: https://chromium-review.googlesource.com/462000
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Tested-by: David Tseng <dtseng@chromium.org>
(cherry picked from commit 6e3b4c46834f5ee80b50c7728ac7682bdf99fdfc)
Reviewed-on: https://chromium-review.googlesource.com/464070
Reviewed-by: David Tseng <dtseng@chromium.org>
Commit-Queue: David Tseng <dtseng@chromium.org>
Trybot-Ready: David Tseng <dtseng@chromium.org>

[modify] https://crrev.com/93ef941bd0fca02e851c7cd241665f95f8771c29/speech_synthesis/patts/tts_service_x86-64.nexe.zip
[modify] https://crrev.com/93ef941bd0fca02e851c7cd241665f95f8771c29/speech_synthesis/patts/tts_service_x86-32.nexe.zip
[modify] https://crrev.com/93ef941bd0fca02e851c7cd241665f95f8771c29/speech_synthesis/patts/tts_service_arm.nexe.zip

Project Member

Comment 8 by sheriffbot@chromium.org, Apr 3 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: lpalmaro@chromium.org dtseng@chromium.org dmazz...@chromium.org
 Issue 707592  has been merged into this issue.
 Issue 707439  has been merged into this issue.
 Issue 707536  has been merged into this issue.
 Issue 705986  has been merged into this issue.
Project Member

Comment 13 by sheriffbot@chromium.org, Apr 7 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: fixed (was: Assigned)
Can someone help verify on m58?
Status: Verified (was: Fixed)

Sign in to add a comment