Protect new header extension by field trial experiment to allow hardcoding it in SDP

Protect new header extension by field trial experiment to allow hardcoding it in SDP

BUG=chrome:718738
Review-Url: https://codereview.webrtc.org/2922683002
Cr-Commit-Position: refs/heads/master@{#18409}
Committed: https://chromium.googlesource.com/external/webrtc/+/cafa1d6bbef79d9a8507c6e782f3c7824ef1070b

[ilnik, 2017-06-02 10:39:02]

ilnik@webrtc.org changed reviewers:
+ sprang@webrtc.org

[ilnik, 2017-06-02 10:40:56]

The CQ bit was checked by ilnik@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-02 10:41:01]

Dry run: CQ is trying da patch.

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

[danilchap, 2017-06-02 10:42:16]

Description was changed from

==========
Protect new header extension by finch experiment to allow hardcoding it in SDP

BUG=chrome:718738
==========

to

==========
Protect new header extension by finch experiment to allow hardcoding it in SDP

BUG=chromium:718738
==========

[sprang_webrtc, 2017-06-02 10:42:29]

Description was changed from

==========
Protect new header extension by finch experiment to allow hardcoding it in SDP

BUG=chromium:718738
==========

to

==========
Protect new header extension by field trial experiment to allow hardcoding it in
SDP

BUG=chrome:718738
==========

[sprang_webrtc, 2017-06-02 10:42:38]

lgtm

[commit-bot: I haz the power, 2017-06-02 10:48:29]

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

[commit-bot: I haz the power, 2017-06-02 10:48:29]

Dry run: Try jobs failed on following builders:
  mac_asan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_asan/builds/25137)

[ilnik, 2017-06-02 11:09:50]

The CQ bit was checked by ilnik@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-02 11:09:58]

Dry run: CQ is trying da patch.

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

[ilnik, 2017-06-02 11:11:26]

ilnik@webrtc.org changed reviewers:
+ asapersson@webrtc.org

[ilnik, 2017-06-02 11:11:26]

+assapersson,

Asa, your approval is needed for modules/rtp_rtcp.

[commit-bot: I haz the power, 2017-06-02 11:16:59]

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

[commit-bot: I haz the power, 2017-06-02 11:17:00]

Dry run: Try jobs failed on following builders:
  linux_asan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_asan/builds/25216)
  linux_ubsan_vptr on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_ubsan_vptr/builds...)

[ilnik, 2017-06-02 11:23:54]

The CQ bit was checked by ilnik@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-02 11:23:59]

Dry run: CQ is trying da patch.

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

[åsapersson, 2017-06-02 11:48:30]

lgtm

[commit-bot: I haz the power, 2017-06-02 11:58:16]

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

[commit-bot: I haz the power, 2017-06-02 11:58:17]

Dry run: This issue passed the CQ dry run.

[ilnik, 2017-06-02 12:47:19]

The CQ bit was checked by ilnik@webrtc.org

[ilnik, 2017-06-02 12:47:19]

The patchset sent to the CQ was uploaded after l-g-t-m from sprang@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2922683002/#ps40001 (title:
"Fix typos and add missing field trial forcing")

[commit-bot: I haz the power, 2017-06-02 12:47:24]

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-02 12:49:40]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1496407639115140,
"parent_rev": "58c742ce7dbf6143295245f32019deadf03ffd87", "commit_rev":
"cafa1d6bbef79d9a8507c6e782f3c7824ef1070b"}

[commit-bot: I haz the power, 2017-06-02 12:49:43]

Description was changed from

==========
Protect new header extension by field trial experiment to allow hardcoding it in
SDP

BUG=chrome:718738
==========

to

==========
Protect new header extension by field trial experiment to allow hardcoding it in
SDP

BUG=chrome:718738

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

[commit-bot: I haz the power, 2017-06-02 12:49:44]

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

[ilnik, 2017-06-02 14:29:51]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.webrtc.org/2922723002/ by ilnik@webrtc.org.

The reason for reverting is: Breaks tests in downstream projects..

[ilnik, 2017-06-07 15:15:08]

Patchset #4 (id:60001) has been deleted

[ilnik, 2017-06-07 15:20:25]

Patchset #4 (id:80001) has been deleted