Change getBuiltInDescriptor to getBuiltInDataDescriptor for clarity.

mozilla · Dec 3, 2024 · ad7d0a6 · ad7d0a6
1 parent c14ec1c
commit ad7d0a6
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/rhino/src/main/java/org/mozilla/javascript/IdScriptableObject.java b/rhino/src/main/java/org/mozilla/javascript/IdScriptableObject.java
@@ -922,15 +922,15 @@ protected ScriptableObject getOwnPropertyDescriptor(Context cx, Object id) {
         ScriptableObject desc = super.getOwnPropertyDescriptor(cx, id);
         if (desc == null) {
             if (id instanceof String) {
-                return getBuiltInDescriptor((String) id);
+                return getBuiltInDataDescriptor((String) id);
             }
 
             if (ScriptRuntime.isSymbol(id)) {
                 if (id instanceof SymbolKey) {
-                    return getBuiltInDescriptor((SymbolKey) id);
+                    return getBuiltInDataDescriptor((SymbolKey) id);
                 }
 
-                return getBuiltInDescriptor(((NativeSymbol) id).getKey());
+                return getBuiltInDataDescriptor(((NativeSymbol) id).getKey());
             }
         }
         return desc;
@@ -954,7 +954,7 @@ private Slot queryOrFakeSlot(Context cx, Object id) {
         return slot;
     }
 
-    private ScriptableObject getBuiltInDescriptor(String name) {
+    private ScriptableObject getBuiltInDataDescriptor(String name) {
         Scriptable scope = getParentScope();
         if (scope == null) {
             scope = this;
@@ -990,7 +990,7 @@ private Slot getBuiltInSlot(String name) {
         return null;
     }
 
-    private ScriptableObject getBuiltInDescriptor(Symbol key) {
+    private ScriptableObject getBuiltInDataDescriptor(Symbol key) {
         Scriptable scope = getParentScope();
         if (scope == null) {
             scope = this;