Fix AggressiveInlineAliases bug inlining across chunks

Most of this CL is just tracking chunks of GlobalNamespace Refs. The only behavioral changes are around line 1764 of GlobalNamespace.java & line 894 of InlineAndCollapseProperties.java.

PiperOrigin-RevId: 718879701

Author: lauraharker

src/com/google/javascript/jscomp/DestructuringGlobalNameExtractor.java

@@ -20,6 +20,7 @@
 
 import com.google.javascript.jscomp.GlobalNamespace.AstChange;
 import com.google.javascript.jscomp.GlobalNamespace.Ref;
+import com.google.javascript.jscomp.GlobalNamespace.RefBasedAstChange;
 import com.google.javascript.rhino.IR;
 import com.google.javascript.rhino.Node;
 import com.google.javascript.rhino.Token;
@@ -76,7 +77,7 @@ static void reassignDestructringLvalue(
             ? stringKey.getOnlyChild().getFirstChild()
             : stringKey.getOnlyChild();
     if (newNodes != null) {
-      newNodes.add(new AstChange(ref.scope, newName));
+      newNodes.add(new RefBasedAstChange(ref, newName));
     }
     Node rvalue = makeNewRvalueForDestructuringKey(stringKey, newName, newNodes, ref);
 
@@ -102,7 +103,7 @@ static void reassignDestructringLvalue(
       } else {
         newRvalue = originalRvalue.cloneTree();
         if (newNodes != null) {
-          newNodes.add(new AstChange(ref.scope, newRvalue));
+          newNodes.add(new RefBasedAstChange(ref, newRvalue));
         }
       }
       addAfter(lvalueToReassign, newPattern, newRvalue);
@@ -182,7 +183,7 @@ private static Node makeNewRvalueForDestructuringKey(
       // references to it. This ignores getters/setters.
       Node rvalueForSheq = rvalue.cloneTree();
       if (newNodes != null) {
-        newNodes.add(new AstChange(ref.scope, rvalueForSheq));
+        newNodes.add(new RefBasedAstChange(ref, rvalueForSheq));
       }
       // `void 0 === rvalue ? defaultValue : rvalue`
       rvalue =