message: Remove unnecessary SecretKeyAccess.requireAccess calls.
  Either these, or the explicit nullpointer checks are unneeded. Which of the two depends a bit on how one thinks about the nullable annotation. I would suggest we go with removing the requireAccess call.
diff: @@ -29,7 +29,6 @@ public final class TinkProtoKeysetFormat {
  if (access == null) {
  throw new NullPointerException("SecretKeyAccess cannot be null");
  }
  - SecretKeyAccess.requireAccess(access);
  try {
  return CleartextKeysetHandle.read(BinaryKeysetReader.withBytes(serializedKeyset));
  } catch (IOException e) {
  @@ -43,7 +42,6 @@ public final class TinkProtoKeysetFormat {
  if (access == null) {
  throw new NullPointerException("SecretKeyAccess cannot be null");
  }
  - SecretKeyAccess.requireAccess(access);
  try {
  ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
  CleartextKeysetHandle.write(keysetHandle, BinaryKeysetWriter.withOutputStream(outputStream));
  