Don't allow changing ICE pool size after SetLocalDescription.

Don't allow changing ICE pool size after SetLocalDescription.

This was the decision at IETF 97
(see: https://github.com/rtcweb-wg/jsep/issues/381). It's simpler to not
allow this (since there's no real need for it) rather than try to decide
complex rules for it.

BUG=webrtc:6864
Committed: https://crrev.com/6de92f9255b8746ad1be920fa508c19cb71dea63
Cr-Commit-Position: refs/heads/master@{#15559}

[Taylor Brandstetter, 2016-12-10 09:33:57]

deadbeef@webrtc.org changed reviewers:
+ pthatcher@webrtc.org

[Taylor Brandstetter, 2016-12-10 09:33:58]

PTAL Peter. This was the last major thing holding me back from making ICE
candidate pooling a feature available in chromium (the spec wasn't clear about
what should happen here; now that it's not allowed at all that problem is
solved).

[pthatcher1, 2016-12-12 23:40:12]

lgtm

[Taylor Brandstetter, 2016-12-13 02:03:03]

The CQ bit was checked by deadbeef@webrtc.org

[Taylor Brandstetter, 2016-12-13 02:03:04]

The patchset sent to the CQ was uploaded after l-g-t-m from pthatcher@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2566833002/#ps40001 (title:
".")

[commit-bot: I haz the power, 2016-12-13 02:03:11]

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-12-13 02:16:41]

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

[commit-bot: I haz the power, 2016-12-13 02:16:42]

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

[Taylor Brandstetter, 2016-12-13 02:20:49]

The CQ bit was checked by deadbeef@webrtc.org

[Taylor Brandstetter, 2016-12-13 02:20:49]

The patchset sent to the CQ was uploaded after l-g-t-m from pthatcher@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2566833002/#ps60001 (title:
"Fixing signed/unsigned compare warning.")

[commit-bot: I haz the power, 2016-12-13 02:20:51]

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

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1481595649074320,
"parent_rev": "50024f6a270210f9cf4a8cb4b05fb998b52806a1", "commit_rev":
"64d949d6091ecdb1b876ef06d5a71a0eae006709"}

[commit-bot: I haz the power, 2016-12-13 02:49:36]

Description was changed from

==========
Don't allow changing ICE pool size after SetLocalDescription.

This was the decision at IETF 97
(see: https://github.com/rtcweb-wg/jsep/issues/381). It's simpler to not
allow this (since there's no real need for it) rather than try to decide
complex rules for it.

BUG=webrtc:6864
==========

to

==========
Don't allow changing ICE pool size after SetLocalDescription.

This was the decision at IETF 97
(see: https://github.com/rtcweb-wg/jsep/issues/381). It's simpler to not
allow this (since there's no real need for it) rather than try to decide
complex rules for it.

BUG=webrtc:6864

Review-Url: https://codereview.webrtc.org/2566833002
==========

[commit-bot: I haz the power, 2016-12-13 02:49:37]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-12-13 02:49:42]

Description was changed from

==========
Don't allow changing ICE pool size after SetLocalDescription.

This was the decision at IETF 97
(see: https://github.com/rtcweb-wg/jsep/issues/381). It's simpler to not
allow this (since there's no real need for it) rather than try to decide
complex rules for it.

BUG=webrtc:6864

Review-Url: https://codereview.webrtc.org/2566833002
==========

to

==========
Don't allow changing ICE pool size after SetLocalDescription.

This was the decision at IETF 97
(see: https://github.com/rtcweb-wg/jsep/issues/381). It's simpler to not
allow this (since there's no real need for it) rather than try to decide
complex rules for it.

BUG=webrtc:6864

Committed: https://crrev.com/6de92f9255b8746ad1be920fa508c19cb71dea63
Cr-Commit-Position: refs/heads/master@{#15559}
==========

[commit-bot: I haz the power, 2016-12-13 02:49:43]

Patchset 4 (id:??) landed as
https://crrev.com/6de92f9255b8746ad1be920fa508c19cb71dea63
Cr-Commit-Position: refs/heads/master@{#15559}