diff --git a/services/core/java/com/android/server/hdmi/HdmiCecConfig.java b/services/core/java/com/android/server/hdmi/HdmiCecConfig.java index a4564a9e0bbee..ca78f35b9f1f4 100644 --- a/services/core/java/com/android/server/hdmi/HdmiCecConfig.java +++ b/services/core/java/com/android/server/hdmi/HdmiCecConfig.java @@ -283,9 +283,7 @@ public class HdmiCecConfig { } private int getIntValue(@NonNull Value value) { - return value.getHexValue() != null - ? Integer.decode(value.getHexValue()) - : value.getIntValue(); + return Integer.decode(value.getIntValue()); } /** diff --git a/services/core/xsd/cec-config/cec-config.xsd b/services/core/xsd/cec-config/cec-config.xsd index 2129044bebcb9..b59c93cce332f 100644 --- a/services/core/xsd/cec-config/cec-config.xsd +++ b/services/core/xsd/cec-config/cec-config.xsd @@ -24,7 +24,6 @@ - - + diff --git a/services/core/xsd/cec-config/schema/current.txt b/services/core/xsd/cec-config/schema/current.txt index 896ddfcdb40de..75872d4fb8a7e 100644 --- a/services/core/xsd/cec-config/schema/current.txt +++ b/services/core/xsd/cec-config/schema/current.txt @@ -22,11 +22,9 @@ package com.android.server.hdmi.cec.config { public class Value { ctor public Value(); - method public String getHexValue(); - method public int getIntValue(); + method public String getIntValue(); method public String getStringValue(); - method public void setHexValue(String); - method public void setIntValue(int); + method public void setIntValue(String); method public void setStringValue(String); } diff --git a/services/tests/servicestests/src/com/android/server/hdmi/HdmiCecConfigTest.java b/services/tests/servicestests/src/com/android/server/hdmi/HdmiCecConfigTest.java index 046bba3e0fc5b..a27f7331e4073 100644 --- a/services/tests/servicestests/src/com/android/server/hdmi/HdmiCecConfigTest.java +++ b/services/tests/servicestests/src/com/android/server/hdmi/HdmiCecConfigTest.java @@ -410,10 +410,10 @@ public final class HdmiCecConfigTest { + " value-type=\"int\"" + " user-configurable=\"true\">" + " " - + " " - + " " + + " " + + " " + " " - + " " + + " " + " " + "", null); assertThat(hdmiCecConfig.getAllowedIntValues( @@ -556,10 +556,10 @@ public final class HdmiCecConfigTest { + " value-type=\"int\"" + " user-configurable=\"true\">" + " " - + " " - + " " + + " " + + " " + " " - + " " + + " " + " " + "", null); assertThat(hdmiCecConfig.getDefaultIntValue( @@ -741,10 +741,10 @@ public final class HdmiCecConfigTest { + " value-type=\"int\"" + " user-configurable=\"true\">" + " " - + " " - + " " + + " " + + " " + " " - + " " + + " " + " " + "", null); assertThat(hdmiCecConfig.getIntValue( @@ -988,10 +988,10 @@ public final class HdmiCecConfigTest { + " value-type=\"int\"" + " user-configurable=\"true\">" + " " - + " " - + " " + + " " + + " " + " " - + " " + + " " + " " + "", null); hdmiCecConfig.setIntValue(HdmiControlManager.CEC_SETTING_NAME_HDMI_CEC_ENABLED,