Ensure that remoting_unittests fail when playout delay > 0.

Ensure that remoting_unittests fail when playout delay > 0.

Added NOTREACHED() WebrtcVideoRendererAdapter for the case when playout
delay > 0. Also updated ConnectionTest_Video to receive 5 tests. With
this change remoting_unittests will fail if playout delay extension
is not sent by the host or is ignored on the receiving side. This will
help to prevent regressions like crbug.com/webrtc/7590 in the future.
Previously the adapter was only logging a warning in that case.

BUG=722490
Review-Url: https://codereview.chromium.org/2887183004
Cr-Commit-Position: refs/heads/master@{#473323}
Committed: https://chromium.googlesource.com/chromium/src/+/69008afaa78dd9d2fc012b3fcb8f2e0a56fc0a1d

[Sergey Ulanov, 2017-05-18 19:51:00]

Patchset #1 (id:1) has been deleted

[Sergey Ulanov, 2017-05-18 19:51:20]

sergeyu@chromium.org changed reviewers:
+ jamiewalch@chromium.org

[Jamie, 2017-05-18 21:49:57]

lgtm

[Sergey Ulanov, 2017-05-18 21:51:07]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2017-05-18 21:53:10]

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, 2017-05-19 02:14:41]

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

[commit-bot: I haz the power, 2017-05-19 02:14:42]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Sergey Ulanov, 2017-05-19 20:48:42]

The CQ bit was checked by sergeyu@chromium.org

[Sergey Ulanov, 2017-05-19 20:48:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
jamiewalch@chromium.org
Link to the patchset: https://codereview.chromium.org/2887183004/#ps40001
(title: "fix failure on windows")

[commit-bot: I haz the power, 2017-05-19 20:49:16]

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, 2017-05-19 21:21:39]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495226922304890,
"parent_rev": "909c175cf1510665dd94e26ab23a08a83fe29fce", "commit_rev":
"69008afaa78dd9d2fc012b3fcb8f2e0a56fc0a1d"}

[commit-bot: I haz the power, 2017-05-19 21:21:52]

Description was changed from

==========
Ensure that remoting_unittests fail when playout delay > 0.

Added NOTREACHED() WebrtcVideoRendererAdapter for the case when playout
delay > 0. Also updated ConnectionTest_Video to receive 5 tests. With
this change remoting_unittests will fail if playout delay extension
is not sent by the host or is ignored on the receiving side. This will
help to prevent regressions like crbug.com/webrtc/7590 in the future.
Previously the adapter was only logging a warning in that case.

BUG=722490
==========

to

==========
Ensure that remoting_unittests fail when playout delay > 0.

Added NOTREACHED() WebrtcVideoRendererAdapter for the case when playout
delay > 0. Also updated ConnectionTest_Video to receive 5 tests. With
this change remoting_unittests will fail if playout delay extension
is not sent by the host or is ignored on the receiving side. This will
help to prevent regressions like crbug.com/webrtc/7590 in the future.
Previously the adapter was only logging a warning in that case.

BUG=722490

Review-Url: https://codereview.chromium.org/2887183004
Cr-Commit-Position: refs/heads/master@{#473323}
Committed:
https://chromium.googlesource.com/chromium/src/+/69008afaa78dd9d2fc012b3fcb8f...
==========

[commit-bot: I haz the power, 2017-05-19 21:21:53]

Committed patchset #2 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/69008afaa78dd9d2fc012b3fcb8f...