Closed Bug 1463496 Opened 2 years ago Closed 2 years ago

Enable speech synthesis on Android

Categories

(Core :: Web Speech, enhancement)

62 Branch
ARM
Android
enhancement
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla62
Tracking Status
firefox62 --- verified

People

(Reporter: eeejay, Assigned: eeejay)

References

Details

(Keywords: dev-doc-complete, Whiteboard: [webcompat])

Attachments

(1 file, 1 obsolete file)

No description provided.
Attachment #8979685 - Flags: review?(bugs)
Comment on attachment 8979685 [details] [diff] [review]
Enable WebSpeech synthesis in Android. r?smaug

rs+ I guess.
How much testing has it got? Does it work with geckoview?
Attachment #8979685 - Flags: review?(bugs) → review+
(In reply to Olli Pettay [:smaug] from comment #2)
> Comment on attachment 8979685 [details] [diff] [review]
> Enable WebSpeech synthesis in Android. r?smaug
> 
> rs+ I guess.
> How much testing has it got? Does it work with geckoview?

Not much testing, that's why I think it should be enabled. Would you do this differently? Only enable in pre-release? I figured if it rides the trains we should uncover any issues there may be.

It works in geckoview.
Would it be helpful to have the web compat team do some testing in Nightly? Are there particular sites we are concerned about?
Whiteboard: [webcompat]
(In reply to Panos Astithas [:past] (please ni?) from comment #4)
> Would it be helpful to have the web compat team do some testing in Nightly?
> Are there particular sites we are concerned about?

Thanks! This kind of testing would be amazing. I don't actually know of popular sites that use this :)

I made a demo site a while back: http://eeejay.github.io/webspeechdemos/

Note, to compare apples to apples, you need to run this against android chrome. the android platform tts is not great, so we couldn't implement every feature, like pause. chrome had to leave things unimplemented as well.
Oops. Forgot to tweak the interfaces test..
Attachment #8982290 - Flags: review?(bugs)
Attachment #8979685 - Attachment is obsolete: true
Attachment #8982290 - Flags: review?(bugs) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7fa83e49f708
Enable WebSpeech synthesis in Android. r=smaug
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/7fa83e49f708
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Devices:
 - Nexus 5 (Android 6.0.1);
 - Nexus 9 (Android 6.0.1);
 - Samsung Galaxy Tab 3 (Android 7.0).


Verified this using the site provided by Eltan in Comment 5. There was no noticeable difference between Chrome and FF when testing this, speech worked perfectly on all of the devices using a variety of pitch, rate and volume.
Status: RESOLVED → VERIFIED
OS: Unspecified → Android
Hardware: Unspecified → ARM
Version: unspecified → 62 Branch
You need to log in before you can comment on or make changes to this bug.