Modify ReplaceCssNames to store the entire class name (instead of splitting by part)

PiperOrigin-RevId: 551876747

Author: nanaze

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

@@ -180,7 +180,7 @@ public void process(Node externs, Node root) {
         cssNamesBySymbol.put(getCssNameInstance.getCssClosureClassesMemberName(), cssClassName);
         classesObjectsQualifiedNames.add(getCssNameInstance.getCssClosureClassesQualifiedName());
       } else {
-        updateCssNamesCount(cssClassName);
+        cssNameCollector.add(cssClassName);
       }
       getCssNameInstance.replaceWithExpression();
     }
@@ -316,24 +316,6 @@ private class TraversalState {
     }
   }
 
-  /**
-   * Update the count of CSS class names.
-   *
-   * <p>We maintain a count of references to CSS classes, so that unused classes can be flagged.
-   *
-   * <p>Unused class matching is always done by-part: if a CSS class definition or reference
-   * contains hyphens, each hyphenated section is checked separately. For example, if the CSS class
-   * contains .myComponent-highlight, it can be satisfied by separate calls to
-   * goog.getCssName('myComponent') and goog.getCssName('highlight'). A single call to
-   * goog.getCssName('myComponent-highlight') will also work.
-   */
-  private void updateCssNamesCount(String cssClassName) {
-    String[] parts = cssClassName.split("-", -1);
-    for (String element : parts) {
-      cssNameCollector.add(element);
-    }
-  }
-
   private boolean isNotInCssClosureFile(Node node) {
     String sourceFileName = node.getSourceFileName();
     return sourceFileName == null || !sourceFileName.endsWith(".css.closure.js");
@@ -585,7 +567,7 @@ public void visit(NodeTraversal t, Node n, Node parent) {
         return;
       }
       String className = cssNamesBySymbol.get(classQualifiedName);
-      updateCssNamesCount(className);
+      cssNameCollector.add(className);
     }
   }
 }

test/com/google/javascript/jscomp/ReplaceCssNamesTest.java

@@ -180,7 +180,7 @@ public void testDoNotUseReplacementMap() {
         "setClass(goog.getCssName('active-buttonbar'))", //
         "setClass('active-buttonbar')");
     assertThat(cssNames)
-        .containsExactly("goog", "footer", "active", "colorswatch", "disabled", "buttonbar");
+        .containsExactly("goog-footer-active", "goog-colorswatch-disabled", "active-buttonbar");
   }
 
   @Test
@@ -228,7 +228,7 @@ private void oneArgWithCompositeClassNames() {
         "setClass(goog.getCssName('active-buttonbar'))", //
         "setClass('a-b')");
     assertThat(cssNames)
-        .containsExactly("goog", "footer", "active", "colorswatch", "disabled", "buttonbar");
+        .containsExactly("goog-footer-active", "goog-colorswatch-disabled", "active-buttonbar");
   }
 
   @Test