grpc-flamingo 1.11.0 → 1.15.0

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grpc-flamingo
 version: !ruby/object:Gem::Version
-  version: 1.11.0
+  version: 1.15.0
 platform: ruby
 authors:
 - gRPC Authors
 autorequire: 
 bindir: src/ruby/bin
 cert_chain: []
-date: 2018-04-13 00:00:00.000000000 Z
+date: 2018-09-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-protobuf
@@ -24,26 +24,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.1'
-- !ruby/object:Gem::Dependency
-  name: googleauth
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '0.7'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.5.1
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '0.7'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.5.1
 - !ruby/object:Gem::Dependency
   name: googleapis-common-protos-types
   requirement: !ruby/object:Gem::Requirement
@@ -198,6 +178,26 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.7.0
+- !ruby/object:Gem::Dependency
+  name: googleauth
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.5.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.5.1
 description: Send RPCs from Ruby using GRPC
 email: temiola@google.com
 executables: []
@@ -230,6 +230,7 @@ files:
 - include/grpc/impl/codegen/gpr_slice.h
 - include/grpc/impl/codegen/gpr_types.h
 - include/grpc/impl/codegen/grpc_types.h
+- include/grpc/impl/codegen/log.h
 - include/grpc/impl/codegen/port_platform.h
 - include/grpc/impl/codegen/propagation_bits.h
 - include/grpc/impl/codegen/slice.h
@@ -263,12 +264,14 @@ files:
 - include/grpc/support/time.h
 - include/grpc/support/workaround_list.h
 - src/boringssl/err_data.c
-- src/core/ext/census/grpc_context.cc
+- src/core/ext/filters/census/grpc_context.cc
 - src/core/ext/filters/client_channel/backup_poller.cc
 - src/core/ext/filters/client_channel/backup_poller.h
 - src/core/ext/filters/client_channel/channel_connectivity.cc
 - src/core/ext/filters/client_channel/client_channel.cc
 - src/core/ext/filters/client_channel/client_channel.h
+- src/core/ext/filters/client_channel/client_channel_channelz.cc
+- src/core/ext/filters/client_channel/client_channel_channelz.h
 - src/core/ext/filters/client_channel/client_channel_factory.cc
 - src/core/ext/filters/client_channel/client_channel_factory.h
 - src/core/ext/filters/client_channel/client_channel_plugin.cc
@@ -283,17 +286,21 @@ files:
 - src/core/ext/filters/client_channel/lb_policy/grpclb/client_load_reporting_filter.cc
 - src/core/ext/filters/client_channel/lb_policy/grpclb/client_load_reporting_filter.h
 - src/core/ext/filters/client_channel/lb_policy/grpclb/grpclb.cc
+- src/core/ext/filters/client_channel/lb_policy/grpclb/grpclb.h
 - src/core/ext/filters/client_channel/lb_policy/grpclb/grpclb_channel.h
 - src/core/ext/filters/client_channel/lb_policy/grpclb/grpclb_channel_secure.cc
 - src/core/ext/filters/client_channel/lb_policy/grpclb/grpclb_client_stats.cc
 - src/core/ext/filters/client_channel/lb_policy/grpclb/grpclb_client_stats.h
 - src/core/ext/filters/client_channel/lb_policy/grpclb/load_balancer_api.cc
 - src/core/ext/filters/client_channel/lb_policy/grpclb/load_balancer_api.h
+- src/core/ext/filters/client_channel/lb_policy/grpclb/proto/grpc/lb/v1/google/protobuf/duration.pb.c
+- src/core/ext/filters/client_channel/lb_policy/grpclb/proto/grpc/lb/v1/google/protobuf/duration.pb.h
+- src/core/ext/filters/client_channel/lb_policy/grpclb/proto/grpc/lb/v1/google/protobuf/timestamp.pb.c
+- src/core/ext/filters/client_channel/lb_policy/grpclb/proto/grpc/lb/v1/google/protobuf/timestamp.pb.h
 - src/core/ext/filters/client_channel/lb_policy/grpclb/proto/grpc/lb/v1/load_balancer.pb.c
 - src/core/ext/filters/client_channel/lb_policy/grpclb/proto/grpc/lb/v1/load_balancer.pb.h
 - src/core/ext/filters/client_channel/lb_policy/pick_first/pick_first.cc
 - src/core/ext/filters/client_channel/lb_policy/round_robin/round_robin.cc
