Second try to get "Support for video file instead of camera and output video out to file" accepted

Second try to get "Support for video file instead of camera and output video out to file" accepted

The old CL can be found here: https://codereview.webrtc.org/2273573003/

The orginal broke down stream, this CL tries to solve those issues.

BUG=webrtc:6545
Committed: https://crrev.com/64e1a32e2f3fa790927e71c783520c2d1ed2ead1
Cr-Commit-Position: refs/heads/master@{#14665}

[mandermo, 2016-10-18 13:59:13]

mandermo@webrtc.org changed reviewers:
+ kjellander@webrtc.org, phoglund@webrtc.org

[mandermo, 2016-10-18 13:59:14]

Fix for the broken downstream.

The first patch is a revert of the revert. The second patch removes the
offending logging line.

[mandermo, 2016-10-18 14:06:59]

Description was changed from

==========
Second try get "Support for video file instead of camera and output video out to
file" accepted

The orginal broke down stream, this CL tries to solve those issues.
==========

to

==========
Second try to get "Support for video file instead of camera and output video out
to file" accepted

The orginal broke down stream, this CL tries to solve those issues.
==========

[kjellander_webrtc, 2016-10-18 14:07:03]

lgtm assuming you've verified there are no more similar errors.
Please refer to bug and the old CL in the description

[mandermo, 2016-10-18 14:10:17]

Description was changed from

==========
Second try to get "Support for video file instead of camera and output video out
to file" accepted

The orginal broke down stream, this CL tries to solve those issues.
==========

to

==========
Second try to get "Support for video file instead of camera and output video out
to file" accepted

The old CL can be found here: https://codereview.webrtc.org/2273573003/

The orginal broke down stream, this CL tries to solve those issues.

BUG=webrtc:6545
==========

[phoglund, 2016-10-18 14:18:02]

fix lgtm

you need a rubberstamp from an appropriate owner, perhaps from the old review if
one reviewer has enough privileges to approve everything.

[mandermo, 2016-10-18 14:22:52]

mandermo@webrtc.org changed reviewers:
+ sakal@webrtc.org

[mandermo, 2016-10-18 14:22:52]

Adding one more reviewer.

[sakal, 2016-10-18 14:25:00]

I am trusting you didn't change anything else than remove the breaking line.

lgtm

[mandermo, 2016-10-18 14:52:08]

The CQ bit was checked by mandermo@webrtc.org

[commit-bot: I haz the power, 2016-10-18 14:52:13]

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

[commit-bot: I haz the power, 2016-10-18 15:15:59]

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

[commit-bot: I haz the power, 2016-10-18 15:16:00]

Try jobs failed on following builders:
  presubmit on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/9274)

[mandermo, 2016-10-18 15:33:46]

mandermo@webrtc.org changed reviewers:
+ magjed@webrtc.org

[mandermo, 2016-10-18 15:33:46]

+magjed because he is needed, because he is owner of files.

[magjed_webrtc, 2016-10-18 15:38:19]

lgtm. In the future, revert the revert instead of manually making a new CL. The
CL description will then be filled in for you with appropriate links and
comments.

[mandermo, 2016-10-18 15:39:26]

The CQ bit was checked by mandermo@webrtc.org

[commit-bot: I haz the power, 2016-10-18 15:39:35]

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

[commit-bot: I haz the power, 2016-10-18 15:47:53]

Description was changed from

==========
Second try to get "Support for video file instead of camera and output video out
to file" accepted

The old CL can be found here: https://codereview.webrtc.org/2273573003/

The orginal broke down stream, this CL tries to solve those issues.

BUG=webrtc:6545
==========

to

==========
Second try to get "Support for video file instead of camera and output video out
to file" accepted

The old CL can be found here: https://codereview.webrtc.org/2273573003/

The orginal broke down stream, this CL tries to solve those issues.

BUG=webrtc:6545
==========

[commit-bot: I haz the power, 2016-10-18 15:47:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-18 15:48:05]

Description was changed from

==========
Second try to get "Support for video file instead of camera and output video out
to file" accepted

The old CL can be found here: https://codereview.webrtc.org/2273573003/

The orginal broke down stream, this CL tries to solve those issues.

BUG=webrtc:6545
==========

to

==========
Second try to get "Support for video file instead of camera and output video out
to file" accepted

The old CL can be found here: https://codereview.webrtc.org/2273573003/

The orginal broke down stream, this CL tries to solve those issues.

BUG=webrtc:6545

Committed: https://crrev.com/64e1a32e2f3fa790927e71c783520c2d1ed2ead1
Cr-Commit-Position: refs/heads/master@{#14665}
==========

[commit-bot: I haz the power, 2016-10-18 15:48:06]

Patchset 2 (id:??) landed as
https://crrev.com/64e1a32e2f3fa790927e71c783520c2d1ed2ead1
Cr-Commit-Position: refs/heads/master@{#14665}