Use newer Closure deps builder in JS build (#2513)

* Use newer Closure deps builder in JS build

* Update JS compiler build file reference and instructions

Co-authored-by: penmetsaa <penmetsaa@google.com>

javascript/README.md

@@ -32,8 +32,10 @@ If you don't checkout the dependencies next to libphonenumber:
 
 How to compile:
 ===============
-1. Build Closure's compiler.jar:
-  `mvn -DskipTests`
+1. Build the Closure Compiler JAR file by following the directions on the
+   [Closure Compiler README](https://github.com/google/closure-compiler/tree/master/README.md).
+   If this step doesn't work, try updating your local copy of each of the
+   repositories listed above before filing an issue.
 
 2. Compile the demo.js and all its dependencies to one file: `demo-compiled.js`:
   `ant -f javascript/build.xml compile-demo`

javascript/build.xml

@@ -4,7 +4,7 @@
   <property name="closure-compiler.dir"
             value="${basedir}/../../closure-compiler" />
   <property name="closure-compiler.jar"
-            value="${closure-compiler.dir}/target/closure-compiler-1.0-SNAPSHOT.jar" />
+            value="${closure-compiler.dir}/bazel-bin/compiler_unshaded_deploy.jar" />
   <property name="closure-library.dir"
             value="${basedir}/../../closure-library" />
   <property name="closure-linter.dir"
@@ -21,10 +21,10 @@
     <element name="extraflags" optional="yes" />
     <sequential>
       <exec executable="python" failonerror="true" logError="true">
-        <arg value="${closure-library.dir}/closure/bin/calcdeps.py" />
+        <arg value="${closure-library.dir}/closure/bin/build/closurebuilder.py" />
         <arg line='-i "@{inputfile}"' />
         <arg line='--output_file "@{outputfile}"' />
-        <arg line='-p "${closure-library.dir}"' />
+        <arg line='--root "${closure-library.dir}"' />
         <arg line="-o @{outputmode}" />
         <arg line='-c "${closure-compiler.jar}"' />
         <arg line='-f "--output_wrapper=@{outputwrapper}"' />
@@ -98,7 +98,7 @@
     <closure-compile inputfile="i18n/phonenumbers/demo.js"
                      outputfile="i18n/phonenumbers/demo-compiled.js">
       <extraflags>
-        <arg line='-p "i18n/phonenumbers"' />
+        <arg line='--root "i18n/phonenumbers"' />
         <arg line='-e "i18n/phonenumbers/metadatafortesting.js"' />
         <arg line='-e "i18n/phonenumbers/metadatalite.js"' />
         <arg line='-e "i18n/phonenumbers/regioncodefortesting.js"' />
@@ -110,7 +110,7 @@
     <closure-compile inputfile="i18n/phonenumbers/asyoutypeformatter_test.js"
                      outputfile="${null.device}">
       <extraflags>
-        <arg line='-p "i18n/phonenumbers"' />
+        <arg line='--root "i18n/phonenumbers"' />
         <arg line='-e "i18n/phonenumbers/metadata.js"' />
         <arg line='-e "i18n/phonenumbers/metadatalite.js"' />
       </extraflags>
@@ -118,7 +118,7 @@
     <closure-compile inputfile="i18n/phonenumbers/phonenumberutil_test.js"
                      outputfile="${null.device}">
       <extraflags>
-        <arg line='-p "i18n/phonenumbers"' />
+        <arg line='--root "i18n/phonenumbers"' />
         <arg line='-e "i18n/phonenumbers/metadata.js"' />
         <arg line='-e "i18n/phonenumbers/metadatalite.js"' />
       </extraflags>
@@ -126,7 +126,7 @@
     <closure-compile inputfile="i18n/phonenumbers/shortnumberinfo_test.js"
                      outputfile="${null.device}">
       <extraflags>
-        <arg line='-p "i18n/phonenumbers"' />
+        <arg line='--root "i18n/phonenumbers"' />
         <arg line='-e "i18n/phonenumbers/metadata.js"' />
         <arg line='-e "i18n/phonenumbers/metadatalite.js"' />
       </extraflags>

javascript/i18n/phonenumbers/asyoutypeformatter_test.js

@@ -24,6 +24,8 @@
  *
  * @author Nikolaos Trogkanis
  */
+goog.provide('i18n.phonenumbers.AsYouTypeFormatterTest');
+goog.setTestOnly();
 
 goog.require('goog.testing.jsunit');
 goog.require('i18n.phonenumbers.AsYouTypeFormatter');

javascript/i18n/phonenumbers/demo.js

@@ -20,6 +20,7 @@
  *
  * @author Nikolaos Trogkanis
  */
+goog.provide('i18n.phonenumbers.demo');
 
 goog.require('goog.dom');
 goog.require('goog.json');

javascript/i18n/phonenumbers/phonenumberutil_test.js

@@ -24,6 +24,8 @@
  *
  * @author Nikolaos Trogkanis
  */
+goog.provide('i18n.phonenumbers.PhoneNumberUtilTest');
+goog.setTestOnly();
 
 goog.require('goog.array');
 goog.require('goog.string.StringBuffer');

javascript/i18n/phonenumbers/shortnumberinfo_test.js

@@ -25,6 +25,8 @@
  *
  * @author James Wright
  */
+goog.provide('i18n.phonenumbers.ShotNumberInfoTest');
+goog.setTestOnly();
 
 goog.require('goog.testing.jsunit');
 goog.require('i18n.phonenumbers.PhoneNumber');