Added InitTts function to speech synthesis library

Added InitTts function to speech synthesis library

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=49260

[oshima, 2010-06-02 22:49:53]

can you check compilation error in the try bot?

http://codereview.chromium.org/2510001/diff/1/4
File chrome/browser/chromeos/cros/speech_synthesis_library.h (right):

http://codereview.chromium.org/2510001/diff/1/4#newcode26
chrome/browser/chromeos/cros/speech_synthesis_library.h:26: // Checks if the
engine is ready.
Not sure if this comment is correct. This sounds to me that the method
initialize the TTS with a callback that is used to tell if engine is ready?

[Chaitanya, 2010-06-02 22:57:24]

http://codereview.chromium.org/2510001/diff/1/4
File chrome/browser/chromeos/cros/speech_synthesis_library.h (right):

http://codereview.chromium.org/2510001/diff/1/4#newcode26
chrome/browser/chromeos/cros/speech_synthesis_library.h:26: // Checks if the
engine is ready.
On 2010/06/02 22:49:53, oshima wrote:
> Not sure if this comment is correct. This sounds to me that the method
> initialize the TTS with a callback that is used to tell if engine is ready?

Done.

[Chaitanya, 2010-06-02 23:46:02]

This CL depends on 2471004. So I will not submit this until after 2471004 is in.

On 2010/06/02 22:57:24, Chaitanya wrote:
> http://codereview.chromium.org/2510001/diff/1/4
> File chrome/browser/chromeos/cros/speech_synthesis_library.h (right):
> 
> http://codereview.chromium.org/2510001/diff/1/4#newcode26
> chrome/browser/chromeos/cros/speech_synthesis_library.h:26: // Checks if the
> engine is ready.
> On 2010/06/02 22:49:53, oshima wrote:
> > Not sure if this comment is correct. This sounds to me that the method
> > initialize the TTS with a callback that is used to tell if engine is ready?
> 
> Done.

[oshima, 2010-06-07 18:28:40]

ping me when 2471004 is in.

On 2010/06/02 23:46:02, Chaitanya wrote:
> This CL depends on 2471004. So I will not submit this until after 2471004 is
in.
> 
> On 2010/06/02 22:57:24, Chaitanya wrote:
> > http://codereview.chromium.org/2510001/diff/1/4
> > File chrome/browser/chromeos/cros/speech_synthesis_library.h (right):
> > 
> > http://codereview.chromium.org/2510001/diff/1/4#newcode26
> > chrome/browser/chromeos/cros/speech_synthesis_library.h:26: // Checks if the
> > engine is ready.
> > On 2010/06/02 22:49:53, oshima wrote:
> > > Not sure if this comment is correct. This sounds to me that the method
> > > initialize the TTS with a callback that is used to tell if engine is
ready?
> > 
> > Done.

[oshima, 2010-06-08 02:12:40]

lgtm. make sure linux_chromeos bot does compile.


On 2010/06/07 18:28:40, oshima wrote:
> ping me when 2471004 is in.
> 
> On 2010/06/02 23:46:02, Chaitanya wrote:
> > This CL depends on 2471004. So I will not submit this until after 2471004 is
> in.
> > 
> > On 2010/06/02 22:57:24, Chaitanya wrote:
> > > http://codereview.chromium.org/2510001/diff/1/4
> > > File chrome/browser/chromeos/cros/speech_synthesis_library.h (right):
> > > 
> > > http://codereview.chromium.org/2510001/diff/1/4#newcode26
> > > chrome/browser/chromeos/cros/speech_synthesis_library.h:26: // Checks if
the
> > > engine is ready.
> > > On 2010/06/02 22:49:53, oshima wrote:
> > > > Not sure if this comment is correct. This sounds to me that the method
> > > > initialize the TTS with a callback that is used to tell if engine is
> ready?
> > > 
> > > Done.

[Nikita (slow), 2010-06-09 16:28:13]

Please make sure that trybots are green before committing next time.
This CL broke build.

[Chaitanya, 2010-06-09 17:08:25]

On 2010/06/09 16:28:13, Nikita Kostylev wrote:
> Please make sure that trybots are green before committing next time.
> This CL broke build.

Sorry about that. I thought I was looking at the try bots, but clearly I messed
something up. Will take care henceforth.