Rework transition to closed to match other transitions.

Fixes #178
On 10/21/14 5:33 PM, Nils Ohlmeier wrote:
>
> In media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp:
>
> > @@ -2267,12 +2262,12 @@ void
> >  PeerConnectionImpl::SetSignalingState_m(PCImplSignalingState aSignalingState)
> >  {
> >    PC_AUTO_ENTER_API_CALL_NO_CHECK();
> > -  if (mSignalingState == aSignalingState ||
> > -      mSignalingState == PCImplSignalingState::SignalingClosed) {
> > +  if (mSignalingState == aSignalingState) {
>
> Doesn't this now allow the code to change the signaling state after 
> state Closed has been reached?
> We should not allow that.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/unicorn-wg/gecko-dev/pull/217/files#r19188903>.
>
    You're right, I must have read that check as aSignalingState == 
SignalingClosed. Will fix.