Make the state transition for a PortAllocatorSession in each derived class

Make the state transition for a PortAllocatorSession in each derived class instead of in the base class.

Putting them in the base class may potentially break subclasses if they
have not called the same method in the base class.

R=deadbeef@webrtc.org, pthatcher@webrtc.org

Committed: https://crrev.com/d8f6fc46563995b5e7d1788e1c862fc4699dd1e3
Cr-Commit-Position: refs/heads/master@{#13372}

[honghaiz3, 2016-07-02 00:00:22]

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

[honghaiz3, 2016-07-02 00:00:30]

Patchset #1 (id:1) has been deleted

[Taylor Brandstetter, 2016-07-02 00:01:59]

lgtm

[pthatcher1, 2016-07-02 00:03:18]

lgtm

[honghaiz3, 2016-07-02 00:04:07]

The CQ bit was checked by honghaiz@webrtc.org

[commit-bot: I haz the power, 2016-07-02 00:04:12]

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

[Taylor Brandstetter, 2016-07-02 00:05:09]

Description was changed from

==========
Make the state transition for a PortAllocatorSession in each derived class
instead of in the base class.
Putting them in the base class may potentially break subclasses if they
have not called the same method in the base class.

BUG=
==========

to

==========
Make the state transition for a PortAllocatorSession in each derived class
instead of in the base class.

Putting them in the base class may potentially break subclasses if they
have not called the same method in the base class.
==========

[honghaiz3, 2016-07-02 00:07:02]

The CQ bit was unchecked by honghaiz@webrtc.org

[honghaiz3, 2016-07-02 00:08:27]

The CQ bit was checked by honghaiz@webrtc.org

[honghaiz3, 2016-07-02 00:08:27]

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

[commit-bot: I haz the power, 2016-07-02 00:08:37]

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

[honghaiz3, 2016-07-02 00:31:29]

Description was changed from

==========
Make the state transition for a PortAllocatorSession in each derived class
instead of in the base class.

Putting them in the base class may potentially break subclasses if they
have not called the same method in the base class.
==========

to

==========
Make the state transition for a PortAllocatorSession in each derived class
instead of in the base class.

Putting them in the base class may potentially break subclasses if they
have not called the same method in the base class.

R=deadbeef@webrtc.org, pthatcher@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/d8f6fc46563995b5e7d1788e1...
==========

[honghaiz3, 2016-07-02 00:31:30]

Committed patchset #2 (id:40001) manually as
d8f6fc46563995b5e7d1788e1c862fc4699dd1e3 (presubmit successful).

[commit-bot: I haz the power, 2016-07-02 00:31:32]

Description was changed from

==========
Make the state transition for a PortAllocatorSession in each derived class
instead of in the base class.

Putting them in the base class may potentially break subclasses if they
have not called the same method in the base class.

R=deadbeef@webrtc.org, pthatcher@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/d8f6fc46563995b5e7d1788e1...
==========

to

==========
Make the state transition for a PortAllocatorSession in each derived class
instead of in the base class.

Putting them in the base class may potentially break subclasses if they
have not called the same method in the base class.

R=deadbeef@webrtc.org, pthatcher@webrtc.org

Committed: https://crrev.com/d8f6fc46563995b5e7d1788e1c862fc4699dd1e3
Cr-Commit-Position: refs/heads/master@{#13372}
==========