Delete unused methods in ObjectType and PropertyMap

PiperOrigin-RevId: 556977782

Author: lauraharker

src/com/google/javascript/rhino/jstype/NoObjectType.java

@@ -125,11 +125,6 @@ final boolean defineProperty(String propertyName, JSType type,
     return true;
   }
 
-  @Override
-  public final boolean removeProperty(String name) {
-    return false;
-  }
-
   @Override
   public final void setPropertyJSDocInfo(String propertyName, JSDocInfo info) {
     // Do nothing, specific properties do not have JSDocInfo.

src/com/google/javascript/rhino/jstype/ObjectType.java

@@ -411,17 +411,6 @@ public final boolean defineInferredProperty(String propertyName,
   abstract boolean defineProperty(String propertyName, JSType type,
       boolean inferred, Node propertyNode);
 
-  /**
-   * Removes the declared or inferred property from this ObjectType.
-   *
-   * @param propertyName the property's name
-   * @return true if the property was removed successfully. False if the
-   *         property did not exist, or could not be removed.
-   */
-  public boolean removeProperty(String propertyName) {
-    return false;
-  }
-
   /**
    * Gets the node corresponding to the definition of the specified property. This could be the node
    * corresponding to declaration of the property or the node corresponding to the first reference

src/com/google/javascript/rhino/jstype/PropertyMap.java

@@ -129,37 +129,6 @@ private Iterable<ObjectType> getSecondaryParentObjects() {
     return parentSource.getCtorExtendedInterfaces();
   }
 
-  @Nullable
-  OwnedProperty findTopMost(String name) {
-    // Check primary parents which always has precendence over secondary.
-    OwnedProperty found = null;
-    for (PropertyMap map = this; map != null; map = map.getPrimaryParent()) {
-      Property prop = map.properties.get(name);
-      if (prop != null) {
-        found = new OwnedProperty(map.parentSource, prop);
-      }
-    }
-    if (found != null) {
-      return found;
-    }
-
-    // Recurse into secondary parents. Note that there is no single top most definition with
-    // interfaces so we simple return the first result.
-    for (PropertyMap map = this; map != null; map = map.getPrimaryParent()) {
-      for (ObjectType o : map.getSecondaryParentObjects()) {
-        PropertyMap parent = o.getPropertyMap();
-        if (parent != null) {
-          OwnedProperty e = parent.findTopMost(name);
-          if (e != null) {
-            return e;
-          }
-        }
-      }
-    }
-
-    return null;
-  }
-
   @Nullable
   OwnedProperty findClosest(String name) {
     // Check primary parents which always has precendence over secondary.
@@ -256,15 +225,6 @@ private void collectAllAncestors(Set<PropertyMap> ancestors) {
     }
   }
 
-  boolean removeProperty(String name) {
-    if (properties.remove(name) == null) {
-      return false;
-    }
-
-    this.incrementCachedKeySetCounter();
-    return true;
-  }
-
   void putProperty(String name, Property newProp) {
     Property oldProp = properties.get(name);
 

src/com/google/javascript/rhino/jstype/PrototypeObjectType.java

@@ -207,11 +207,6 @@ boolean defineProperty(String name, JSType type, boolean inferred,
     return true;
   }
 
-  @Override
-  public boolean removeProperty(String name) {
-    return properties.removeProperty(name);
-  }
-
   @Override
   public void setPropertyJSDocInfo(String propertyName, JSDocInfo info) {
     if (info != null) {

src/com/google/javascript/rhino/jstype/ProxyObjectType.java

@@ -305,11 +305,6 @@ boolean defineProperty(String propertyName, JSType type, boolean inferred, Node
         || referencedObjType.defineProperty(propertyName, type, inferred, propertyNode);
   }
 
-  @Override
-  public final boolean removeProperty(String name) {
-    return referencedObjType == null ? false : referencedObjType.removeProperty(name);
-  }
-
   @Override
   protected JSType findPropertyTypeWithoutConsideringTemplateTypes(String propertyName) {
     return referencedType.findPropertyType(propertyName);