AudioConfMixer: Add muted variable to ParticipantFramePair

AudioConfMixer: Add muted variable to ParticipantFramePair

Renaming the ParticipantFramePair to ParticipantFrameStruct. The muted
variable is not yet used.

BUG=webrtc:5609
Committed: https://crrev.com/3a0a0f4b0dae11b20ebacdbd79d6c7f94edeac7b
Cr-Commit-Position: refs/heads/master@{#12780}

[hlundin-webrtc, 2016-05-17 10:25:02]

henrik.lundin@webrtc.org changed reviewers:
+ minyue@webrtc.org

[hlundin-webrtc, 2016-05-17 10:25:02]

Minyue, PTAL at this change.

[minyue-webrtc, 2016-05-17 15:27:30]

LGTM + one question (no action needed)

https://codereview.webrtc.org/1981243002/diff/1/webrtc/modules/audio_conferen...
File webrtc/modules/audio_conference_mixer/source/audio_conference_mixer_impl.cc
(right):

https://codereview.webrtc.org/1981243002/diff/1/webrtc/modules/audio_conferen...
webrtc/modules/audio_conference_mixer/source/audio_conference_mixer_impl.cc:26:
AudioFrame* audioFrame;
I know it is late to ask this, but would it be possible to add muted in
AudioFrame

[hlundin-webrtc, 2016-05-17 18:09:08]

https://codereview.webrtc.org/1981243002/diff/1/webrtc/modules/audio_conferen...
File webrtc/modules/audio_conference_mixer/source/audio_conference_mixer_impl.cc
(right):

https://codereview.webrtc.org/1981243002/diff/1/webrtc/modules/audio_conferen...
webrtc/modules/audio_conference_mixer/source/audio_conference_mixer_impl.cc:26:
AudioFrame* audioFrame;
On 2016/05/17 15:27:30, minyue-webrtc wrote:
> I know it is late to ask this, but would it be possible to add muted in
> AudioFrame

We did consider that. I had discussions with solenberg@, and we concluded that
we did not want to pollute the AudioFrame with anything more; it is already
quite messy. In hindsight, after having integrated the feature into the
conference mixer, I might have changed my position.

[hlundin-webrtc, 2016-05-17 19:24:38]

The CQ bit was checked by henrik.lundin@webrtc.org

[commit-bot: I haz the power, 2016-05-17 19:24:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1981243002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1981243002/1

[commit-bot: I haz the power, 2016-05-17 21:25:29]

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

[commit-bot: I haz the power, 2016-05-17 21:25:31]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[hlundin-webrtc, 2016-05-18 00:14:20]

The CQ bit was checked by henrik.lundin@webrtc.org

[commit-bot: I haz the power, 2016-05-18 00:14:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1981243002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1981243002/1

[commit-bot: I haz the power, 2016-05-18 00:16:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-18 00:16:11]

Description was changed from

==========
AudioConfMixer: Add muted variable to ParticipantFramePair

Renaming the ParticipantFramePair to ParticipantFrameStruct. The muted
variable is not yet used.

BUG=webrtc:5609
==========

to

==========
AudioConfMixer: Add muted variable to ParticipantFramePair

Renaming the ParticipantFramePair to ParticipantFrameStruct. The muted
variable is not yet used.

BUG=webrtc:5609

Committed: https://crrev.com/3a0a0f4b0dae11b20ebacdbd79d6c7f94edeac7b
Cr-Commit-Position: refs/heads/master@{#12780}
==========

[commit-bot: I haz the power, 2016-05-18 00:16:13]

Patchset 1 (id:??) landed as
https://crrev.com/3a0a0f4b0dae11b20ebacdbd79d6c7f94edeac7b
Cr-Commit-Position: refs/heads/master@{#12780}