Strip comments and whitespace from javascript resources

third_party/closure_compiler/compile2.py

Index: third_party/closure_compiler/compile2.py
diff --git a/third_party/closure_compiler/compile2.py b/third_party/closure_compiler/compile2.py
index df844d129109dc082700ee9e89fe45d6dc534117..6a5008d1dc3df9c1925bd33a567f5ec83b228058 100755
--- a/third_party/closure_compiler/compile2.py
+++ b/third_party/closure_compiler/compile2.py
@@ -224,7 +224,7 @@ class Checker(object):
     self._log_debug("Dependencies: %s" % deps)
     self._log_debug("Target: %s" % self._target)
 
-    js_args = deps + [self._target] if self._target else []
+    js_args = deps + ([self._target] if self._target else [])
 
     if not custom_sources:
       # TODO(dbeam): compiler.jar automatically detects "@externs" in a --js arg
@@ -270,7 +270,8 @@ class Checker(object):
     errors = stderr.strip().split("\n\n")
     maybe_summary = errors.pop()
 
-    if re.search(".*error.*warning.*typed", maybe_summary):
+    summary = re.search("(?P<error_count>\d+).*error.*warning", maybe_summary)
+    if summary:
       self._log_debug("Summary: %s" % maybe_summary)
     else:
       # Not a summary. Running the jar failed. Bail.
@@ -278,7 +279,7 @@ class Checker(object):
       self._nuke_temp_files()
       sys.exit(1)
 
-    if errors and out_file:
+    if summary.group('error_count') != "0" and out_file:
       if os.path.exists(out_file):
         os.remove(out_file)
       if os.path.exists(self._MAP_FILE_FORMAT % out_file):