ReceiveChannel.onReceiveOrClosed throws when selecting on already-closed channel

[zach-klippenstein]

When using onReceiveOrClosed (#330) to select on a channel that has already been closed, the select builder fails.

Extremely simple to reproduce:

runBlocking {
  val channel = Channel<Unit>(RENDEZVOUS)
  channel.close()
  select<Unit> {
    channel.onReceiveOrClosed {
      println(it)
    }
  }
}

Exception:

java.lang.AssertionError
	at kotlinx.coroutines.selects.SelectBuilderImpl.resumeWith(Select.kt:444)
	at kotlinx.coroutines.intrinsics.UndispatchedKt.startCoroutineUnintercepted(Undispatched.kt:171)
	at kotlinx.coroutines.channels.AbstractChannel.registerSelectReceiveOrClosed(AbstractChannel.kt:749)
	at kotlinx.coroutines.channels.AbstractChannel.access$registerSelectReceiveOrClosed(AbstractChannel.kt:472)
	at kotlinx.coroutines.channels.AbstractChannel$onReceiveOrClosed$1.registerSelectClause1(AbstractChannel.kt:733)
	at kotlinx.coroutines.selects.SelectBuilderImpl.invoke(Select.kt:415)
	at com.squareup.workflow.Repro$asplode$1.invokeSuspend(Repro.kt:44)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
	at kotlinx.coroutines.DispatchedTask.run(Dispatched.kt:241)
	at kotlinx.coroutines.EventLoopImplBase.processNextEvent(EventLoop.common.kt:270)
	at kotlinx.coroutines.BlockingCoroutine.joinBlocking(Builders.kt:79)
	at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking(Builders.kt:54)
	at kotlinx.coroutines.BuildersKt.runBlocking(Unknown Source)
	at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking$default(Builders.kt:36)
	at kotlinx.coroutines.BuildersKt.runBlocking$default(Unknown Source)

Replacing onReceiveOrClosed with onReceiveOrNull works as expected – null is returned from select and the program finishes normally.

Versions:
Kotlin: 1.3.50
Coroutines: 1.3.1