s/WebRtcVideoChannel2/WebRtcVideoChannel and s/WebRtcVideoEngine2/WebRtcVideoEngine

s/WebRtcVideoChannel2/WebRtcVideoChannel and s/WebRtcVideoEngine2/WebRtcVideoEngine

WebRtcVideoChannel and and WebRtcVideoEngine seem to have been removed, and only WebRtcVideoChannel2 and WebRtcVideoEngine2 remain, which removes the need for the "2" postfix.

BUG=None

Review-Url: https://codereview.webrtc.org/2932073002
Cr-Commit-Position: refs/heads/master@{#18531}
Committed: https://chromium.googlesource.com/external/webrtc/+/f184138a5f4738be432762a4c621ad3f2b3f665e

[eladalon, 2017-06-09 11:09:55]

Description was changed from

==========
s/WebRtcVideoChannel2/WebRtcVideoChannel and
s/WebRtcVideoEngine2/WebRtcVideoEngine

WebRtcVideoChannel and and WebRtcVideoEngine seem to have been removed, and only
WebRtcVideoChannel2 and WebRtcVideoEngine2 remain, which removes the need for
the "2" postfix.

BUG=None
==========

to

==========
s/WebRtcVideoChannel2/WebRtcVideoChannel and
s/WebRtcVideoEngine2/WebRtcVideoEngine

WebRtcVideoChannel and and WebRtcVideoEngine seem to have been removed, and only
WebRtcVideoChannel2 and WebRtcVideoEngine2 remain, which removes the need for
the "2" postfix.

BUG=None
==========

[eladalon, 2017-06-09 11:09:55]

eladalon@webrtc.org changed reviewers:
+ stefan@webrtc.org

[eladalon, 2017-06-09 11:10:02]

PTAL

[stefan-webrtc, 2017-06-09 11:17:17]

https://codereview.webrtc.org/2932073002/diff/1/webrtc/media/engine/webrtcvid...
File webrtc/media/engine/webrtcvideoengine.cc (right):

https://codereview.webrtc.org/2932073002/diff/1/webrtc/media/engine/webrtcvid...
webrtc/media/engine/webrtcvideoengine.cc:465: // 3
Remove?

https://codereview.webrtc.org/2932073002/diff/1/webrtc/media/engine/webrtcvid...
File webrtc/media/engine/webrtcvideoengine2.h (right):

https://codereview.webrtc.org/2932073002/diff/1/webrtc/media/engine/webrtcvid...
webrtc/media/engine/webrtcvideoengine2.h:15: #include
<webrtc/media/engine/webrtcvideoengine.h>
#include "webrtc/media/engine/webrtcvideoengine.h"?

[eladalon, 2017-06-09 11:48:37]

https://codereview.webrtc.org/2932073002/diff/1/webrtc/media/engine/webrtcvid...
File webrtc/media/engine/webrtcvideoengine.cc (right):

https://codereview.webrtc.org/2932073002/diff/1/webrtc/media/engine/webrtcvid...
webrtc/media/engine/webrtcvideoengine.cc:465: // 3
On 2017/06/09 11:17:17, stefan-webrtc wrote:
> Remove?

Oops! Thanks for catching!

https://codereview.webrtc.org/2932073002/diff/1/webrtc/media/engine/webrtcvid...
File webrtc/media/engine/webrtcvideoengine2.h (right):

https://codereview.webrtc.org/2932073002/diff/1/webrtc/media/engine/webrtcvid...
webrtc/media/engine/webrtcvideoengine2.h:15: #include
<webrtc/media/engine/webrtcvideoengine.h>
On 2017/06/09 11:17:17, stefan-webrtc wrote:
> #include "webrtc/media/engine/webrtcvideoengine.h"?

Done.

[stefan-webrtc, 2017-06-09 14:45:48]

lgtm

[eladalon, 2017-06-09 17:37:27]

eladalon@webrtc.org changed reviewers:
+ deadbeef@webrtc.org, solenberg@webrtc.org

[eladalon, 2017-06-09 17:37:28]

Adding additional owners for the other directories - PTAL.

[eladalon, 2017-06-09 17:38:05]

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

[pbos-webrtc, 2017-06-09 17:50:48]

lgtm

[pbos-webrtc, 2017-06-09 17:51:06]

On 2017/06/09 17:50:48, pbos-webrtc wrote:
> lgtm

(just be careful + make sure the sheriff is aware when landing)

[pbos-webrtc, 2017-06-09 17:51:24]

pbos@webrtc.org changed reviewers:
+ noahric@chromium.org

[pbos-webrtc, 2017-06-09 17:51:25]

+noahric@ fyi

[Taylor Brandstetter, 2017-06-09 18:50:57]

lgtm

[eladalon, 2017-06-12 07:42:22]

The CQ bit was checked by eladalon@webrtc.org

[commit-bot: I haz the power, 2017-06-12 07:42:28]

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, 2017-06-12 08:16:47]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1497253342947650,
"parent_rev": "a8e781aedf16102e1b6f68a1b2bea371cd2aed2d", "commit_rev":
"f184138a5f4738be432762a4c621ad3f2b3f665e"}

[commit-bot: I haz the power, 2017-06-12 08:16:51]

Description was changed from

==========
s/WebRtcVideoChannel2/WebRtcVideoChannel and
s/WebRtcVideoEngine2/WebRtcVideoEngine

WebRtcVideoChannel and and WebRtcVideoEngine seem to have been removed, and only
WebRtcVideoChannel2 and WebRtcVideoEngine2 remain, which removes the need for
the "2" postfix.

BUG=None
==========

to

==========
s/WebRtcVideoChannel2/WebRtcVideoChannel and
s/WebRtcVideoEngine2/WebRtcVideoEngine

WebRtcVideoChannel and and WebRtcVideoEngine seem to have been removed, and only
WebRtcVideoChannel2 and WebRtcVideoEngine2 remain, which removes the need for
the "2" postfix.

BUG=None

Review-Url: https://codereview.webrtc.org/2932073002
Cr-Commit-Position: refs/heads/master@{#18531}
Committed:
https://chromium.googlesource.com/external/webrtc/+/f184138a5f4738be432762a4c...
==========

[commit-bot: I haz the power, 2017-06-12 08:16:57]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/f184138a5f4738be432762a4c...