-- src/core/ext/filters/client_channel/lb_policy/subchannel_list.cc
 - src/core/ext/filters/client_channel/lb_policy/subchannel_list.h
 - src/core/ext/filters/client_channel/lb_policy_factory.cc
 - src/core/ext/filters/client_channel/lb_policy_factory.h
@@ -310,11 +317,15 @@ files:
 - src/core/ext/filters/client_channel/resolver.cc
 - src/core/ext/filters/client_channel/resolver.h
 - src/core/ext/filters/client_channel/resolver/dns/c_ares/dns_resolver_ares.cc
+- src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_ev_driver.cc
 - src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_ev_driver.h
 - src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_ev_driver_posix.cc
+- src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_ev_driver_windows.cc
 - src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_wrapper.cc
 - src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_wrapper.h
 - src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_wrapper_fallback.cc
+- src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_wrapper_posix.cc
+- src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_wrapper_windows.cc
 - src/core/ext/filters/client_channel/resolver/dns/native/dns_resolver.cc
 - src/core/ext/filters/client_channel/resolver/fake/fake_resolver.cc
 - src/core/ext/filters/client_channel/resolver/fake/fake_resolver.h
@@ -341,10 +352,6 @@ files:
 - src/core/ext/filters/http/message_compress/message_compress_filter.h
 - src/core/ext/filters/http/server/http_server_filter.cc
 - src/core/ext/filters/http/server/http_server_filter.h
-- src/core/ext/filters/load_reporting/server_load_reporting_filter.cc
-- src/core/ext/filters/load_reporting/server_load_reporting_filter.h
-- src/core/ext/filters/load_reporting/server_load_reporting_plugin.cc
-- src/core/ext/filters/load_reporting/server_load_reporting_plugin.h
 - src/core/ext/filters/max_age/max_age_filter.cc
 - src/core/ext/filters/max_age/max_age_filter.h
 - src/core/ext/filters/message_size/message_size_filter.cc
@@ -424,8 +431,10 @@ files:
 - src/core/lib/channel/channel_stack_builder.h
 - src/core/lib/channel/channel_trace.cc
 - src/core/lib/channel/channel_trace.h
-- src/core/lib/channel/channel_trace_registry.cc
-- src/core/lib/channel/channel_trace_registry.h
+- src/core/lib/channel/channelz.cc
+- src/core/lib/channel/channelz.h
+- src/core/lib/channel/channelz_registry.cc
+- src/core/lib/channel/channelz_registry.h
 - src/core/lib/channel/connected_channel.cc
 - src/core/lib/channel/connected_channel.h
 - src/core/lib/channel/context.h
@@ -456,6 +465,7 @@ files:
 - src/core/lib/debug/trace.cc
 - src/core/lib/debug/trace.h
 - src/core/lib/gpr/alloc.cc
+- src/core/lib/gpr/alloc.h
 - src/core/lib/gpr/arena.cc
 - src/core/lib/gpr/arena.h
 - src/core/lib/gpr/atm.cc
@@ -467,8 +477,6 @@ files:
 - src/core/lib/gpr/env_linux.cc
 - src/core/lib/gpr/env_posix.cc
 - src/core/lib/gpr/env_windows.cc
-- src/core/lib/gpr/fork.cc
-- src/core/lib/gpr/fork.h
 - src/core/lib/gpr/host_port.cc
 - src/core/lib/gpr/host_port.h
 - src/core/lib/gpr/log.cc
@@ -511,9 +519,12 @@ files:
 - src/core/lib/gprpp/atomic_with_atm.h
 - src/core/lib/gprpp/atomic_with_std.h
 - src/core/lib/gprpp/debug_location.h
+- src/core/lib/gprpp/fork.cc
+- src/core/lib/gprpp/fork.h
 - src/core/lib/gprpp/inlined_vector.h
 - src/core/lib/gprpp/manual_constructor.h
 - src/core/lib/gprpp/memory.h
+- src/core/lib/gprpp/mutex_lock.h
 - src/core/lib/gprpp/orphanable.h
 - src/core/lib/gprpp/ref_counted.h
 - src/core/lib/gprpp/ref_counted_ptr.h
