summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndrew Hayden <andrewhayden@chromium.org>2015-01-20 09:50:57 +0000
committerAndrew Hayden <andrewhayden@chromium.org>2015-01-20 09:52:33 +0000
commit86948ef332b3eb11aa1cf67a2ce3935f4b91e788 (patch)
tree780c2abd8faafd8ce8256c424501b3194a979b8b
parent3b6397eb333a26a34a93a7b8db2de1431964c672 (diff)
downloadchromium_src-86948ef332b3eb11aa1cf67a2ce3935f4b91e788.zip
chromium_src-86948ef332b3eb11aa1cf67a2ce3935f4b91e788.tar.gz
chromium_src-86948ef332b3eb11aa1cf67a2ce3935f4b91e788.tar.bz2
Skip direct map from apk check for Samsung Mega.
Avoid a warning from Samsung knox by skipping attempts to mmap the apk file for UMA data. Instead return false, appropriate since any attempt later to really map the file would similarly trigger this knox warning. BUG=448084 Review URL: https://codereview.chromium.org/857833002 Cr-Commit-Position: refs/heads/master@{#312138} (cherry picked from commit fba86e9cdbf10b89e6b644dbf1451cd67fa53e4f) R=rmcilroy@chromium.org Review URL: https://codereview.chromium.org/858983002 Cr-Commit-Position: refs/branch-heads/2272@{#56} Cr-Branched-From: 827a380cfdb31aa54c8d56e63ce2c3fd8c3ba4d4-refs/heads/master@{#310958}
-rw-r--r--base/android/java/src/org/chromium/base/library_loader/Linker.java10
1 files changed, 10 insertions, 0 deletions
diff --git a/base/android/java/src/org/chromium/base/library_loader/Linker.java b/base/android/java/src/org/chromium/base/library_loader/Linker.java
index d58d1fc..dfcc141 100644
--- a/base/android/java/src/org/chromium/base/library_loader/Linker.java
+++ b/base/android/java/src/org/chromium/base/library_loader/Linker.java
@@ -845,6 +845,16 @@ public class Linker {
*/
public static boolean checkMapExecSupport(String apkFile) {
assert apkFile != null;
+
+ // https://code.google.com/p/chromium/issues/detail?id=448084
+ // Do not check if the device is Samsung Mega.
+ final String model = android.os.Build.MODEL;
+ if (model != null && model.equals("GT-I9205")) {
+ if (DEBUG) Log.i(TAG, "checkMapExecSupport: model is '" + model
+ + "', returning false");
+ return false;
+ }
+
synchronized (Linker.class) {
ensureInitializedLocked();