Stop marking explicit visibility for members that are effectively private

PiperOrigin-RevId: 893047126

Author: blickly

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

@@ -60,7 +60,7 @@ private static enum Color {
       this.controlCharacter = controlCharacter;
     }
 
-    public String getControlCharacter() {
+    String getControlCharacter() {
       return controlCharacter;
     }
   }

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

@@ -1182,7 +1182,7 @@ private static enum IdentifierBehaviour {
     ES6_CLASS_INVOCATION,
     ES6_CLASS_NAMESPACE;
 
-    public static IdentifierBehaviour select(Node target) {
+    static IdentifierBehaviour select(Node target) {
       JSType type = target.getJSType();
       if (type == null || !type.isFunctionType()) {
         // If we aren't sure what we're dealing with be more strict.

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

@@ -311,7 +311,7 @@ private class RenameInternalsToExternalsCallback extends AbstractPostOrderCallba
     private final Map<String, String> exports;
     private final Node namespaceBlock;
 
-    public RenameInternalsToExternalsCallback(
+    RenameInternalsToExternalsCallback(
         String namespaceName, Map<String, String> exports, Node namespaceBlock) {
       this.namespaceName = namespaceName;
       this.exports = exports;

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

@@ -97,8 +97,7 @@ private class RemovableAssignment {
      * @param assignNode The parent ASSIGN node
      * @param traversal Access to further levels, assumed to start at 1
      */
-    public RemovableAssignment(Node nameNode, Node assignNode,
-        NodeTraversal traversal) {
+    RemovableAssignment(Node nameNode, Node assignNode, NodeTraversal traversal) {
       this.node = nameNode;
       this.parent = assignNode;
 
@@ -111,10 +110,8 @@ public RemovableAssignment(Node nameNode, Node assignNode,
       lastAncestor = ancestor.getParent();
     }
 
-    /**
-     * Remove this node.
-     */
-    public void remove() {
+    /** Remove this node. */
+    void remove() {
       Node rhs = node.getNext();
       Node last = parent;
       for (Node ancestor : assignAncestors) {

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

@@ -308,11 +308,11 @@ private static final class ScriptDescription {
     // For recognizing top level names.
     Node rootNode;
 
-    public void addChildScript(ScriptDescription childScript) {
+    void addChildScript(ScriptDescription childScript) {
       childScripts.addLast(childScript);
     }
 
-    public ScriptDescription removeFirstChildScript() {
+    ScriptDescription removeFirstChildScript() {
       return childScripts.removeFirst();
     }
 

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

@@ -644,7 +644,7 @@ void setCrossingVariables(BitSet[] interferenceBitSet) {
     }
 
     /** Returns whether any LiveRangeChecker would be interested in the node. */
-    public static boolean shouldVisit(Node n) {
+    static boolean shouldVisit(Node n) {
       return (n.isName() || (n.hasChildren() && n.getFirstChild().isName()));
     }
 

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

@@ -60,10 +60,10 @@ private abstract static class MappedCodePrinter extends CodeConsumer {
     private final boolean createSrcMap;
     private final SourceMap.DetailLevel sourceMapDetailLevel;
     private final @Nullable LicenseTracker licenseTracker;
-    protected final StringBuilder code = new StringBuilder(1024);
-    protected final int lineLengthThreshold;
-    protected int lineLength = 0;
-    protected int lineIndex = 0;
+    final StringBuilder code = new StringBuilder(1024);
+    final int lineLengthThreshold;
+    int lineLength = 0;
+    int lineIndex = 0;
 
     MappedCodePrinter(
         int lineLengthThreshold,
@@ -207,7 +207,7 @@ private static FilePosition convertPositionAfterLineCut(
       }
     }
 
-    public String getCode() {
+    String getCode() {
       return code.toString();
     }
 
@@ -216,11 +216,11 @@ char getLastChar() {
       return (code.length() > 0) ? code.charAt(code.length() - 1) : '\0';
     }
 
-    protected final int getCurrentCharIndex() {
+    final int getCurrentCharIndex() {
       return lineLength;
     }
 
-    protected final int getCurrentLineIndex() {
+    final int getCurrentLineIndex() {
       return lineIndex;
     }
 

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

@@ -300,9 +300,9 @@ public SourceFile loadSource(String filename) {
    * given originalPath+sourceMapPath, what sourceFile did we find?
    */
   private static class ResolvedSourceMap {
-    public String originalPath;
-    public String sourceMapPath;
-    public String relativePath;
+    String originalPath;
+    String sourceMapPath;
+    String relativePath;
   }
 
   /**
@@ -2806,11 +2806,11 @@ private abstract static class SeenSetLicenseTracker implements LicenseTracker {
     private final Set<String> currentlySeenLicenses = new LinkedHashSet<>();
     private String lastSeenFile = "";
 
-    public SeenSetLicenseTracker(AbstractCompiler compiler) {
+    SeenSetLicenseTracker(AbstractCompiler compiler) {
       this.compiler = compiler;
     }
 
-    protected boolean shouldUseLicenseInfo(Node node) {
+    boolean shouldUseLicenseInfo(Node node) {
       return !node.isRoot() && !node.isScript();
     }
 

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

@@ -640,7 +640,7 @@ private final class JsAst {
     private @Nullable Node root;
     private FeatureSet features;
 
-    public Node getAstRoot(AbstractCompiler compiler) {
+    Node getAstRoot(AbstractCompiler compiler) {
       if (this.isParsed()) {
         return this.root;
       }
@@ -660,7 +660,7 @@ public Node getAstRoot(AbstractCompiler compiler) {
       return this.root;
     }
 
-    public void clearAst() {
+    void clearAst() {
       root = null;
       // While we're at it, clear out any saved text in the source file on
       // the assumption that if we're dumping the parse tree, then we probably

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

@@ -348,7 +348,7 @@ public boolean shouldTraverse(NodeTraversal t, Node n, Node parent) {
       return true;
     }
 
-    public void visitScript(NodeTraversal t, Node script) {
+    void visitScript(NodeTraversal t, Node script) {
       checkState(script.isScript());
       JSChunk chunk = t.getChunk();
       ImmutableList<JSChunk> chunkDependencies = chunk.getDependencies();