Don't add API to Optional

src/core/ext/filters/client_channel/lb_policy/xds/xds.cc

@@ -1530,8 +1530,8 @@ void XdsLb::UpdateLocked(UpdateArgs args) {
     return;
   }
   ProcessAddressesAndChannelArgsLocked(args.addresses, *args.args);
-  locality_map_.UpdateLocked(locality_serverlist_, child_policy_config_,
+  locality_map_.UpdateLocked(locality_serverlist_, child_policy_config_, args_,
-                             args_, this);
+                             this);
   // Update the existing fallback policy. The fallback policy config and/or the
   // fallback addresses may be new.
   if (fallback_policy_ != nullptr) UpdateFallbackPolicyLocked();

src/core/ext/filters/client_channel/resolver_result_parsing.cc

@@ -401,8 +401,8 @@ ClientChannelServiceConfigParser::ParseGlobalParams(const grpc_json* json,
         error_list.push_back(GRPC_ERROR_CREATE_FROM_STATIC_STRING(
             "field:retryThrottling error:Duplicate entry"));
       } else {
-        Optional<int> max_milli_tokens(false, 0);
+        Optional<int> max_milli_tokens;
-        Optional<int> milli_token_ratio(false, 0);
+        Optional<int> milli_token_ratio;
         for (grpc_json* sub_field = field->child; sub_field != nullptr;
              sub_field = sub_field->next) {
           if (sub_field->key == nullptr) continue;

src/core/lib/gprpp/optional.h

@@ -26,9 +26,6 @@ namespace grpc_core {
 template <typename T>
 class Optional {
  public:
-  Optional() = default;
-
-  Optional(bool set, T value) : set_(set), value_(value) {}
   void set(const T& val) {
     value_ = val;
     set_ = true;
@@ -42,7 +39,7 @@ class Optional {
 
  private:
   bool set_ = false;
-  T value_;
+  T value_ = {};
 };
 
 } /* namespace grpc_core */