code review changes

test/core/channelz/channel_trace_test.cc

@@ -163,7 +163,7 @@ MATCHER_P(IsEmptyChannelTrace, num_events_logged_expected,
 
 void ValidateJsonProtoTranslation(const Json& json) {
   std::string json_str = JsonDump(json);
-  grpc::testing::ValidateChannelTraceProtoJsonTranslation(json_str.c_str());
+  grpc::testing::ValidateChannelTraceProtoJsonTranslation(json_str);
 }
 
 }  // anonymous namespace

test/cpp/util/channel_trace_proto_helper.cc

@@ -41,7 +41,7 @@ namespace {
 // then back to json. This ensures that the json string was correctly formatted
 // according to https://developers.google.com/protocol-buffers/docs/proto3#json
 template <typename Message>
-void VaidateProtoJsonTranslation(const std::string& json_str) {
+void VaidateProtoJsonTranslation(absl::string_view json_str) {
   Message msg;
   grpc::protobuf::json::JsonParseOptions parse_options;
   // If the following line is failing, then uncomment the last line of the
@@ -73,42 +73,42 @@ void VaidateProtoJsonTranslation(const std::string& json_str) {
 
 namespace testing {
 
-void ValidateChannelTraceProtoJsonTranslation(const std::string& json_string) {
+void ValidateChannelTraceProtoJsonTranslation(absl::string_view json_string) {
   VaidateProtoJsonTranslation<grpc::channelz::v1::ChannelTrace>(json_string);
 }
 
-void ValidateChannelProtoJsonTranslation(const std::string& json_string) {
+void ValidateChannelProtoJsonTranslation(absl::string_view json_string) {
   VaidateProtoJsonTranslation<grpc::channelz::v1::Channel>(json_string);
 }
 
 void ValidateGetTopChannelsResponseProtoJsonTranslation(
-    const std::string& json_string) {
+    absl::string_view json_string) {
   VaidateProtoJsonTranslation<grpc::channelz::v1::GetTopChannelsResponse>(
       json_string);
 }
 
 void ValidateGetChannelResponseProtoJsonTranslation(
-    const std::string& json_string) {
+    absl::string_view json_string) {
   VaidateProtoJsonTranslation<grpc::channelz::v1::GetChannelResponse>(
       json_string);
 }
 
 void ValidateGetServerResponseProtoJsonTranslation(
-    const std::string& json_string) {
+    absl::string_view json_string) {
   VaidateProtoJsonTranslation<grpc::channelz::v1::GetServerResponse>(
       json_string);
 }
 
-void ValidateSubchannelProtoJsonTranslation(const std::string& json_string) {
+void ValidateSubchannelProtoJsonTranslation(absl::string_view json_string) {
   VaidateProtoJsonTranslation<grpc::channelz::v1::Subchannel>(json_string);
 }
 
-void ValidateServerProtoJsonTranslation(const std::string& json_string) {
+void ValidateServerProtoJsonTranslation(absl::string_view json_string) {
   VaidateProtoJsonTranslation<grpc::channelz::v1::Server>(json_string);
 }
 
 void ValidateGetServersResponseProtoJsonTranslation(
-    const std::string& json_string) {
+    absl::string_view json_string) {
   VaidateProtoJsonTranslation<grpc::channelz::v1::GetServersResponse>(
       json_string);
 }

test/cpp/util/channel_trace_proto_helper.h

@@ -19,23 +19,23 @@
 #ifndef GRPC_TEST_CPP_UTIL_CHANNEL_TRACE_PROTO_HELPER_H
 #define GRPC_TEST_CPP_UTIL_CHANNEL_TRACE_PROTO_HELPER_H
 
-#include <string>
+#include "absl/strings/string_view.h"
 
 namespace grpc {
 namespace testing {
 
-void ValidateChannelTraceProtoJsonTranslation(const std::string& json_string);
-void ValidateChannelProtoJsonTranslation(const std::string& json_string);
+void ValidateChannelTraceProtoJsonTranslation(absl::string_view json_string);
+void ValidateChannelProtoJsonTranslation(absl::string_view json_string);
 void ValidateGetTopChannelsResponseProtoJsonTranslation(
-    const std::string& json_string);
+    absl::string_view json_string);
 void ValidateGetChannelResponseProtoJsonTranslation(
-    const std::string& json_string);
+    absl::string_view json_string);
 void ValidateGetServerResponseProtoJsonTranslation(
-    const std::string& json_string);
-void ValidateSubchannelProtoJsonTranslation(const std::string& json_string);
-void ValidateServerProtoJsonTranslation(const std::string& json_string);
+    absl::string_view json_string);
+void ValidateSubchannelProtoJsonTranslation(absl::string_view json_string);
+void ValidateServerProtoJsonTranslation(absl::string_view json_string);
 void ValidateGetServersResponseProtoJsonTranslation(
-    const std::string& json_string);
+    absl::string_view json_string);
 
 }  // namespace testing
 }  // namespace grpc