Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1210)

Unified Diff: content/shell/android/shell_apk/src/org/chromium/content_shell_apk/ContentShellApplication.java

Issue 1147213004: Store and load icudtl.dat directly from the apk rather than extracting on start-up (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@load-from-apk
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/shell/android/shell_apk/src/org/chromium/content_shell_apk/ContentShellApplication.java
diff --git a/content/shell/android/shell_apk/src/org/chromium/content_shell_apk/ContentShellApplication.java b/content/shell/android/shell_apk/src/org/chromium/content_shell_apk/ContentShellApplication.java
index bd0b00e303e398986b88042c83b582c108fba19b..d675a0b67187edf6339959c8c72905d3d95e860a 100644
--- a/content/shell/android/shell_apk/src/org/chromium/content_shell_apk/ContentShellApplication.java
+++ b/content/shell/android/shell_apk/src/org/chromium/content_shell_apk/ContentShellApplication.java
@@ -16,15 +16,8 @@ import org.chromium.content.app.ContentApplication;
public class ContentShellApplication extends ContentApplication {
public static final String COMMAND_LINE_FILE = "/data/local/tmp/content-shell-command-line";
- /**
- * icudtl.dat provides ICU (i18n library) with all the data for its
- * operation. We use to link it statically to our binary, but not any more
- * so that we have to install it along with other mandatory pak files.
- * See src/third_party/icu/README.chromium.
- */
private static final String[] MANDATORY_PAK_FILES = new String[] {
"content_shell.pak",
- "icudtl.dat",
};
private static final String PRIVATE_DATA_DIRECTORY_SUFFIX = "content_shell";

Powered by Google App Engine
This is Rietveld 408576698