Update tts to include new voices

Update tts to include new voices

BUG=333288
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=251970

[Dmitry Polukhin, 2014-02-15 05:16:22]

lgtm

https://codereview.chromium.org/167893003/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/speech_synthesis/manifest.json (right):

https://codereview.chromium.org/167893003/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/speech_synthesis/manifest.json:27: {
We usually avoid HTTP in permissions for component apps. Do we really need it?

[Zachary Kuznia, 2014-02-18 23:49:12]

https://codereview.chromium.org/167893003/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/speech_synthesis/manifest.json (right):

https://codereview.chromium.org/167893003/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/speech_synthesis/manifest.json:27: {
On 2014/02/15 05:16:22, Dmitry Polukhin wrote:
> We usually avoid HTTP in permissions for component apps. Do we really need it?

Done.

[Zachary Kuznia, 2014-02-18 23:49:34]

Hello thakis,

Could you do an OWNERS review of this?

Thanks,
-Zach

[Nico, 2014-02-19 00:45:50]

lgtm stamp, but I have no idea what's going on here :-)

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
File chrome/browser/resources/chromeos/speech_synthesis/manifest.json (right):

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
chrome/browser/resources/chromeos/speech_synthesis/manifest.json:4: "name": "US
English Female Text-to-speech (by Google)",
"US English"? Looks like it has many more languages?

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
chrome/browser/resources/chromeos/speech_synthesis/manifest.json:19:
"voice_data_hmm_pt-BR_2.js",
I don't see these files checked in, I suppose that's expected?

Does it make sense to move the foo_2.js files to foo.js instead of changing this
file instead?

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
File chrome/browser/resources/chromeos/speech_synthesis/manifest_guest.json
(right):

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
chrome/browser/resources/chromeos/speech_synthesis/manifest_guest.json:4:
"name": "US English Female Text-to-speech (by Google)",
same question

[Zachary Kuznia, 2014-02-19 01:02:47]

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
File chrome/browser/resources/chromeos/speech_synthesis/manifest.json (right):

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
chrome/browser/resources/chromeos/speech_synthesis/manifest.json:4: "name": "US
English Female Text-to-speech (by Google)",
On 2014/02/19 00:45:50, Nico wrote:
> "US English"? Looks like it has many more languages?

I'll change this to the old name

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
chrome/browser/resources/chromeos/speech_synthesis/manifest.json:19:
"voice_data_hmm_pt-BR_2.js",
On 2014/02/19 00:45:50, Nico wrote:
> I don't see these files checked in, I suppose that's expected?
> 
> Does it make sense to move the foo_2.js files to foo.js instead of changing
this
> file instead?

This has to be different from the old one so that it doesn't conflict with the
old voices on the server, so the files can't be easily renamed.  They are
checked into the chromeos-assets package.

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
File chrome/browser/resources/chromeos/speech_synthesis/manifest_guest.json
(right):

https://codereview.chromium.org/167893003/diff/110001/chrome/browser/resource...
chrome/browser/resources/chromeos/speech_synthesis/manifest_guest.json:4:
"name": "US English Female Text-to-speech (by Google)",
On 2014/02/19 00:45:50, Nico wrote:
> same question

Done.

[Zachary Kuznia, 2014-02-19 01:02:58]

The CQ bit was checked by zork@chromium.org

[commit-bot: I haz the power, 2014-02-19 01:05:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zork@chromium.org/167893003/230001

[commit-bot: I haz the power, 2014-02-19 06:28:49]

Change committed as 251970