@@ -717,6 +728,8 @@ files:
 - src/core/lib/security/credentials/jwt/jwt_credentials.h
 - src/core/lib/security/credentials/jwt/jwt_verifier.cc
 - src/core/lib/security/credentials/jwt/jwt_verifier.h
+- src/core/lib/security/credentials/local/local_credentials.cc
+- src/core/lib/security/credentials/local/local_credentials.h
 - src/core/lib/security/credentials/oauth2/oauth2_credentials.cc
 - src/core/lib/security/credentials/oauth2/oauth2_credentials.h
 - src/core/lib/security/credentials/plugin/plugin_credentials.cc
@@ -725,6 +738,12 @@ files:
 - src/core/lib/security/credentials/ssl/ssl_credentials.h
 - src/core/lib/security/security_connector/alts_security_connector.cc
 - src/core/lib/security/security_connector/alts_security_connector.h
+- src/core/lib/security/security_connector/load_system_roots.h
+- src/core/lib/security/security_connector/load_system_roots_fallback.cc
+- src/core/lib/security/security_connector/load_system_roots_linux.cc
+- src/core/lib/security/security_connector/load_system_roots_linux.h
+- src/core/lib/security/security_connector/local_security_connector.cc
+- src/core/lib/security/security_connector/local_security_connector.h
 - src/core/lib/security/security_connector/security_connector.cc
 - src/core/lib/security/security_connector/security_connector.h
 - src/core/lib/security/transport/auth_filters.h
@@ -866,6 +885,9 @@ files:
 - src/core/tsi/alts_transport_security.h
 - src/core/tsi/fake_transport_security.cc
 - src/core/tsi/fake_transport_security.h
+- src/core/tsi/grpc_shadow_boringssl.h
+- src/core/tsi/local_transport_security.cc
+- src/core/tsi/local_transport_security.h
 - src/core/tsi/ssl/session_cache/ssl_session.h
 - src/core/tsi/ssl/session_cache/ssl_session_boringssl.cc
 - src/core/tsi/ssl/session_cache/ssl_session_cache.cc
@@ -876,15 +898,9 @@ files:
 - src/core/tsi/ssl_types.h
 - src/core/tsi/transport_security.cc
 - src/core/tsi/transport_security.h
-- src/core/tsi/transport_security_adapter.cc
-- src/core/tsi/transport_security_adapter.h
 - src/core/tsi/transport_security_grpc.cc
 - src/core/tsi/transport_security_grpc.h
 - src/core/tsi/transport_security_interface.h
-- src/ruby/bin/apis/google/protobuf/empty.rb
-- src/ruby/bin/apis/pubsub_demo.rb
-- src/ruby/bin/apis/tech/pubsub/proto/pubsub.rb
-- src/ruby/bin/apis/tech/pubsub/proto/pubsub_services.rb
 - src/ruby/bin/math_client.rb
 - src/ruby/bin/math_pb.rb
 - src/ruby/bin/math_server.rb
@@ -948,8 +964,6 @@ files:
 - src/ruby/pb/src/proto/grpc/testing/messages_pb.rb
 - src/ruby/pb/src/proto/grpc/testing/test_pb.rb
 - src/ruby/pb/src/proto/grpc/testing/test_services_pb.rb
-- src/ruby/pb/test/client.rb
-- src/ruby/pb/test/server.rb
 - src/ruby/spec/call_credentials_spec.rb
 - src/ruby/spec/call_spec.rb
 - src/ruby/spec/channel_connection_spec.rb
@@ -971,9 +985,6 @@ files:
 - src/ruby/spec/google_rpc_status_utils_spec.rb
 - src/ruby/spec/pb/duplicate/codegen_spec.rb
 - src/ruby/spec/pb/health/checker_spec.rb
-- src/ruby/spec/pb/package_with_underscore/checker_spec.rb
-- src/ruby/spec/pb/package_with_underscore/data.proto
-- src/ruby/spec/pb/package_with_underscore/service.proto
 - src/ruby/spec/server_credentials_spec.rb
 - src/ruby/spec/server_spec.rb
 - src/ruby/spec/spec_helper.rb
@@ -1045,6 +1056,7 @@ files:
 - third_party/boringssl/crypto/chacha/chacha.c
 - third_party/boringssl/crypto/cipher_extra/cipher_extra.c
 - third_party/boringssl/crypto/cipher_extra/derive_key.c
