/*
 * Copyright 2015 The Closure Compiler Authors.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package com.google.javascript.jscomp.lint;

import static com.google.javascript.rhino.jstype.JSTypeNative.ARRAY_TYPE;

import com.google.common.base.Preconditions;
import com.google.javascript.jscomp.AbstractCompiler;
import com.google.javascript.jscomp.DiagnosticType;
import com.google.javascript.jscomp.HotSwapCompilerPass;
import com.google.javascript.jscomp.NodeTraversal;
import com.google.javascript.jscomp.NodeUtil;
import com.google.javascript.rhino.Node;
import com.google.javascript.rhino.jstype.JSType;
import com.google.javascript.rhino.jstype.JSTypeRegistry;

/**
 * Checks when the pattern for (x in arr) { ... } where arr is an array,
 * or an union type containing an array
 *
 */
public final class CheckForInOverArray
    extends NodeTraversal.AbstractPostOrderCallback
    implements HotSwapCompilerPass {
  final AbstractCompiler compiler;
  final JSTypeRegistry typeRegistry;

  public static final DiagnosticType FOR_IN_OVER_ARRAY =
      DiagnosticType.warning(
          "JSC_FOR_IN_OVER_ARRAY",
          "For..in over array is discouraged.");

  public CheckForInOverArray(AbstractCompiler compiler) {
    this.compiler = compiler;
    this.typeRegistry = compiler.getTypeRegistry();
  }

  public boolean isForInOverArray(Node n) {
    if (NodeUtil.isForIn(n)) {
      Preconditions.checkState(n.getChildCount() == 3);
      // get the second child, which represents
      // B in construct "for (A in B) { C }"
      Node child = n.getFirstChild().getNext();
      JSType type = child.getJSType();
      if (type != null && containsArray(type)) {
        return true;
      }
    }
    return false;
  }

  private boolean containsArray(JSType type) {
    if (type.isArrayType()) {
      return true;
    }
    if (type.isUnionType()) {
      JSType arrayType = typeRegistry.getNativeType(ARRAY_TYPE);
      if (arrayType.isSubtype(type)) {
        return true;
      }
    }
    return false;
  }

  @Override
  public void visit(NodeTraversal t, Node n, Node parent) {
    if (isForInOverArray(n)) {
      compiler.report(t.makeError(parent, FOR_IN_OVER_ARRAY));
    }
  }

  @Override
  public void process(Node externs, Node root) {
    NodeTraversal.traverseEs6(compiler, root, this);
  }

  @Override
  public void hotSwapScript(Node scriptRoot, Node originalRoot) {
    NodeTraversal.traverseEs6(compiler, originalRoot, this);
  }
}

@@ -2948,24 +2948,22 @@
 mpiler, 
-original
+script
 Root, th

Fix hotswap ------------- Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=101984717