Suppose I call Bob, and then place the call on hold. If either I or Bob adds a new stream, then the channel's HoldState will return to PendingHold. But, SetStreamHeld is not emitted on the new stream, so the channel will never return to state Hold. Adding streams without taking the call off hold is a bit suspicious, but I think this should be symmetrical: regardless of who adds the stream, it shouldn't make the holder think they're not on hold, since the peer still thinks they're on hold.
The attached branch fixes this bug, and adds a regression test.
branc looks good
Merged, will be in 0.7.28.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.