Remove the video channel id completely.

Remove the video channel id completely.

BUG=webrtc:5079
Committed: https://crrev.com/0dbf0090a961c5e5fb7362937108337564b4a91f
Cr-Commit-Position: refs/heads/master@{#10324}

[mflodman, 2015-10-19 09:48:42]

mflodman@webrtc.org changed reviewers:
+ pbos@webrtc.org

[pbos-webrtc, 2015-10-19 10:04:15]

lgtm, but I don't think you should have to build more strings here.

https://codereview.webrtc.org/1412143002/diff/20001/webrtc/video_engine/vie_e...
File webrtc/video_engine/vie_encoder.cc (right):

https://codereview.webrtc.org/1412143002/diff/20001/webrtc/video_engine/vie_e...
webrtc/video_engine/vie_encoder.cc:690: LOG(LS_INFO) << "Video suspend state
changed " << video_is_suspended
Just log ssrc_streams_.front() (should always be non-empty, right?) instead and
remove the stringstream so we don't have to do any string construction here.

[mflodman, 2015-10-19 11:52:59]

https://codereview.webrtc.org/1412143002/diff/20001/webrtc/video_engine/vie_e...
File webrtc/video_engine/vie_encoder.cc (right):

https://codereview.webrtc.org/1412143002/diff/20001/webrtc/video_engine/vie_e...
webrtc/video_engine/vie_encoder.cc:690: LOG(LS_INFO) << "Video suspend state
changed " << video_is_suspended
On 2015/10/19 10:04:14, pbos-webrtc wrote:
> Just log ssrc_streams_.front() (should always be non-empty, right?) instead
and
> remove the stringstream so we don't have to do any string construction here.

I agree. I had it that way originally, but then changed to log all SSRCs this
way. I'll change, to .begin()->first, and submit.

[pbos-webrtc, 2015-10-19 12:02:39]

On 2015/10/19 11:52:59, mflodman wrote:
>
https://codereview.webrtc.org/1412143002/diff/20001/webrtc/video_engine/vie_e...
> File webrtc/video_engine/vie_encoder.cc (right):
> 
>
https://codereview.webrtc.org/1412143002/diff/20001/webrtc/video_engine/vie_e...
> webrtc/video_engine/vie_encoder.cc:690: LOG(LS_INFO) << "Video suspend state
> changed " << video_is_suspended
> On 2015/10/19 10:04:14, pbos-webrtc wrote:
> > Just log ssrc_streams_.front() (should always be non-empty, right?) instead
> and
> > remove the stringstream so we don't have to do any string construction here.
> 
> I agree. I had it that way originally, but then changed to log all SSRCs this
> way. I'll change, to .begin()->first, and submit.

sgtm

[pbos-webrtc, 2015-10-19 12:02:56]

(lgtm)

[mflodman, 2015-10-19 12:08:54]

The CQ bit was checked by mflodman@webrtc.org

[commit-bot: I haz the power, 2015-10-19 12:09:00]

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

[commit-bot: I haz the power, 2015-10-19 13:30:21]

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

[commit-bot: I haz the power, 2015-10-19 13:30:22]

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

[pbos-webrtc, 2015-10-19 13:34:09]

The CQ bit was checked by pbos@webrtc.org

[commit-bot: I haz the power, 2015-10-19 13:34:18]

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

[commit-bot: I haz the power, 2015-10-19 13:36:03]

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

[commit-bot: I haz the power, 2015-10-19 13:36:03]

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

[mflodman, 2015-10-19 14:00:40]

The CQ bit was checked by mflodman@webrtc.org

[commit-bot: I haz the power, 2015-10-19 14:00:50]

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

[commit-bot: I haz the power, 2015-10-19 15:12:15]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-10-19 15:12:29]

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