Replace the use of detail::TrimString with common::Trim

Author: psx95

api/include/opentelemetry/trace/propagation/detail/string.h

@@ -56,31 +56,6 @@ inline size_t SplitString(nostd::string_view s,
 
   return filled;
 }
-
-inline nostd::string_view TrimString(nostd::string_view s)
-{
-  size_t str_size = s.size();
-  size_t trimmed_str_start_pos = 0;
-  size_t trimmed_str_end_pos = 0;
-  bool start_pos_found = false;
-  for (size_t i = 0; i < str_size; i++)
-  {
-    if (!isspace(s[i]))
-    {
-      if (!start_pos_found)
-      {
-        trimmed_str_start_pos = i;
-        trimmed_str_end_pos = i;
-        start_pos_found = true;
-      } else
-      {
-        trimmed_str_end_pos = i;
-      }
-    }
-  }
-  return s.substr(trimmed_str_start_pos, trimmed_str_end_pos - trimmed_str_start_pos + 1);
-}
-
 }  // namespace detail
 }  // namespace propagation
 }  // namespace trace

api/include/opentelemetry/trace/propagation/http_trace_context.h

@@ -183,7 +183,7 @@ class HttpTraceContext : public context::propagation::TextMapPropagator
   static SpanContext ExtractImpl(const context::propagation::TextMapCarrier &carrier)
   {
     // Get trace_parent after trimming the leading and trailing whitespaces
-    nostd::string_view trace_parent = detail::TrimString(carrier.Get(kTraceParent));
+    nostd::string_view trace_parent = common::StringUtil::Trim(carrier.Get(kTraceParent));
     nostd::string_view trace_state  = carrier.Get(kTraceState);
     if (trace_parent == "")
     {

api/test/trace/propagation/detail/string_test.cc

@@ -7,7 +7,7 @@
 
 #include "opentelemetry/nostd/string_view.h"
 
-TEST(StringTest, TrimStrings)
+TEST(StringTest, SplitString)
 {
   //TODO(psx95): Add tests
 }