Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The crash in the WebRTC audio quality browser tests was caused by one misplaced line in the javascript code.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org

[Ivo-OOO until feb 6, 2016-08-16 16:15:49]

Description was changed from

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The problem was one misplaced line in the javascript code that caused a crash
during some WebRTC perf tests.

BUG=631927
==========

to

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The problem was one misplaced line in the javascript code that caused a crash
during some WebRTC perf tests.

BUG=631927
==========

[Ivo-OOO until feb 6, 2016-08-16 16:15:49]

ivoc@chromium.org changed reviewers:
+ phoglund@chromium.org

[Ivo-OOO until feb 6, 2016-08-16 16:17:02]

Description was changed from

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The problem was one misplaced line in the javascript code that caused a crash
during some WebRTC perf tests.

BUG=631927
==========

to

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The problem was one misplaced line in the javascript code that caused a crash
during some WebRTC perf tests.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org
==========

[Ivo-OOO until feb 6, 2016-08-16 16:17:02]

ivoc@chromium.org changed reviewers:
+ jochen@chromium.org, minyue@chromium.org

[Ivo-OOO until feb 6, 2016-08-16 16:17:54]

Description was changed from

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The problem was one misplaced line in the javascript code that caused a crash
during some WebRTC perf tests.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org
==========

to

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The crash was caused by one misplaced line in the javascript code.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org
==========

[Ivo-OOO until feb 6, 2016-08-16 16:18:13]

Description was changed from

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The crash was caused by one misplaced line in the javascript code.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org
==========

to

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The crash in the WebRTC perf test was caused by one misplaced line in the
javascript code.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org
==========

[Ivo-OOO until feb 6, 2016-08-16 16:18:41]

Description was changed from

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The crash in the WebRTC perf test was caused by one misplaced line in the
javascript code.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org
==========

to

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The crash in the WebRTC audio quality tests was caused by one misplaced line in
the javascript code.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org
==========

[Ivo-OOO until feb 6, 2016-08-16 16:18:54]

Description was changed from

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The crash in the WebRTC audio quality tests was caused by one misplaced line in
the javascript code.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org
==========

to

==========
Reland of "Adds a WebRTC browser_test with opus dtx enabled.".

Original CL: https://codereview.chromium.org/2190533002/
Original commit: e9df939bf67e2dea8e3ca1d9a0857272324611b8
Revert CL: https://codereview.chromium.org/2195293002/
Revert commit: 0dc179b22f88b81c6a083670bb09f22108420a4e

The crash in the WebRTC audio quality browser tests was caused by one misplaced
line in the javascript code.

BUG=631927
TBR=phoglund@chromium.org,jochen@chromium.org,minyue@chromium.org
==========

[Ivo-OOO until feb 6, 2016-08-17 11:16:32]

The CQ bit was checked by ivoc@chromium.org

[commit-bot: I haz the power, 2016-08-17 11:16:55]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-17 11:16:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-17 11:16:57]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Henrik Grunell, 2016-08-17 15:30:07]

The CQ bit was checked by grunell@chromium.org

[commit-bot: I haz the power, 2016-08-17 15:30:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-17 15:30:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-17 15:30:26]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[minyue, 2016-09-20 07:27:29]

On 2016/08/17 15:30:26, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet. 
> CQ run can only be started by full committers or once the patch has
> received an L-G-T-M from a full committer.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

Did this get landed?

[Henrik Grunell, 2016-09-20 07:35:40]

On 2016/09/20 07:27:29, minyue wrote:
> On 2016/08/17 15:30:26, commit-bot: I haz the power wrote:
> > No L-G-T-M from a valid reviewer yet. 
> > CQ run can only be started by full committers or once the patch has
> > received an L-G-T-M from a full committer.
> > Even if an L-G-T-M may have been provided, it was from a non-committer,
> > _not_ a full super star committer.
> > See http://www.chromium.org/getting-involved/become-a-committer
> > Note that this has nothing to do with OWNERS files.
> 
> Did this get landed?

It was rolled by hbos@ in https://codereview.chromium.org/2352673002.

[Henrik Grunell, 2016-09-20 07:36:29]

On 2016/09/20 07:35:40, Henrik Grunell wrote:
> On 2016/09/20 07:27:29, minyue wrote:
> > On 2016/08/17 15:30:26, commit-bot: I haz the power wrote:
> > > No L-G-T-M from a valid reviewer yet. 
> > > CQ run can only be started by full committers or once the patch has
> > > received an L-G-T-M from a full committer.
> > > Even if an L-G-T-M may have been provided, it was from a non-committer,
> > > _not_ a full super star committer.
> > > See http://www.chromium.org/getting-involved/become-a-committer
> > > Note that this has nothing to do with OWNERS files.
> > 
> > Did this get landed?
> 
> It was rolled by hbos@ in https://codereview.chromium.org/2352673002.

Oops, sorry. Wrong CL. Please ignore. :)

[Henrik Grunell, 2016-09-20 07:38:06]

On 2016/09/20 07:36:29, Henrik Grunell wrote:
> On 2016/09/20 07:35:40, Henrik Grunell wrote:
> > On 2016/09/20 07:27:29, minyue wrote:
> > > On 2016/08/17 15:30:26, commit-bot: I haz the power wrote:
> > > > No L-G-T-M from a valid reviewer yet. 
> > > > CQ run can only be started by full committers or once the patch has
> > > > received an L-G-T-M from a full committer.
> > > > Even if an L-G-T-M may have been provided, it was from a non-committer,
> > > > _not_ a full super star committer.
> > > > See http://www.chromium.org/getting-involved/become-a-committer
> > > > Note that this has nothing to do with OWNERS files.
> > > 
> > > Did this get landed?
> > 
> > It was rolled by hbos@ in https://codereview.chromium.org/2352673002.
> 
> Oops, sorry. Wrong CL. Please ignore. :)

However, I did land it here: https://codereview.chromium.org/2254133002/

[minyue, 2016-09-20 07:52:18]

On 2016/09/20 07:38:06, Henrik Grunell wrote:
> On 2016/09/20 07:36:29, Henrik Grunell wrote:
> > On 2016/09/20 07:35:40, Henrik Grunell wrote:
> > > On 2016/09/20 07:27:29, minyue wrote:
> > > > On 2016/08/17 15:30:26, commit-bot: I haz the power wrote:
> > > > > No L-G-T-M from a valid reviewer yet. 
> > > > > CQ run can only be started by full committers or once the patch has
> > > > > received an L-G-T-M from a full committer.
> > > > > Even if an L-G-T-M may have been provided, it was from a
non-committer,
> > > > > _not_ a full super star committer.
> > > > > See http://www.chromium.org/getting-involved/become-a-committer
> > > > > Note that this has nothing to do with OWNERS files.
> > > > 
> > > > Did this get landed?
> > > 
> > > It was rolled by hbos@ in https://codereview.chromium.org/2352673002.
> > 
> > Oops, sorry. Wrong CL. Please ignore. :)
> 
> However, I did land it here: https://codereview.chromium.org/2254133002/

ok. thanks. We can then delete this CL, right?