Fix typo in AudioStreamIndicator causing hangs

Fix typo in AudioStreamIndicator from r182364 causing hangs.

NOTRY=true
TBR=sky,bauerb

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

[commit-bot: I haz the power, 2013-02-14 06:09:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/justinlin@chromium.org/12260035/1

[commit-bot: I haz the power, 2013-02-14 06:11:30]

Commit queue rejected this change because the description was changed
between the time the change entered the commit queue and the time it
was ready to commit. You can safely check the commit box again.

[commit-bot: I haz the power, 2013-02-14 06:14:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/justinlin@chromium.org/12260035/1

[commit-bot: I haz the power, 2013-02-14 06:17:25]

Change committed as 182401

[Bernhard Bauer, 2013-02-14 07:09:44]

LGTM, thanks!

[miu, 2013-02-14 20:15:10]

On 2013/02/14 07:09:44, Bernhard Bauer wrote:
> LGTM, thanks!

So, will this need to be merged into M26 branch?

[Bernhard Bauer, 2013-02-15 10:09:42]

On 2013/02/14 20:15:10, Yuri wrote:
> On 2013/02/14 07:09:44, Bernhard Bauer wrote:
> > LGTM, thanks!
> 
> So, will this need to be merged into M26 branch?

Probably... how often do the hangs occur? Do we have a bug?

[Bernhard Bauer, 2013-02-15 14:48:27]

On 2013/02/14 20:15:10, Yuri wrote:
> On 2013/02/14 07:09:44, Bernhard Bauer wrote:
> > LGTM, thanks!
> 
> So, will this need to be merged into M26 branch?

Hm, M26 was branched at r181864, so I think the offending CL did not make it in.

[justinlin, 2013-02-15 17:55:01]

It actually was merged into something:
https://codereview.chromium.org/12207180/

I thought that was M26, but seems seems it's just a release branch:
https://src.chromium.org/viewvc/chrome/branches/1412/src/chrome/browser/media...

[Bernhard Bauer, 2013-02-18 10:48:35]

On 2013/02/15 17:55:01, justinlin wrote:
> It actually was merged into something:
> https://codereview.chromium.org/12207180/
> 
> I thought that was M26, but seems seems it's just a release branch:
>
https://src.chromium.org/viewvc/chrome/branches/1412/src/chrome/browser/media...

Yes, I think 1412 was used for a canary version.

[anagy4698, 2013-02-18 14:54:24]

Yes, I think 1412 was used for a canary version.:::(issue 12260035" "


2013/2/18 <bauerb@chromium.org>

> On 2013/02/15 17:55:01, justinlin wrote:
>
>> It actually was merged into something:
>>
https://codereview.chromium.**org/12207180/<https://codereview.chromium.org/1...
>>
>
>  I thought that was M26, but seems seems it's just a release branch:
>>
>
> https://src.chromium.org/**viewvc/chrome/branches/1412/**
> src/chrome/browser/media/**audio_stream_indicator.cc?**
>
view=log&pathrev=182502<https://src.chromium.org/viewvc/chrome/branches/1412/src/chrome/browser/media/audio_stream_indicator.cc?view=log&pathrev=182502>
>
> Yes, I think 1412 was used for a canary version.
>
>
https://chromiumcodereview.**appspot.com/12260035/<https://chromiumcodereview...
>



-- 
anagy4698@gmail.com.