Only need to do state transition based on RR LB's state without looki…

…ng into each subchannel's state.

core/src/main/java/io/grpc/util/RoundRobinLoadBalancer.java

@@ -142,13 +142,7 @@ private void processSubchannelState(Subchannel subchannel, ConnectivityStateInfo
  if (stateInfo.getState() == IDLE) {
  subchannel.requestConnection();
  }
- Ref<ConnectivityStateInfo> subchannelStateRef = getSubchannelStateInfoRef(subchannel);
- if (subchannelStateRef.value.getState().equals(TRANSIENT_FAILURE)) {
- if (stateInfo.getState().equals(CONNECTING)) {
- return;
- }
- }
- subchannelStateRef.value = stateInfo;
+ getSubchannelStateInfoRef(subchannel).value = stateInfo;
  updateBalancingState();
  }
 
@@ -203,6 +197,9 @@ private void updateBalancingState() {
 
  private void updateBalancingState(ConnectivityState state, RoundRobinPicker picker) {
  if (state != currentState || !picker.isEquivalentTo(currentPicker)) {
+ if (currentState == TRANSIENT_FAILURE && state != READY) {
+ return;
+ }
  helper.updateBalancingState(state, picker);
  currentState = state;
  currentPicker = picker;

core/src/test/java/io/grpc/util/RoundRobinLoadBalancerTest.java

@@ -311,6 +311,13 @@ public void stayTransientFailureUntilReady() {
  }
  inOrder.verify(mockHelper).updateBalancingState(eq(TRANSIENT_FAILURE), isA(EmptyPicker.class));
 
+ for (Subchannel sc : loadBalancer.getSubchannels()) {
+ deliverSubchannelState(
+ sc,
+ ConnectivityStateInfo.forNonError(IDLE));
+ }
+ inOrder.verifyNoMoreInteractions();
+
  for (Subchannel sc : loadBalancer.getSubchannels()) {
  deliverSubchannelState(
  sc,