Save intermediate text encoded buffer copies

Author: franz1981

src/main/java/io/vertx/core/http/impl/WebSocketImplBase.java

@@ -44,6 +44,7 @@
 import javax.net.ssl.SSLPeerUnverifiedException;
 import javax.net.ssl.SSLSession;
 import javax.security.cert.X509Certificate;
+import java.nio.charset.StandardCharsets;
 import java.security.cert.Certificate;
 import java.util.List;
 import java.util.UUID;
@@ -308,7 +309,13 @@ public final S writeBinaryMessage(Buffer data, Handler<AsyncResult<Void>> handle
 
   @Override
   public Future<Void> writeTextMessage(String text) {
-    return writePartialMessage(WebSocketFrameType.TEXT, Buffer.buffer(text), 0);
+    byte[] utf8Bytes = text.getBytes(StandardCharsets.UTF_8);
+    boolean isFinal = utf8Bytes.length <= maxWebSocketFrameSize;
+    if (isFinal) {
+      return writeFrame(new WebSocketFrameImpl(WebSocketFrameType.TEXT, utf8Bytes, true));
+    }
+    // we could save to copy the byte[] if the unsafe heap version of Netty ByteBuf could expose wrapping byte[] directly
+    return writePartialMessage(WebSocketFrameType.TEXT, Buffer.buffer(utf8Bytes), 0);
   }
 
   @Override

src/main/java/io/vertx/core/http/impl/ws/WebSocketFrameImpl.java

@@ -79,6 +79,15 @@ public WebSocketFrameImpl(String textData) {
     this(textData, true);
   }
 
+  /**
+   * Creates a new raw frame from with the specified encoded content.
+   */
+  public WebSocketFrameImpl(WebSocketFrameType type, byte[] utf8TextData, boolean isFinalFrame) {
+    this.type = type;
+    this.isFinalFrame = isFinalFrame;
+    this.binaryData = Unpooled.wrappedBuffer(utf8TextData);
+  }
+
   /**
    * Creates a new text frame from with the specified string.
    */