+- third_party/boringssl/crypto/cipher_extra/e_aesccm.c
 - third_party/boringssl/crypto/cipher_extra/e_aesctrhmac.c
 - third_party/boringssl/crypto/cipher_extra/e_aesgcmsiv.c
 - third_party/boringssl/crypto/cipher_extra/e_chacha20poly1305.c
@@ -1059,6 +1071,7 @@ files:
 - third_party/boringssl/crypto/conf/conf.c
 - third_party/boringssl/crypto/conf/conf_def.h
 - third_party/boringssl/crypto/conf/internal.h
+- third_party/boringssl/crypto/cpu-aarch64-fuchsia.c
 - third_party/boringssl/crypto/cpu-aarch64-linux.c
 - third_party/boringssl/crypto/cpu-arm-linux.c
 - third_party/boringssl/crypto/cpu-arm.c
@@ -1066,7 +1079,6 @@ files:
 - third_party/boringssl/crypto/cpu-ppc64le.c
 - third_party/boringssl/crypto/crypto.c
 - third_party/boringssl/crypto/curve25519/spake25519.c
-- third_party/boringssl/crypto/curve25519/x25519-x86_64.c
 - third_party/boringssl/crypto/dh/check.c
 - third_party/boringssl/crypto/dh/dh.c
 - third_party/boringssl/crypto/dh/dh_asn1.c
@@ -1141,12 +1153,11 @@ files:
 - third_party/boringssl/crypto/fipsmodule/ec/internal.h
 - third_party/boringssl/crypto/fipsmodule/ec/oct.c
 - third_party/boringssl/crypto/fipsmodule/ec/p224-64.c
-- third_party/boringssl/crypto/fipsmodule/ec/p256-64.c
 - third_party/boringssl/crypto/fipsmodule/ec/p256-x86_64-table.h
 - third_party/boringssl/crypto/fipsmodule/ec/p256-x86_64.c
 - third_party/boringssl/crypto/fipsmodule/ec/p256-x86_64.h
 - third_party/boringssl/crypto/fipsmodule/ec/simple.c
-- third_party/boringssl/crypto/fipsmodule/ec/util-64.c
+- third_party/boringssl/crypto/fipsmodule/ec/util.c
 - third_party/boringssl/crypto/fipsmodule/ec/wnaf.c
 - third_party/boringssl/crypto/fipsmodule/ecdsa/ecdsa.c
 - third_party/boringssl/crypto/fipsmodule/hmac/hmac.c
@@ -1154,6 +1165,7 @@ files:
 - third_party/boringssl/crypto/fipsmodule/md4/md4.c
 - third_party/boringssl/crypto/fipsmodule/md5/md5.c
 - third_party/boringssl/crypto/fipsmodule/modes/cbc.c
+- third_party/boringssl/crypto/fipsmodule/modes/ccm.c
 - third_party/boringssl/crypto/fipsmodule/modes/cfb.c
 - third_party/boringssl/crypto/fipsmodule/modes/ctr.c
 - third_party/boringssl/crypto/fipsmodule/modes/gcm.c
@@ -1169,10 +1181,13 @@ files:
 - third_party/boringssl/crypto/fipsmodule/rsa/padding.c
 - third_party/boringssl/crypto/fipsmodule/rsa/rsa.c
 - third_party/boringssl/crypto/fipsmodule/rsa/rsa_impl.c
+- third_party/boringssl/crypto/fipsmodule/self_check/self_check.c
 - third_party/boringssl/crypto/fipsmodule/sha/sha1-altivec.c
 - third_party/boringssl/crypto/fipsmodule/sha/sha1.c
 - third_party/boringssl/crypto/fipsmodule/sha/sha256.c
 - third_party/boringssl/crypto/fipsmodule/sha/sha512.c
+- third_party/boringssl/crypto/fipsmodule/tls/internal.h
+- third_party/boringssl/crypto/fipsmodule/tls/kdf.c
 - third_party/boringssl/crypto/hkdf/hkdf.c
 - third_party/boringssl/crypto/internal.h
 - third_party/boringssl/crypto/lhash/lhash.c
