Add an abstract class for IceTransport

Add an abstract class for IceTransport

P2PTransportChannl will be renamed to IceTransport and this class will be
the base class of IceTransport. By doing this, the Dtls related methods 
can be separated from the IceTransport.

For more detail, https://docs.google.com/document/d/1g9RA0s4RV7hFAcWiAM2b6H5ZohAVpBqcYienEDj6IcY/edit

BUG=none

Review-Url: https://codereview.webrtc.org/2577183004
Cr-Commit-Position: refs/heads/master@{#15690}
Committed: https://chromium.googlesource.com/external/webrtc/+/86abd6f2fdff54c53af7d96e8893bce32eaa71b8

[zhihuang1, 2016-12-16 02:46:22]

Patchset #1 (id:1) has been deleted

[zhihuang1, 2016-12-16 02:56:12]

Patchset #1 (id:20001) has been deleted

[zhihuang1, 2016-12-16 03:03:35]

Patchset #1 (id:40001) has been deleted

[zhihuang1, 2016-12-16 03:03:42]

Patchset #1 (id:60001) has been deleted

[zhihuang1, 2016-12-16 03:03:47]

Patchset #1 (id:80001) has been deleted

[Taylor Brandstetter, 2016-12-19 16:36:06]

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

[Taylor Brandstetter, 2016-12-19 16:36:06]

lgtm

https://codereview.webrtc.org/2577183004/diff/120001/webrtc/p2p/base/icetrans...
File webrtc/p2p/base/icetransportinternal.h (right):

https://codereview.webrtc.org/2577183004/diff/120001/webrtc/p2p/base/icetrans...
webrtc/p2p/base/icetransportinternal.h:43: // split from this class.
Could add a link to "https://www.w3.org/TR/webrtc/#rtcicetransport-interface" to
tell what's meant by "public interface"

[Zhi Huang, 2016-12-19 18:41:03]

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

[commit-bot: I haz the power, 2016-12-19 18:41:06]

Dry run: 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-19 19:35:20]

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

[commit-bot: I haz the power, 2016-12-19 19:35:20]

Dry run: This issue passed the CQ dry run.

[Zhi Huang, 2016-12-19 19:38:46]

The CQ bit was checked by zhihuang@webrtc.org

[Zhi Huang, 2016-12-19 19:38:47]

The patchset sent to the CQ was uploaded after l-g-t-m from deadbeef@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2577183004/#ps140001 (title:
"Make the comments more clear by adding a link.")

[commit-bot: I haz the power, 2016-12-19 19:38:50]

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

[Zhi Huang, 2016-12-19 19:39:22]

The CQ bit was unchecked by zhihuang@webrtc.org

[Zhi Huang, 2016-12-19 19:45:33]

The CQ bit was checked by zhihuang@webrtc.org

[Zhi Huang, 2016-12-19 19:45:39]

The CQ bit was unchecked by zhihuang@webrtc.org

[Zhi Huang, 2016-12-19 19:45:39]

The CQ bit was checked by zhihuang@webrtc.org

[Zhi Huang, 2016-12-19 19:45:40]

The CQ bit was unchecked by zhihuang@webrtc.org

[Zhi Huang, 2016-12-19 19:49:11]

Description was changed from

==========
Add an abstract class for IceTransport

BUG=none
==========

to

==========
Add an abstract class for IceTransport

P2PTransportChannl will be renamed to IceTransport and this class will be
the base class of IceTransport. By doing this, the Dtls related methods 
can be separated from the IceTransport.

For more detail,
https://docs.google.com/document/d/1g9RA0s4RV7hFAcWiAM2b6H5ZohAVpBqcYienEDj6I...

BUG=none
==========

[Zhi Huang, 2016-12-19 19:49:46]

The CQ bit was checked by zhihuang@webrtc.org

[commit-bot: I haz the power, 2016-12-19 19:49:50]

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-19 19:54:06]

CQ is committing da patch.

Bot data: {"patchset_id": 140001, "attempt_start_ts": 1482176986909950,
"parent_rev": "627188dc49d22ff4902e31fd3d12e8c5212174ee", "commit_rev":
"86abd6f2fdff54c53af7d96e8893bce32eaa71b8"}

[commit-bot: I haz the power, 2016-12-19 19:54:08]

Description was changed from

==========
Add an abstract class for IceTransport

P2PTransportChannl will be renamed to IceTransport and this class will be
the base class of IceTransport. By doing this, the Dtls related methods 
can be separated from the IceTransport.

For more detail,
https://docs.google.com/document/d/1g9RA0s4RV7hFAcWiAM2b6H5ZohAVpBqcYienEDj6I...

BUG=none
==========

to

==========
Add an abstract class for IceTransport

P2PTransportChannl will be renamed to IceTransport and this class will be
the base class of IceTransport. By doing this, the Dtls related methods 
can be separated from the IceTransport.

For more detail,
https://docs.google.com/document/d/1g9RA0s4RV7hFAcWiAM2b6H5ZohAVpBqcYienEDj6I...

BUG=none

Review-Url: https://codereview.webrtc.org/2577183004
Cr-Commit-Position: refs/heads/master@{#15690}
Committed:
https://chromium.googlesource.com/external/webrtc/+/86abd6f2fdff54c53af7d96e8...
==========

[commit-bot: I haz the power, 2016-12-19 19:54:09]

Committed patchset #3 (id:140001) as
https://chromium.googlesource.com/external/webrtc/+/86abd6f2fdff54c53af7d96e8...