grpc / grpc-java / 3100

87%

master: 89%

push

travis-ci

Run createTransportRunnable outside of lock.

Because `scheduleConnection()` is run under lock, if we ran
`createTransportRunnable` inside `scheduleConnection()`,
`savedDelayedTransport.setTransport()` will be under lock which violates
the assumption made in https://github.com/grpc/grpc-java/issues/1408 that

> there is an implicit rule today that channel layer will not hold any lock while calling into transport

and had caused deadlock with `InProcessTransport`.

This is a partial back-port of 0e14516f5
from master. A large portion of changes on tests is not ported because
of conflict caused by API changes in master.

19 of 19 new or added lines in 1 file covered. (100.0%)

5619 of 6450 relevant lines covered (87.12%)

0.87 hits per line