Closed
Bug 1222697
Opened 9 years ago
Closed 8 years ago
[e10s] Audio Indicator doesn't appear when using Web Speech Synthesis
Categories
(Core :: Web Speech, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla48
People
(Reporter: m_kato, Assigned: m_kato)
References
Details
Attachments
(1 file, 1 obsolete file)
This is e10s only. When turned on e10s, audio indicator doesn't appear.
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Comment 2•9 years ago
|
||
Humm, DisptachEnd will crash on try server....
Comment 3•9 years ago
|
||
Should this block e10s rollout?
tracking-e10s:
--- → ?
Flags: needinfo?(m_kato)
Assignee | ||
Comment 4•9 years ago
|
||
(In reply to Jim Mathies [:jimm] from comment #3) > Should this block e10s rollout? Yes. Web Speech Synthesis API supports on 45+ as default, so we should support this since it works on non-e10s. (But Chrome doesn't support it when using speech API :-<.)
Flags: needinfo?(m_kato)
Assignee | ||
Comment 5•9 years ago
|
||
Of course, rollout is 45+, it should be blocker. If not, it shouldn't.
Updated•8 years ago
|
Comment 6•8 years ago
|
||
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:46.0) Gecko/20100101 Firefox/46.0 Build ID: 20160207004019 Reproducible also on the latest Aurora 46.0a2 (tested on Mac OS X 10.10, Ubuntu 12.04 and Windows 10) - Audio indicator is not displayed when using Web Speech Synthesis and e10s is enabled. Used the following demo: http://eeejay.github.io/webspeechdemos/
Assignee | ||
Comment 9•8 years ago
|
||
Web Speech Synthesis API is still turned off release build, but I have a fix for this.
Flags: needinfo?(m_kato)
Comment 10•8 years ago
|
||
(In reply to Makoto Kato [:m_kato] from comment #9) > Web Speech Synthesis API is still turned off release build, but I have a fix > for this. Oh, ok, I'll move this to our P1 post first rollout list.
Priority: -- → P1
Assignee | ||
Updated•8 years ago
|
Attachment #8684762 -
Attachment is obsolete: true
Assignee | ||
Comment 11•8 years ago
|
||
Create audio channel agent even if e10s. Dispatch*Inner won't be called from child process's SpeechSynthesisRequestChild, so it is created at Dispatch*Impl. Review commit: https://reviewboard.mozilla.org/r/43371/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/43371/
Attachment #8736542 -
Flags: review?(eitan)
Comment 12•8 years ago
|
||
https://reviewboard.mozilla.org/r/43369/#review40563 Looks good. I tested, and muting the tab doesn't work, obviously. This is still a bit misleading. But I guess that is another bug.
Comment 13•8 years ago
|
||
Comment on attachment 8736542 [details] MozReview Request: Bug 1222697 - Audio Indicator doesn't appear when using Web Speech Synthesis. r?eeejay https://reviewboard.mozilla.org/r/43371/#review40565
Attachment #8736542 -
Flags: review?(eitan) → review+
Comment 15•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5027257b611e
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Updated•8 years ago
|
status-firefox47:
--- → wontfix
You need to log in
before you can comment on or make changes to this bug.
Description
•