/*
 * Copyright 2014 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;

import com.google.common.collect.ImmutableCollection;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableSet;
import com.google.errorprone.annotations.Immutable;
import com.google.javascript.rhino.Node;
import com.google.javascript.rhino.NominalTypeBuilder;
import com.google.javascript.rhino.jstype.FunctionType;

/**
 * Coding convention used by the Chrome team to compile Chrome's JS.
 */
@Immutable
public final class ChromeCodingConvention extends CodingConventions.Proxy {
  private final ImmutableSet<String> indirectlyDeclaredProperties;

  public ChromeCodingConvention() {
    this(CodingConventions.getDefault());
  }

  public ChromeCodingConvention(CodingConvention wrapped) {
    super(wrapped);
    indirectlyDeclaredProperties = new ImmutableSet.Builder<String>()
        .add("instance_", "getInstance")
        .addAll(wrapped.getIndirectlyDeclaredProperties())
        .build();
  }

  @Override
  public String getSingletonGetterClassName(Node callNode) {
    Node callArg = callNode.getFirstChild();
    if (!callArg.matchesQualifiedName("cr.addSingletonGetter") || !callNode.hasTwoChildren()) {
      return super.getSingletonGetterClassName(callNode);
    }
    return callArg.getNext().getQualifiedName();
  }

  @Override
  public void applySingletonGetter(NominalTypeBuilder classType, FunctionType getterType) {
    Node defSite = classType.constructor().getSource();
    classType.declareConstructorProperty("getInstance", getterType, defSite);
    classType.declareConstructorProperty("instance_", classType.instance(), defSite);
  }

  @Override
  public ImmutableCollection<String> getIndirectlyDeclaredProperties() {
    return indirectlyDeclaredProperties;
  }

  @Override
  public ImmutableCollection<AssertionFunctionSpec> getAssertionFunctions() {
    return ImmutableList.of(
        AssertionFunctionSpec.forTruthy().setFunctionName("assert").build(),
        AssertionFunctionSpec.forMatchesReturn().setFunctionName("cr.ui.decorate").build());
  }

  @Override
  public boolean isFunctionCallThatAlwaysThrows(Node n) {
    return CodingConventions.defaultIsFunctionCallThatAlwaysThrows(
        n, "assertNotReached");
  }
}

@@ -703,56 +703,8 @@
 on;%0A
-import com.google.common.collect.ImmutableList;%0A
 impo
@@ -2356,350 +2356,147 @@
 lic 
-ImmutableCollection%3CAssertionFunctionSpec%3E getAssertionFunctions() %7B%0A    return ImmutableList.of(%0A        AssertionFunctionSpec.forTruthy().setFunctionName(%22assert%22).build(),%0A        AssertionFunctionSpec.forMatchesReturn().setFunctionName(%22cr.ui.decorate%22).build());%0A  %7D%0A%0A  @Override%0A  public boolean isFunctionCallThatAlwaysThrows(Node n) %7B
+boolean isFunctionCallThatAlwaysThrows(Node n) %7B%0A    // TODO(dbeam): does @closurePrimitive %7Basserts.fail%7D make it possible to remove this?
 %0A   
@@ -2559,25 +2559,16 @@
 sThrows(
-%0A        
 n, %22asse

Remove assertion functions from ChromeCodingConvention