@@ -1382,6 +1397,7 @@ files:
 - third_party/boringssl/ssl/d1_srtp.cc
 - third_party/boringssl/ssl/dtls_method.cc
 - third_party/boringssl/ssl/dtls_record.cc
+- third_party/boringssl/ssl/handoff.cc
 - third_party/boringssl/ssl/handshake.cc
 - third_party/boringssl/ssl/handshake_client.cc
 - third_party/boringssl/ssl/handshake_server.cc
@@ -1412,7 +1428,9 @@ files:
 - third_party/boringssl/ssl/tls_method.cc
 - third_party/boringssl/ssl/tls_record.cc
 - third_party/boringssl/third_party/fiat/curve25519.c
+- third_party/boringssl/third_party/fiat/curve25519_tables.h
 - third_party/boringssl/third_party/fiat/internal.h
+- third_party/boringssl/third_party/fiat/p256.c
 - third_party/cares/ares_build.h
 - third_party/cares/cares/ares.h
 - third_party/cares/cares/ares__close_sockets.c
@@ -1548,39 +1566,36 @@ signing_key:
 specification_version: 4
 summary: GRPC system in Ruby
 test_files:
-- src/ruby/spec/time_consts_spec.rb
-- src/ruby/spec/compression_options_spec.rb
-- src/ruby/spec/client_auth_spec.rb
-- src/ruby/spec/pb/duplicate/codegen_spec.rb
-- src/ruby/spec/pb/health/checker_spec.rb
-- src/ruby/spec/pb/package_with_underscore/checker_spec.rb
-- src/ruby/spec/pb/package_with_underscore/service.proto
-- src/ruby/spec/pb/package_with_underscore/data.proto
-- src/ruby/spec/spec_helper.rb
-- src/ruby/spec/server_spec.rb
-- src/ruby/spec/google_rpc_status_utils_spec.rb
-- src/ruby/spec/client_server_spec.rb
-- src/ruby/spec/channel_credentials_spec.rb
-- src/ruby/spec/call_credentials_spec.rb
-- src/ruby/spec/call_spec.rb
-- src/ruby/spec/channel_connection_spec.rb
-- src/ruby/spec/channel_spec.rb
-- src/ruby/spec/error_sanity_spec.rb
-- src/ruby/spec/support/helpers.rb
 - src/ruby/spec/support/services.rb
-- src/ruby/spec/server_credentials_spec.rb
-- src/ruby/spec/generic/interceptor_registry_spec.rb
-- src/ruby/spec/generic/client_stub_spec.rb
+- src/ruby/spec/support/helpers.rb
+- src/ruby/spec/generic/client_interceptors_spec.rb
 - src/ruby/spec/generic/server_interceptors_spec.rb
-- src/ruby/spec/generic/rpc_server_spec.rb
+- src/ruby/spec/generic/client_stub_spec.rb
 - src/ruby/spec/generic/rpc_desc_spec.rb
-- src/ruby/spec/generic/client_interceptors_spec.rb
 - src/ruby/spec/generic/service_spec.rb
+- src/ruby/spec/generic/rpc_server_spec.rb
 - src/ruby/spec/generic/active_call_spec.rb
 - src/ruby/spec/generic/rpc_server_pool_spec.rb
+- src/ruby/spec/generic/interceptor_registry_spec.rb
+- src/ruby/spec/google_rpc_status_utils_spec.rb
 - src/ruby/spec/testdata/client.key
 - src/ruby/spec/testdata/README
-- src/ruby/spec/testdata/ca.pem
 - src/ruby/spec/testdata/client.pem
 - src/ruby/spec/testdata/server1.pem
+- src/ruby/spec/testdata/ca.pem
 - src/ruby/spec/testdata/server1.key
+- src/ruby/spec/server_spec.rb
+- src/ruby/spec/error_sanity_spec.rb
+- src/ruby/spec/compression_options_spec.rb
+- src/ruby/spec/client_server_spec.rb
+- src/ruby/spec/call_credentials_spec.rb
+- src/ruby/spec/pb/health/checker_spec.rb
+- src/ruby/spec/pb/duplicate/codegen_spec.rb
+- src/ruby/spec/channel_spec.rb
+- src/ruby/spec/channel_connection_spec.rb
+- src/ruby/spec/spec_helper.rb
+- src/ruby/spec/server_credentials_spec.rb
+- src/ruby/spec/channel_credentials_spec.rb
+- src/ruby/spec/time_consts_spec.rb
+- src/ruby/spec/client_auth_spec.rb
+- src/ruby/spec/call_spec.rb

