Replace QuicString with std::string, pass 1

This replaces QuicString with std::string in all of the "QUIC proper".  I will
delete QuicString once all code using it is gone.

gfe-relnote: n/a (no functional change)
PiperOrigin-RevId: 237872023
Change-Id: I82de62c9855516b15039734d05155917e68ff4ee
diff --git a/quic/core/tls_handshaker.h b/quic/core/tls_handshaker.h
index ceba9a9..9658923 100644
--- a/quic/core/tls_handshaker.h
+++ b/quic/core/tls_handshaker.h
@@ -38,7 +38,7 @@
   bool ProcessInput(QuicStringPiece input, EncryptionLevel level) override;
   size_t InputBytesRemaining() const override { return 0; }
   QuicErrorCode error() const override { return parser_error_; }
-  const QuicString& error_detail() const override {
+  const std::string& error_detail() const override {
     return parser_error_detail_;
   }
 
@@ -53,7 +53,7 @@
   virtual void AdvanceHandshake() = 0;
 
   virtual void CloseConnection(QuicErrorCode error,
-                               const QuicString& reason_phrase) = 0;
+                               const std::string& reason_phrase) = 0;
 
   // Creates an SSL_CTX and configures it with the options that are appropriate
   // for both client and server. The caller is responsible for ownership of the
@@ -135,7 +135,7 @@
   QuicSession* session_;
 
   QuicErrorCode parser_error_ = QUIC_NO_ERROR;
-  QuicString parser_error_detail_;
+  std::string parser_error_detail_;
 
   bssl::UniquePtr<SSL> ssl_;
 };