This looks like a mistaken removal to me, but I can't really be sure.  

I'd expect the call state to move to ongoing after an upgrade (reading other code suggests it would might be in the pending state up until then), which is what it looks like the above line does.  Interestingly, there are no tests in the tree at all for any sort of change to the call object at all after the media object fires connection-upgrade.  This might or might not be an inadvertent omission.  It's not clear to me what the actual consequences of the code removal are, if any, but I'm reluctant to r+ and merge this without more context.

@Standard8, do you have any insight here?