data/src/core/ext/filters/client_channel/lb_policy/subchannel_list.cc

@@ -1,253 +0,0 @@
-/*
- *
- * Copyright 2015 gRPC authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- *
- */
-
-#include <grpc/support/port_platform.h>
-
-#include <string.h>
-
-#include <grpc/support/alloc.h>
-
-#include "src/core/ext/filters/client_channel/lb_policy/subchannel_list.h"
-#include "src/core/lib/channel/channel_args.h"
-#include "src/core/lib/debug/trace.h"
-#include "src/core/lib/iomgr/closure.h"
-#include "src/core/lib/iomgr/combiner.h"
-#include "src/core/lib/iomgr/sockaddr_utils.h"
-#include "src/core/lib/transport/connectivity_state.h"
-
-void grpc_lb_subchannel_data_unref_subchannel(grpc_lb_subchannel_data* sd,
-                                              const char* reason) {
-  if (sd->subchannel != nullptr) {
-    if (sd->subchannel_list->tracer->enabled()) {
-      gpr_log(GPR_DEBUG,
-              "[%s %p] subchannel list %p index %" PRIuPTR " of %" PRIuPTR
-              " (subchannel %p): unreffing subchannel",
-              sd->subchannel_list->tracer->name(), sd->subchannel_list->policy,
-              sd->subchannel_list,
-              static_cast<size_t>(sd - sd->subchannel_list->subchannels),
-              sd->subchannel_list->num_subchannels, sd->subchannel);
-    }
-    GRPC_SUBCHANNEL_UNREF(sd->subchannel, reason);
-    sd->subchannel = nullptr;
-    sd->connected_subchannel.reset();
-    if (sd->user_data != nullptr) {
-      GPR_ASSERT(sd->user_data_vtable != nullptr);
-      sd->user_data_vtable->destroy(sd->user_data);
-      sd->user_data = nullptr;
-    }
-  }
-}
-
-void grpc_lb_subchannel_data_start_connectivity_watch(
-    grpc_lb_subchannel_data* sd) {
-  if (sd->subchannel_list->tracer->enabled()) {
-    gpr_log(
-        GPR_DEBUG,
-        "[%s %p] subchannel list %p index %" PRIuPTR " of %" PRIuPTR
-        " (subchannel %p): requesting connectivity change "
-        "notification (from %s)",
-        sd->subchannel_list->tracer->name(), sd->subchannel_list->policy,
-        sd->subchannel_list,
-        static_cast<size_t>(sd - sd->subchannel_list->subchannels),
-        sd->subchannel_list->num_subchannels, sd->subchannel,
-        grpc_connectivity_state_name(sd->pending_connectivity_state_unsafe));
-  }
-  sd->connectivity_notification_pending = true;
-  grpc_subchannel_notify_on_state_change(
-      sd->subchannel, sd->subchannel_list->policy->interested_parties(),
-      &sd->pending_connectivity_state_unsafe,
-      &sd->connectivity_changed_closure);
-}
-
-void grpc_lb_subchannel_data_stop_connectivity_watch(
-    grpc_lb_subchannel_data* sd) {
-  if (sd->subchannel_list->tracer->enabled()) {
-    gpr_log(GPR_DEBUG,
-            "[%s %p] subchannel list %p index %" PRIuPTR " of %" PRIuPTR
-            " (subchannel %p): stopping connectivity watch",
-            sd->subchannel_list->tracer->name(), sd->subchannel_list->policy,
-            sd->subchannel_list,
-            static_cast<size_t>(sd - sd->subchannel_list->subchannels),
-            sd->subchannel_list->num_subchannels, sd->subchannel);
-  }
-  GPR_ASSERT(sd->connectivity_notification_pending);
-  sd->connectivity_notification_pending = false;
-}
-
-grpc_lb_subchannel_list* grpc_lb_subchannel_list_create(
-    grpc_core::LoadBalancingPolicy* p, grpc_core::TraceFlag* tracer,
-    const grpc_lb_addresses* addresses, grpc_combiner* combiner,
-    grpc_client_channel_factory* client_channel_factory,
-    const grpc_channel_args& args, grpc_iomgr_cb_func connectivity_changed_cb) {
-  grpc_lb_subchannel_list* subchannel_list =
-      static_cast<grpc_lb_subchannel_list*>(
-          gpr_zalloc(sizeof(*subchannel_list)));
-  if (tracer->enabled()) {
-    gpr_log(GPR_DEBUG,
-            "[%s %p] Creating subchannel list %p for %" PRIuPTR " subchannels",
-            tracer->name(), p, subchannel_list, addresses->num_addresses);
-  }
-  subchannel_list->policy = p;
-  subchannel_list->tracer = tracer;
-  gpr_ref_init(&subchannel_list->refcount, 1);
-  subchannel_list->subchannels = static_cast<grpc_lb_subchannel_data*>(
-      gpr_zalloc(sizeof(grpc_lb_subchannel_data) * addresses->num_addresses));
-  // We need to remove the LB addresses in order to be able to compare the
-  // subchannel keys of subchannels from a different batch of addresses.
-  static const char* keys_to_remove[] = {GRPC_ARG_SUBCHANNEL_ADDRESS,
-                                         GRPC_ARG_LB_ADDRESSES};
-  // Create a subchannel for each address.
-  grpc_subchannel_args sc_args;
-  size_t subchannel_index = 0;
-  for (size_t i = 0; i < addresses->num_addresses; i++) {
-    // If there were any balancer, we would have chosen grpclb policy instead.
-    GPR_ASSERT(!addresses->addresses[i].is_balancer);
-    memset(&sc_args, 0, sizeof(grpc_subchannel_args));
-    grpc_arg addr_arg =
-        grpc_create_subchannel_address_arg(&addresses->addresses[i].address);
-    grpc_channel_args* new_args = grpc_channel_args_copy_and_add_and_remove(
-        &args, keys_to_remove, GPR_ARRAY_SIZE(keys_to_remove), &addr_arg, 1);
-    gpr_free(addr_arg.value.string);
-    sc_args.args = new_args;
-    grpc_subchannel* subchannel = grpc_client_channel_factory_create_subchannel(
-        client_channel_factory, &sc_args);
-    grpc_channel_args_destroy(new_args);
-    if (subchannel == nullptr) {
-      // Subchannel could not be created.
-      if (tracer->enabled()) {
-        char* address_uri =
-            grpc_sockaddr_to_uri(&addresses->addresses[i].address);
-        gpr_log(GPR_DEBUG,
-                "[%s %p] could not create subchannel for address uri %s, "
-                "ignoring",
-                tracer->name(), subchannel_list->policy, address_uri);
-        gpr_free(address_uri);
-      }
-      continue;
-    }
-    if (tracer->enabled()) {
-      char* address_uri =
-          grpc_sockaddr_to_uri(&addresses->addresses[i].address);
-      gpr_log(GPR_DEBUG,
-              "[%s %p] subchannel list %p index %" PRIuPTR
-              ": Created subchannel %p for address uri %s",
-              tracer->name(), p, subchannel_list, subchannel_index, subchannel,
-              address_uri);
-      gpr_free(address_uri);
-    }
-    grpc_lb_subchannel_data* sd =
-        &subchannel_list->subchannels[subchannel_index++];
-    sd->subchannel_list = subchannel_list;
-    sd->subchannel = subchannel;
-    GRPC_CLOSURE_INIT(&sd->connectivity_changed_closure,
-                      connectivity_changed_cb, sd,
-                      grpc_combiner_scheduler(combiner));
-    // We assume that the current state is IDLE.  If not, we'll get a
-    // callback telling us that.
-    sd->prev_connectivity_state = GRPC_CHANNEL_IDLE;
-    sd->curr_connectivity_state = GRPC_CHANNEL_IDLE;
-    sd->pending_connectivity_state_unsafe = GRPC_CHANNEL_IDLE;
-    sd->user_data_vtable = addresses->user_data_vtable;
-    if (sd->user_data_vtable != nullptr) {
-      sd->user_data =
-          sd->user_data_vtable->copy(addresses->addresses[i].user_data);
-    }
-  }
-  subchannel_list->num_subchannels = subchannel_index;
-  subchannel_list->num_idle = subchannel_index;
-  return subchannel_list;
-}
-
-static void subchannel_list_destroy(grpc_lb_subchannel_list* subchannel_list) {
-  if (subchannel_list->tracer->enabled()) {
-    gpr_log(GPR_DEBUG, "[%s %p] Destroying subchannel_list %p",
-            subchannel_list->tracer->name(), subchannel_list->policy,
-            subchannel_list);
-  }
-  for (size_t i = 0; i < subchannel_list->num_subchannels; i++) {
-    grpc_lb_subchannel_data* sd = &subchannel_list->subchannels[i];
-    grpc_lb_subchannel_data_unref_subchannel(sd, "subchannel_list_destroy");
-  }
-  gpr_free(subchannel_list->subchannels);
-  gpr_free(subchannel_list);
-}
-
-void grpc_lb_subchannel_list_ref(grpc_lb_subchannel_list* subchannel_list,
-                                 const char* reason) {
-  gpr_ref_non_zero(&subchannel_list->refcount);
-  if (subchannel_list->tracer->enabled()) {
-    const gpr_atm count = gpr_atm_acq_load(&subchannel_list->refcount.count);
-    gpr_log(GPR_DEBUG, "[%s %p] subchannel_list %p REF %lu->%lu (%s)",
-            subchannel_list->tracer->name(), subchannel_list->policy,
-            subchannel_list, static_cast<unsigned long>(count - 1),
-            static_cast<unsigned long>(count), reason);
-  }
-}
-
-void grpc_lb_subchannel_list_unref(grpc_lb_subchannel_list* subchannel_list,
-                                   const char* reason) {
-  const bool done = gpr_unref(&subchannel_list->refcount);
-  if (subchannel_list->tracer->enabled()) {
-    const gpr_atm count = gpr_atm_acq_load(&subchannel_list->refcount.count);
-    gpr_log(GPR_DEBUG, "[%s %p] subchannel_list %p UNREF %lu->%lu (%s)",
-            subchannel_list->tracer->name(), subchannel_list->policy,
-            subchannel_list, static_cast<unsigned long>(count + 1),
-            static_cast<unsigned long>(count), reason);
-  }
-  if (done) {
-    subchannel_list_destroy(subchannel_list);
-  }
-}
-
-static void subchannel_data_cancel_connectivity_watch(
-    grpc_lb_subchannel_data* sd, const char* reason) {
-  if (sd->subchannel_list->tracer->enabled()) {
-    gpr_log(GPR_DEBUG,
-            "[%s %p] subchannel list %p index %" PRIuPTR " of %" PRIuPTR
-            " (subchannel %p): canceling connectivity watch (%s)",
-            sd->subchannel_list->tracer->name(), sd->subchannel_list->policy,
-            sd->subchannel_list,
-            static_cast<size_t>(sd - sd->subchannel_list->subchannels),
-            sd->subchannel_list->num_subchannels, sd->subchannel, reason);
-  }
-  grpc_subchannel_notify_on_state_change(sd->subchannel, nullptr, nullptr,
-                                         &sd->connectivity_changed_closure);
-}
-
-void grpc_lb_subchannel_list_shutdown_and_unref(
-    grpc_lb_subchannel_list* subchannel_list, const char* reason) {
-  if (subchannel_list->tracer->enabled()) {
-    gpr_log(GPR_DEBUG, "[%s %p] Shutting down subchannel_list %p (%s)",
-            subchannel_list->tracer->name(), subchannel_list->policy,
-            subchannel_list, reason);
-  }
-  GPR_ASSERT(!subchannel_list->shutting_down);
-  subchannel_list->shutting_down = true;
-  for (size_t i = 0; i < subchannel_list->num_subchannels; i++) {
-    grpc_lb_subchannel_data* sd = &subchannel_list->subchannels[i];
-    // If there's a pending notification for this subchannel, cancel it;
-    // the callback is responsible for unreffing the subchannel.
-    // Otherwise, unref the subchannel directly.
-    if (sd->connectivity_notification_pending) {
-      subchannel_data_cancel_connectivity_watch(sd, reason);
-    } else if (sd->subchannel != nullptr) {
-      grpc_lb_subchannel_data_unref_subchannel(sd, reason);
-    }
-  }
-  grpc_lb_subchannel_list_unref(subchannel_list, reason);
-}