commit 473107b398f08a427e850042b9bb0b156ae8ac03
author Nitzan Gur-Furman <nitzan@google.com> Fri May 17 20:23:17 2024 +0000
committer Nitzan Gur-Furman <nitzan@google.com> Tue May 21 07:51:08 2024 +0000
tree fe928826d1cf4bffd984da9e8b3810e1096e5b01
parent 524205fd88116538b84fd87b4eee38d079cccef1

Make LabelIdProtoConverter safe

Release-Notes: skip
Google-Bug-Id: b/335372403
Change-Id: Ida4aba3c0a6dcf8ae9d506bc4b22175d7dc519cf
Forward-Compatible: checked

java/com/google/gerrit/entities/LabelId.java

diff --git a/java/com/google/gerrit/entities/LabelId.java b/java/com/google/gerrit/entities/LabelId.java
index 2426818..e3b3024 100644
--- a/java/com/google/gerrit/entities/LabelId.java
+++ b/java/com/google/gerrit/entities/LabelId.java
@@ -15,8 +15,10 @@
 package com.google.gerrit.entities;
 
 import com.google.auto.value.AutoValue;
+import com.google.gerrit.common.ConvertibleToProto;
 
 @AutoValue
+@ConvertibleToProto
 public abstract class LabelId {
   public static final String LEGACY_SUBMIT_NAME = "SUBM";
   public static final String CODE_REVIEW = "Code-Review";

java/com/google/gerrit/entities/converter/LabelIdProtoConverter.java

diff --git a/java/com/google/gerrit/entities/converter/LabelIdProtoConverter.java b/java/com/google/gerrit/entities/converter/LabelIdProtoConverter.java
index a1894ac..e6e1be7f 100644
--- a/java/com/google/gerrit/entities/converter/LabelIdProtoConverter.java
+++ b/java/com/google/gerrit/entities/converter/LabelIdProtoConverter.java
@@ -20,7 +20,7 @@
 import com.google.protobuf.Parser;
 
 @Immutable
-public enum LabelIdProtoConverter implements ProtoConverter<Entities.LabelId, LabelId> {
+public enum LabelIdProtoConverter implements SafeProtoConverter<Entities.LabelId, LabelId> {
   INSTANCE;
 
   @Override
@@ -37,4 +37,14 @@
   public Parser<Entities.LabelId> getParser() {
     return Entities.LabelId.parser();
   }
+
+  @Override
+  public Class<Entities.LabelId> getProtoClass() {
+    return Entities.LabelId.class;
+  }
+
+  @Override
+  public Class<LabelId> getEntityClass() {
+    return LabelId.class;
+  }
 }