From a28045f8a5ad43dce87197b36b0e21112328dc5c Mon Sep 17 00:00:00 2001
From: dorir <dorir@google.com>
Date: Thu, 2 Mar 2017 16:49:04 -0800
Subject: [PATCH] Use Immutable collections for static final CONSTANT_CASE
 fields.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=149063770
---
 .../javascript/jscomp/AbstractMessageFormatter.java      | 8 ++------
 .../jscomp/GatherSideEffectSubexpressionsCallback.java   | 6 ++----
 src/com/google/javascript/jscomp/NameAnalyzer.java       | 3 +--
 .../google/javascript/jscomp/VariableReferenceCheck.java | 3 ++-
 .../javascript/jscomp/DefaultNameGeneratorTest.java      | 9 ++-------
 5 files changed, 9 insertions(+), 20 deletions(-)

diff --git a/src/com/google/javascript/jscomp/AbstractMessageFormatter.java b/src/com/google/javascript/jscomp/AbstractMessageFormatter.java
index 15fec0ff4a6..38810233e59 100644
--- a/src/com/google/javascript/jscomp/AbstractMessageFormatter.java
+++ b/src/com/google/javascript/jscomp/AbstractMessageFormatter.java
@@ -17,7 +17,6 @@
 
 import com.google.common.collect.ImmutableSet;
 
-import java.util.Set;
 
 /**
  * Abstract message formatter providing default behavior for implementations
@@ -43,11 +42,8 @@ protected final SourceExcerptProvider getSource() {
     return source;
   }
 
-  private static final Set<String> SUPPORTED_COLOR_TERMINALS =
-      ImmutableSet.of("xterm",
-                      "xterm-color",
-                      "xterm-256color",
-                      "screen-bce");
+  private static final ImmutableSet<String> SUPPORTED_COLOR_TERMINALS =
+      ImmutableSet.of("xterm", "xterm-color", "xterm-256color", "screen-bce");
 
   static boolean termSupportsColor(String term) {
     return SUPPORTED_COLOR_TERMINALS.contains(term);
diff --git a/src/com/google/javascript/jscomp/GatherSideEffectSubexpressionsCallback.java b/src/com/google/javascript/jscomp/GatherSideEffectSubexpressionsCallback.java
index a0d539c8651..1821c9f733d 100644
--- a/src/com/google/javascript/jscomp/GatherSideEffectSubexpressionsCallback.java
+++ b/src/com/google/javascript/jscomp/GatherSideEffectSubexpressionsCallback.java
@@ -23,10 +23,8 @@
 import com.google.javascript.rhino.IR;
 import com.google.javascript.rhino.Node;
 import com.google.javascript.rhino.Token;
-
 import java.util.ArrayList;
 import java.util.List;
-import java.util.Set;
 
 /**
  * Callback that gathers subexpressions that may have side effects
@@ -186,8 +184,8 @@ private Node simplifyShortCircuitBranch(Node node) {
     }
   }
 
-  private static final Set<Token> FORBIDDEN_TYPES = ImmutableSet.of(
-      Token.BLOCK, Token.SCRIPT, Token.VAR, Token.EXPR_RESULT, Token.RETURN);
+  private static final ImmutableSet<Token> FORBIDDEN_TYPES =
+      ImmutableSet.of(Token.BLOCK, Token.SCRIPT, Token.VAR, Token.EXPR_RESULT, Token.RETURN);
   private final AbstractCompiler compiler;
   private final SideEffectAccumulator accumulator;
 
diff --git a/src/com/google/javascript/jscomp/NameAnalyzer.java b/src/com/google/javascript/jscomp/NameAnalyzer.java
index cf0cf5bf012..9811dad8020 100644
--- a/src/com/google/javascript/jscomp/NameAnalyzer.java
+++ b/src/com/google/javascript/jscomp/NameAnalyzer.java
@@ -121,8 +121,7 @@ final class NameAnalyzer implements CompilerPass {
   private static final String FUNCTION = "Function";
 
   /** All of these refer to global scope. These can be moved to config */
-  static final Set<String> DEFAULT_GLOBAL_NAMES = ImmutableSet.of(
-      "window", "goog.global");
+  static final ImmutableSet<String> DEFAULT_GLOBAL_NAMES = ImmutableSet.of("window", "goog.global");
 
   /** Whether to remove unreferenced variables in main pass */
   private final boolean removeUnreferenced;
diff --git a/src/com/google/javascript/jscomp/VariableReferenceCheck.java b/src/com/google/javascript/jscomp/VariableReferenceCheck.java
index 5be9a166504..3b08c84991b 100644
--- a/src/com/google/javascript/jscomp/VariableReferenceCheck.java
+++ b/src/com/google/javascript/jscomp/VariableReferenceCheck.java
@@ -16,6 +16,7 @@
 
 package com.google.javascript.jscomp;
 
+import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Sets;
 import com.google.javascript.jscomp.NodeTraversal.AbstractShallowCallback;
 import com.google.javascript.jscomp.ReferenceCollectingCallback.BasicBlock;
@@ -79,7 +80,7 @@ class VariableReferenceCheck implements HotSwapCompilerPass {
 
   // These types do not permit a block-scoped declaration inside them without an explicit block.
   // e.g. if (b) let x;
-  private static final Set<Token> BLOCKLESS_DECLARATION_FORBIDDEN_STATEMENTS =
+  private static final ImmutableSet<Token> BLOCKLESS_DECLARATION_FORBIDDEN_STATEMENTS =
       Sets.immutableEnumSet(Token.IF, Token.FOR, Token.FOR_IN, Token.FOR_OF, Token.WHILE);
 
   public VariableReferenceCheck(AbstractCompiler compiler) {
diff --git a/test/com/google/javascript/jscomp/DefaultNameGeneratorTest.java b/test/com/google/javascript/jscomp/DefaultNameGeneratorTest.java
index a90e55e0dfa..bbc5fe94796 100644
--- a/test/com/google/javascript/jscomp/DefaultNameGeneratorTest.java
+++ b/test/com/google/javascript/jscomp/DefaultNameGeneratorTest.java
@@ -19,17 +19,12 @@
 import static com.google.common.truth.Truth.assertThat;
 
 import com.google.common.collect.ImmutableSet;
-
-import junit.framework.TestCase;
-
 import java.util.Collections;
-import java.util.Set;
+import junit.framework.TestCase;
 
 public final class DefaultNameGeneratorTest extends TestCase {
 
-  private static final Set<String> RESERVED_NAMES = ImmutableSet.of(
-      "ba",
-      "xba");
+  private static final ImmutableSet<String> RESERVED_NAMES = ImmutableSet.of("ba", "xba");
 
   private static String[] generate(
       DefaultNameGenerator ng, String prefix, int num) throws Exception {