Remove the dependency of Extension API from tts_chromeos

Remove the dependency of Extension API from tts_chromeos

Moving the extension api dependency from TtsPlatformImplChromeOs
to TtsExtensionEngine through TtsEngineDelegate.
This is part of an effort to move TTS to content.

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

[mrunal, 2014-08-01 15:37:34]

The above failures doesn't seem to be related to my patch. If you think they are
related to my patch let me know and I will fix it.

[dmazzoni, 2014-08-01 17:14:33]

lgtm

I agree, the failures look unrelated.

https://codereview.chromium.org/434083003/diff/20001/chrome/browser/speech/tt...
File chrome/browser/speech/tts_controller.h (right):

https://codereview.chromium.org/434083003/diff/20001/chrome/browser/speech/tt...
chrome/browser/speech/tts_controller.h:102: // Load the build-in component
extension for ChromeOS
nit: build-in -> built-in
nit: end comment with a period (.)

[commit-bot: I haz the power, 2014-08-01 17:15:26]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mrunal.kapade@intel.com/434083003/20001

[mrunal, 2014-08-01 18:35:31]

https://codereview.chromium.org/434083003/diff/20001/chrome/browser/speech/tt...
File chrome/browser/speech/tts_controller.h (right):

https://codereview.chromium.org/434083003/diff/20001/chrome/browser/speech/tt...
chrome/browser/speech/tts_controller.h:102: // Load the build-in component
extension for ChromeOS
On 2014/08/01 17:14:33, dmazzoni wrote:
> nit: build-in -> built-in
> nit: end comment with a period (.)

Acknowledged. Can I upload a patch to the same issue or do u want me to upload
another patch(issue)?

[dmazzoni, 2014-08-01 21:38:15]

On Fri, Aug 1, 2014 at 11:35 AM, <mrunal.kapade@intel.com> wrote:

> Acknowledged. Can I upload a patch to the same issue or do u want me to
> upload another patch(issue)?
>

Since that was the only issue I clicked the commit box. Feel free to upload
another patch and then click the commit queue box again.

Also, I'm not sure if you're already working on the next patch, but feel
free to upload it before this one lands - I'm always happy to review early
even though we can't easily run try jobs until the first one lands.

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[mrunal, 2014-08-01 22:07:01]

On 2014/08/01 21:38:15, dmazzoni wrote:
> On Fri, Aug 1, 2014 at 11:35 AM, <mailto:mrunal.kapade@intel.com> wrote:
> 
> > Acknowledged. Can I upload a patch to the same issue or do u want me to
> > upload another patch(issue)?
> >
> 
> Since that was the only issue I clicked the commit box. Feel free to upload
> another patch and then click the commit queue box again.
Ok, I will upload the 3rd patch to this review.
> 
> Also, I'm not sure if you're already working on the next patch, but feel
> free to upload it before this one lands - I'm always happy to review early
> even though we can't easily run try jobs until the first one lands.
I have the big patch but I'm uploading it in separate independent patches so
that it will not only be easier to review but also easy to limit the failures to
manageable extent. Since I cannot compile for non-linux platforms I don't want
you to review the code which cannot even compile. :)

[mrunal, 2014-08-01 22:11:15]

The CQ bit was checked by mrunal.kapade@intel.com

[commit-bot: I haz the power, 2014-08-01 22:13:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mrunal.kapade@intel.com/434083003/40001

[commit-bot: I haz the power, 2014-08-02 05:56:03]

Change committed as 287160