Rename some ClassObject/Provider-related methods

The terminology "field" is preferred over "key" for the components of a struct or struct-like object.

RELNOTES: None
PiperOrigin-RevId: 180269374

1 files changed, 6 insertions, 6 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SkylarkAspectFactory.java b/src/main/java/com/google/devtools/build/lib/skyframe/SkylarkAspectFactory.java
index 24112a4fae..d66d30bb55 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/SkylarkAspectFactory.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/SkylarkAspectFactory.java
@@ -116,11 +116,11 @@ public class SkylarkAspectFactory implements ConfiguredAspectFactory {
     } else {
       Info struct = (Info) aspectSkylarkObject;
       Location loc = struct.getCreationLoc();
-      for (String key : struct.getKeys()) {
-        if (key.equals("output_groups")) {
-          addOutputGroups(struct.getValue(key), loc, builder);
-        } else if (key.equals("providers")) {
-          Object value = struct.getValue(key);
+      for (String field : struct.getFieldNames()) {
+        if (field.equals("output_groups")) {
+          addOutputGroups(struct.getValue(field), loc, builder);
+        } else if (field.equals("providers")) {
+          Object value = struct.getValue(field);
           Iterable providers =
               SkylarkType.cast(
                   value,
@@ -131,7 +131,7 @@ public class SkylarkAspectFactory implements ConfiguredAspectFactory {
                   EvalUtils.getDataTypeName(value, false));
           addDeclaredProviders(builder, providers);
         } else {
-          builder.addSkylarkTransitiveInfo(key, struct.getValue(key), loc);
+          builder.addSkylarkTransitiveInfo(field, struct.getValue(field), loc);
         }
       }
     }