Switch to use Utf8.isValidUtf8() on the entire range we copy instead of first validating a subrange and then copying it to a dedicated byte[].

This close to an effective no-op today, but in future changes we should be able to be more efficient when we are validating an entire byte[] instead of only a subrange of one.

PiperOrigin-RevId: 866457946

Author: protobuf-github-bot

java/core/src/main/java/com/google/protobuf/ByteString.java

@@ -409,10 +409,11 @@ static ByteString copyFrom(byte[] bytes, int offset, int size, boolean requireUt
       return EMPTY;
     }
     checkRange(offset, offset + size, bytes.length);
-    if (requireUtf8 && !Utf8.isValidUtf8(bytes, offset, offset + size)) {
+    byte[] copy = byteArrayCopier.copyFrom(bytes, offset, size);
+    if (requireUtf8 && !Utf8.isValidUtf8(copy)) {
       throw InvalidProtocolBufferException.invalidUtf8();
     }
-    return new LiteralByteString(byteArrayCopier.copyFrom(bytes, offset, size));
+    return new LiteralByteString(copy);
   }
 
   /**