Simplify ant signing/zipalign code

build/android/ant/apk-build.xml

 <?xml version="1.0" encoding="UTF-8"?>
 <!--
     Copyright (C) 2005-2008 The Android Open Source Project
 
     Licensed under the Apache License, Version 2.0 (the "License");
     you may not use this file except in compliance with the License.
     You may obtain a copy of the License at
 
          http://www.apache.org/licenses/LICENSE-2.0
 
@@ skipping 192 matching lines @@
       For debug builds, the Android SDK tools create a key in ~/.android and sign the build with it.
       This has caused all kinds of issues. Instead, the debug build should be signed with a key in
       build/android/ant. The SDK tools do not provide any support for overriding that behavior and so
       instead one must use the hack below.
     -->
     <property name="key.store" value="${CHROMIUM_SRC}/build/android/ant/chromium-debug.keystore"/>
     <property name="key.store.password" value="chromium"/>
     <property name="key.alias" value="chromiumdebugkey"/>
     <property name="key.alias.password" value="chromium"/>
 
-    <!-- properties for signing in release mode -->
-    <condition property="has.keystore">
-        <and>
-            <isset property="key.store" />
-            <length string="${key.store}" when="greater" length="0" />
-            <isset property="key.alias" />
-        </and>
-    </condition>
-    <condition property="has.password">
-        <and>
-            <isset property="has.keystore" />
-            <isset property="key.store.password" />
-            <isset property="key.alias.password" />
-        </and>
-    </condition>
-
     <!-- properties for packaging -->
     <property name="build.packaging.nocrunch" value="true" />
 
     <!-- whether we need to fork javac.
          This is only needed on Windows when running Java < 7 -->
     <condition else="false" property="need.javac.fork">
         <and>
             <matches pattern="1\.[56]" string="${java.specification.version}"/>
             <not>
                 <os family="unix"/>
@@ skipping 614 matching lines @@
     </target>
 
     <!-- SDK tools assume that out.packaged.file is signed and name it "...-unaligned" -->
     <property name="out.packaged.file"
       value="${apks.dir}/${ant.project.name}-unsigned.apk" />
     <property name="out.unaligned.file"
       value="${apks.dir}/${ant.project.name}-unaligned.apk" />
 
     <!-- By default, the SDK tools build only aligns the APK in the -do-debug target. -->
     <!-- Builds debug output package -->
+    <target name="-do-sign"
+        depends="-package, -post-package">
+      <sequential>
+        <!-- Signs the APK -->
+        <echo level="info">Signing final apk...</echo>
+        <signapk
+          input="${out.packaged.file}"
+          output="${out.unaligned.file}"
+          keystore="${key.store}"
+          storepass="${key.store.password}"
+          alias="${key.alias}"
+          keypass="${key.alias.password}"/>
+
+        <!-- Zip aligns the APK -->
+        <zipalign-helper
+          in.package="${out.unaligned.file}"
+          out.package="${out.final.file}" />
+        <echo level="info">Release Package: ${out.final.file}</echo>
+      </sequential>
+    </target>
+
     <target name="-do-debug"
-        depends="-set-debug-mode, -debug-obfuscation-check, -package, -post-package">
-      <!-- only create apk if *not* a library project -->
-      <do-only-if-not-library elseText="Library project: do not create apk..." >
-        <sequential>
-          <!-- Signs the APK -->
-          <echo level="info">Signing final apk...</echo>
-          <signapk
-            input="${out.packaged.file}"
-            output="${out.unaligned.file}"
-            keystore="${key.store}"
-            storepass="${key.store.password}"
-            alias="${key.alias}"
-            keypass="${key.alias.password}"/>
-
-          <!-- Zip aligns the APK -->
-          <zipalign-helper
-            in.package="${out.unaligned.file}"
-            out.package="${out.final.file}" />
-          <echo level="info">Release Package: ${out.final.file}</echo>
-        </sequential>
-      </do-only-if-not-library>
+      depends="-set-debug-mode, -debug-obfuscation-check, -do-sign">
       <record-build-info />
     </target>
 
     <!-- Builds debug output package -->
     <target name="debug" depends="-set-debug-files, -do-debug, -post-build"
                 description="Builds the application and signs it with a debug key.">
     </target>
 
-
     <!-- ******************************************************* -->
     <!-- *************** Release specific targets ************** -->
     <!-- ******************************************************* -->
 
-    <!-- called through target 'release'. Only executed if the keystore and
-         key alias are known but not their password. -->
-    <target name="-release-prompt-for-password" if="has.keystore" unless="has.password">
-    </target>
-
-    <!-- called through target 'release'. Only executed if there's no
-         keystore/key alias set -->
-    <target name="-release-nosign" unless="has.keystore">
-        <record-build-info />
-    </target>
-
     <target name="-release-obfuscation-check">
         <echo level="info">proguard.config is ${proguard.config}</echo>
         <condition property="proguard.enabled" value="true" else="false">
             <and>
                 <isset property="build.is.mode.release" />
                 <isset property="proguard.config" />
             </and>
         </condition>
         <if condition="${proguard.enabled}">
             <then>
@@ skipping 32 matching lines @@
             </then>
             <else>
                 <!-- property only set in release mode.
                      Useful for if/unless attributes in target node
                      when using Ant before 1.8 -->
                 <property name="build.is.mode.release" value="true"/>
             </else>
         </if>
     </target>
 
-    <target name="-release-sign" if="has.keystore" >
-        <!-- only create apk if *not* a library project -->
-        <do-only-if-not-library elseText="Library project: do not create apk..." >
-            <sequential>
-                <property name="out.unaligned.file" location="${out.absolute.dir}/${ant.project.name}-release-unaligned.apk" />
-
-                <!-- Signs the APK -->
-                <echo level="info">Signing final apk...</echo>
-                <signapk
-                        input="${out.packaged.file}"
-                        output="${out.unaligned.file}"
-                        keystore="${key.store}"
-                        storepass="${key.store.password}"
-                        alias="${key.alias}"
-                        keypass="${key.alias.password}"/>
-
-                <!-- Zip aligns the APK -->
-                <zipalign-helper
-                        in.package="${out.unaligned.file}"
-                        out.package="${out.final.file}" />
-                <echo level="info">Release Package: ${out.final.file}</echo>
-            </sequential>
-        </do-only-if-not-library>
+    <target name="-release-sign" depends="-do-sign" >
         <record-build-info />
     </target>
 
-    <!-- This runs -package-release and -release-nosign first and then runs
+    <!-- This runs -package-release first and then runs
          only if release-sign is true (set in -release-check,
          called by -release-no-sign)-->
     <target name="release"
-                depends="-set-release-mode, -release-obfuscation-check, -package, -post-package, -release-prompt-for-password, -release-nosign, -release-sign, -post-build"
+                depends="-set-release-mode, -release-obfuscation-check, -package, -post-package, -release-sign, -post-build"
                 description="Builds the application in release mode.">
     </target>
 
 </project>