Upstream: Version WebAPK runtime library

chrome/android/java/src/org/chromium/chrome/browser/webapps/WebApkVersionManager.java

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.webapps;
 
 import android.content.Context;
 import android.content.SharedPreferences;
 
+import org.chromium.base.CommandLine;
 import org.chromium.base.ContextUtils;
 import org.chromium.base.FileUtils;
 import org.chromium.base.ThreadUtils;
+import org.chromium.chrome.browser.ChromeSwitches;
 import org.chromium.webapk.lib.client.DexOptimizer;
+import org.chromium.webapk.lib.client.WebApkVersion;
+import org.chromium.webapk.lib.common.WebApkUtils;
 
 import java.io.File;
 
 /**
  * Updates installed WebAPKs after a Chrome update.
  */
 public class WebApkVersionManager {
     /**
-     * Name of the shared preference to store whether an attempt to extract the WebAPK runtime
-     * library was made.
+     * Name of the shared preference for the version number of the dynamically loaded dex.
      */
-    private static final String TRIED_EXTRACTING_DEX_PREF =
-            "org.chromium.chrome.browser.webapps.TRIED_EXTRACTING_DEX";
+    private static final String EXTRACTED_DEX_VERSION_PREF =
+            "org.chromium.chrome.browser.webapps.extracted_dex_version";
 
     /**
      * Tries to extract the WebAPK runtime dex from the Chrome APK if it has not tried already.
      * Should not be called on UI thread.
      */
     public static void updateWebApksIfNeeded() {
         assert !ThreadUtils.runningOnUiThread();
 
+        // TODO(pkotwicz|hanxi): Detect whether the manifest of installed APKs needs to be updated.
+        // (crbug.com/604513)
+
         SharedPreferences preferences = ContextUtils.getAppSharedPreferences();
-        if (preferences.getBoolean(TRIED_EXTRACTING_DEX_PREF, false)) {
+        int extractedDexVersion = preferences.getInt(EXTRACTED_DEX_VERSION_PREF, -1);
+        if (!CommandLine.getInstance().hasSwitch(
+                    ChromeSwitches.ALWAYS_EXTRACT_WEBAPK_RUNTIME_DEX_ON_STARTUP)

[gone, 2016/05/31 18:09:09]

Something about the indentation here is really, really weird.

[pkotwicz, 2016/05/31 21:39:05]

This is how clang-format formats the line
https://g3doc.corp.google.com/devtools/cymbal/tools/g3doc/clang-format/index....

[gone, 2016/05/31 21:49:39]

Clang consistently screws up Java code.

+                && extractedDexVersion == WebApkVersion.CURRENT_RUNTIME_DEX_VERSION) {
             return;
         }
 
-        preferences.edit().putBoolean(TRIED_EXTRACTING_DEX_PREF, true).apply();
+        SharedPreferences.Editor editor = preferences.edit();
+        editor.putInt(EXTRACTED_DEX_VERSION_PREF, WebApkVersion.CURRENT_RUNTIME_DEX_VERSION);
+        editor.apply();
 
         Context context = ContextUtils.getApplicationContext();
         File dexDir = context.getDir("dex", Context.MODE_PRIVATE);
-        File dexFile = new File(dexDir, "web_apk.dex");
-        if (!FileUtils.extractAsset(context, "web_apk.dex", dexFile)
-                || !DexOptimizer.optimize(dexFile)) {
+        FileUtils.recursivelyDeleteFile(dexDir);
+
+        // Recreate world-executable directory using {@link Context#getDir}.
+        dexDir = context.getDir("dex", Context.MODE_PRIVATE);
+
+        String dexName =
+                WebApkUtils.getRuntimeDexName(WebApkVersion.CURRENT_RUNTIME_DEX_VERSION);
+        File dexFile = new File(dexDir, dexName);
+        if (!FileUtils.extractAsset(context, dexName, dexFile) || !DexOptimizer.optimize(dexFile)) {
             return;
         }
 
         // Make dex file world-readable so that WebAPK can use it.
         dexFile.setReadable(true, false);
     }
 }