AssetManager: use libandroidfw for everything except XML parsing

binary XML parsing still relies on ARSClib for now

src/api-impl-jni/android_content_res_AssetManager.c

@@ -167,14 +167,16 @@ JNIEXPORT jobjectArray JNICALL Java_android_content_res_AssetManager_getArrayStr
 	int bag_count = ResTable_lockBag(res_table, ident, &bag);
 	jobjectArray array = (*env)->NewObjectArray(env, bag_count, (*env)->FindClass(env, "java/lang/String"), NULL);
 	for (i = 0; i < bag_count; i++) {
+		struct Res_value value = bag[i].map.value;
+		ssize_t block = ResTable_resolveReference(res_table, &value, bag[i].stringBlock, NULL, NULL, NULL);
 		if (bag[i].map.value.dataType == TYPE_STRING) {
-			const struct ResStringPool *string_pool = ResTable_getTableStringBlock(res_table, bag[i].stringBlock);
+			const struct ResStringPool *string_pool = ResTable_getTableStringBlock(res_table, block);
 			if (string_pool == NULL)
 				continue;
 			size_t len;
 			const char16_t *string = ResStringPool_stringAt(string_pool, bag[i].map.value.data, &len);
 			(*env)->SetObjectArrayElement(env, array, i, (*env)->NewString(env, string, len));
-		} 
+		}
 	}
 
 	ResTable_unlockBag(res_table, bag);
@@ -270,3 +272,99 @@ JNIEXPORT jint JNICALL Java_android_content_res_AssetManager_loadThemeAttributeV
 	}
 	return block;
 }
+
+JNIEXPORT jint JNICALL Java_android_content_res_AssetManager_getArraySize(JNIEnv *env, jobject this, jint ident)
+{
+	struct AssetManager *asset_manager = _PTR(_GET_LONG_FIELD(this, "mObject"));
+	const struct ResTable *res_table = AssetManager_getResources(asset_manager, true);
+	const struct bag_entry *bag;
+	int bag_count = ResTable_lockBag(res_table, ident, &bag);
+	ResTable_unlockBag(res_table, bag);
+	return bag_count;
+}
+
+JNIEXPORT jint JNICALL Java_android_content_res_AssetManager_retrieveArray(JNIEnv *env, jobject this, jint ident, jintArray outArray)
+{
+	struct AssetManager *asset_manager = _PTR(_GET_LONG_FIELD(this, "mObject"));
+	const struct ResTable *res_table = AssetManager_getResources(asset_manager, true);
+	const struct bag_entry *bag;
+	int i;
+
+	jint *array = (*env)->GetIntArrayElements(env, outArray, NULL);
+	int bag_count = ResTable_lockBag(res_table, ident, &bag);
+	for (i = 0; i < bag_count; i++) {
+		struct Res_value value = bag[i].map.value;
+		uint32_t resId = 0;
+		ssize_t block = ResTable_resolveReference(res_table, &value, bag[i].stringBlock, &resId, NULL, NULL);
+
+		array[i*android_content_res_AssetManager_STYLE_NUM_ENTRIES + android_content_res_AssetManager_STYLE_TYPE] = value.dataType;
+		array[i*android_content_res_AssetManager_STYLE_NUM_ENTRIES + android_content_res_AssetManager_STYLE_DATA] = value.data;
+		array[i*android_content_res_AssetManager_STYLE_NUM_ENTRIES + android_content_res_AssetManager_STYLE_ASSET_COOKIE] = block;
+		array[i*android_content_res_AssetManager_STYLE_NUM_ENTRIES + android_content_res_AssetManager_STYLE_RESOURCE_ID] = resId;
+
+	}
+	ResTable_unlockBag(res_table, bag);
+	(*env)->ReleaseIntArrayElements(env, outArray, array, 0);
+	return bag_count;
+}
+
+JNIEXPORT jstring JNICALL Java_android_content_res_AssetManager_getResourcePackageName(JNIEnv *env, jobject this, jint ident)
+{
+	struct AssetManager *asset_manager = _PTR(_GET_LONG_FIELD(this, "mObject"));
+	const struct ResTable *res_table = AssetManager_getResources(asset_manager, true);
+	struct resource_name res_name;
+	bool ret = ResTable_getResourceName(res_table, ident, false, &res_name);
+	return (ret && res_name.package) ? (*env)->NewString(env, res_name.package, res_name.packageLen) : NULL;
+}
+
+JNIEXPORT jstring JNICALL Java_android_content_res_AssetManager_getResourceTypeName(JNIEnv *env, jobject this, jint ident)
+{
+	struct AssetManager *asset_manager = _PTR(_GET_LONG_FIELD(this, "mObject"));
+	const struct ResTable *res_table = AssetManager_getResources(asset_manager, true);
+	struct resource_name res_name;
+	bool ret = ResTable_getResourceName(res_table, ident, false, &res_name);
+	return (ret && res_name.type) ? (*env)->NewString(env, res_name.type, res_name.typeLen) : NULL;
+}
+
+JNIEXPORT jstring JNICALL Java_android_content_res_AssetManager_getResourceEntryName(JNIEnv *env, jobject this, jint ident)
+{
+	struct AssetManager *asset_manager = _PTR(_GET_LONG_FIELD(this, "mObject"));
+	const struct ResTable *res_table = AssetManager_getResources(asset_manager, true);
+	struct resource_name res_name;
+	bool ret = ResTable_getResourceName(res_table, ident, false, &res_name);
+	return (ret && res_name.name) ? (*env)->NewString(env, res_name.name, res_name.nameLen) : NULL;
+}
+
+JNIEXPORT jint JNICALL Java_android_content_res_AssetManager_loadResourceBagValue(JNIEnv *env, jobject this, jint ident, jint bagEntryId, jobject outValue, jboolean resolve)
+{
+	struct AssetManager *asset_manager = _PTR(_GET_LONG_FIELD(this, "mObject"));
+	const struct ResTable *res_table = AssetManager_getResources(asset_manager, true);
+	const struct bag_entry *bag;
+	int i;
+	ssize_t block = -1;
+
+	int bag_count = ResTable_lockBag(res_table, ident, &bag);
+	for (i = 0; i < bag_count; i++) {
+		if (bag[i].map.name.ident == bagEntryId) {
+			struct Res_value value = bag[i].map.value;
+			uint32_t resId = 0;
+			block = ResTable_resolveReference(res_table, &value, bag[i].stringBlock, &resId, NULL, NULL);
+
+			_SET_INT_FIELD(outValue, "type", value.dataType);
+			_SET_INT_FIELD(outValue, "data", value.data);
+			_SET_INT_FIELD(outValue, "resourceId", resId);
+			if (value.dataType == TYPE_STRING) {
+				const struct ResStringPool *string_pool = ResTable_getTableStringBlock(res_table, block);
+				size_t len;
+				const char16_t *string = ResStringPool_stringAt(string_pool, value.data, &len);
+				_SET_OBJ_FIELD(outValue, "string", "Ljava/lang/CharSequence;", (*env)->NewString(env, string, len));
+			} else {
+				_SET_OBJ_FIELD(outValue, "string", "Ljava/lang/CharSequence;", NULL);
+			}
+			break;
+		}
+	}
+
+	ResTable_unlockBag(res_table, bag);
+	return block;
+}

src/api-impl-jni/generated_headers/android_content_res_AssetManager.h

@@ -89,6 +89,30 @@ JNIEXPORT jobjectArray JNICALL Java_android_content_res_AssetManager_getLocales
 JNIEXPORT jint JNICALL Java_android_content_res_AssetManager_getResourceIdentifier
   (JNIEnv *, jobject, jstring, jstring, jstring);
 
+/*
+ * Class:     android_content_res_AssetManager
+ * Method:    getResourcePackageName
+ * Signature: (I)Ljava/lang/String;
+ */
+JNIEXPORT jstring JNICALL Java_android_content_res_AssetManager_getResourcePackageName
+  (JNIEnv *, jobject, jint);
+
+/*
+ * Class:     android_content_res_AssetManager
+ * Method:    getResourceTypeName
+ * Signature: (I)Ljava/lang/String;
+ */
+JNIEXPORT jstring JNICALL Java_android_content_res_AssetManager_getResourceTypeName
+  (JNIEnv *, jobject, jint);
+
+/*
+ * Class:     android_content_res_AssetManager
+ * Method:    getResourceEntryName
+ * Signature: (I)Ljava/lang/String;
+ */
+JNIEXPORT jstring JNICALL Java_android_content_res_AssetManager_getResourceEntryName
+  (JNIEnv *, jobject, jint);
+
 /*
  * Class:     android_content_res_AssetManager
  * Method:    openAsset

src/api-impl/android/content/res/AssetManager.java

@@ -111,8 +111,6 @@ public final class AssetManager {
 	private boolean mOpen = true;
 	private HashMap<Integer, RuntimeException> mRefStacks;
 
-	private List<TableBlock> tableBlocks;
-
 	/**
 	 * Create a new AssetManager containing only the basic system assets.
 	 * Applications will not generally use this method, instead retrieving the
@@ -121,19 +119,6 @@ public final class AssetManager {
 	 * {@hide}
 	 */
 	public AssetManager() {
-		tableBlocks = new ArrayList<>();
-		// try {
-		// 	Enumeration<URL> resources = ClassLoader.getSystemClassLoader().getResources("resources.arsc");
-		// 	while (resources.hasMoreElements()) {
-		// 		URL resource = resources.nextElement();
-		// 		if (!resource.getFile().contains("com.google.android.gms")) { // ignore MicroG .apk
-		// 			tableBlocks.add(TableBlock.load(resource.openStream()));
-		// 		}
-		// 	}
-		// } catch (IOException e) {
-		// 	Log.e(TAG, "failed to load resources.arsc" + e);
-		// }
-
 		// FIXME: evaluate if this can be axed
 		synchronized (this) {
 			if (DEBUG_REFS) {
@@ -160,21 +145,6 @@ public final class AssetManager {
 		}
 	}
 
-	private Map<Integer, EntryGroup> tableBlockCache = new HashMap<>();
-	private EntryGroup tableBlockSearch(int resId) {
-		if (tableBlockCache.containsKey(resId)) {
-			return tableBlockCache.get(resId);
-		}
-		EntryGroup ret = null;
-		for (TableBlock tableBlock : tableBlocks) {
-			ret = tableBlock.search(resId);
-			if (ret != null)
-				break;
-		}
-		tableBlockCache.put(resId, ret);
-		return ret;
-	}
-
 	private static void ensureSystemAssets() {
 		synchronized (sSync) {
 			if (sSystem == null) {
@@ -238,20 +208,12 @@ public final class AssetManager {
 	 * identifier for the current configuration / skin.
 	 */
 	/*package*/ final CharSequence getResourceBagText(int ident, int bagEntryId) {
-		PluralsBag pluralsBag = PluralsBag.create(tableBlockSearch(ident).pickOne());
+		TypedValue value = new TypedValue();
-		return pluralsBag.getQuantityString(PluralsQuantity.valueOf((short)bagEntryId));
+		int block = loadResourceBagValue(ident, bagEntryId, value, true);
-
+		if (block >= 0) {
-//		synchronized (this) {
+			return value.coerceToString();
-//			TypedValue tmpValue = mValue;
+		}
-//			int block = loadResourceBagValue(ident, bagEntryId, tmpValue, true);
+		return null;
-//			if (block >= 0) {
-//				if (tmpValue.type == TypedValue.TYPE_STRING) {
-//					return mStringBlocks[block].get(tmpValue.data);
-//				}
-//				return tmpValue.coerceToString();
-//			}
-//		}
-//		return null;
 	}
 
 	/**
@@ -277,30 +239,24 @@ public final class AssetManager {
 	 * @param id Resource id of the string array
 	 */
 	/*package*/ final CharSequence[] getResourceTextArray(final int id) {
-		ResValueMap children[] = tableBlockSearch(id).pickOne().getResValueMapArray().getChildes();
+		int n = getArraySize(id);
-		CharSequence values[] = new CharSequence[children.length];
+		int[] valueArray = new int[n * STYLE_NUM_ENTRIES];
-		for (int i = 0; i < children.length; i++) {
+		retrieveArray(id, valueArray);
-			StringItem stringItem = children[i].getDataAsPoolString();
+		CharSequence[] values = new String[n];
-			values[i] = (stringItem != null) ? stringItem.get() : "";
+		TypedValue value = new TypedValue();
+		for (int i = 0; i < n; i++) {
+			value.data = valueArray[i*STYLE_NUM_ENTRIES + STYLE_DATA];
+			value.type = valueArray[i*STYLE_NUM_ENTRIES + STYLE_TYPE];
+			value.assetCookie = valueArray[i*STYLE_NUM_ENTRIES + STYLE_ASSET_COOKIE];
+			if (value.type == TypedValue.TYPE_STRING) {
+				values[i] = getPooledString(value.assetCookie, value.data);
+			} else {
+				values[i] = value.coerceToString();
+			}
 		}
 		return values;
 	}
 
-	public Map<Integer,ValueItem> loadStyle(int resId) {
-		Map<Integer,ValueItem> map = new HashMap<>();
-		EntryGroup entryGroup = tableBlockSearch(resId);
-		while (entryGroup != null) {
-			Entry entry = entryGroup.pickOne();
-			ResValueMapArray array = entry.getResValueMapArray();
-			for (int i = 0; i < array.childesCount(); i++) {
-				map.putIfAbsent(array.get(i).getName(), array.get(i));
-			}
-			int parent_id = ((EntryHeaderMap)entry.getHeader()).getParentId();
-			entryGroup = tableBlockSearch(parent_id);
-		}
-		return map;
-	}
-
 	/*package*/ final boolean getThemeValue(long style, int ident,
 						TypedValue outValue, boolean resolveRefs) {
 		int block = loadThemeAttributeValue(style, ident, outValue, resolveRefs);
@@ -776,17 +732,18 @@ public final class AssetManager {
 	/*package*/ native final int getResourceIdentifier(String name, String type, String defPackage);
 
 	/*package*/ /*native*/ final String getResourceName(int resid) {
-		return tableBlockSearch(resid).pickOne().getName();
+		String name = getResourcePackageName(resid);
-	}
+		String type = getResourceTypeName(resid);
-	/*package*/ /*native*/ final String getResourcePackageName(int resid) {
+		if (type != null)
-		return tableBlockSearch(resid).pickOne().getPackageName();
+			name += ':' + type;
-	}
+		String entry = getResourceEntryName(resid);
-	/*package*/ /*native*/ final String getResourceTypeName(int resid) {
+		if (entry != null)
-		return tableBlockSearch(resid).pickOne().getTypeName();
+			name += '/' + entry;
-	}
+		return name;
-	/*package*/ /*native*/ final String getResourceEntryName(int resid) {
-		return tableBlockSearch(resid).pickOne().getName();
 	}
+	/*package*/ native final String getResourcePackageName(int resid);
+	/*package*/ native final String getResourceTypeName(int resid);
+	/*package*/ native final String getResourceEntryName(int resid);
 
 	private native final int openAsset(String fileName, int accessMode);
 	private final native ParcelFileDescriptor openAssetFd(String fileName,
@@ -810,13 +767,6 @@ public final class AssetManager {
 	private native final int loadResourceValue(int ident, short density, TypedValue outValue,
 						   boolean resolve);
 
-	private int getCookie(ValueItem valueItem) {
-		for (int i=0; i<tableBlocks.size(); i++) {
-			if (valueItem.getStringPool() == tableBlocks.get(i).getStringPool())
-				return i;
-		}
-		return -1;
-	}
 	/**
 	 * Returns true if the resource was found, filling in mRetStringBlock and
 	 *  mRetData.
@@ -833,12 +783,6 @@ public final class AssetManager {
 	/*package*/ /*native*/ final boolean applyStyle(long theme,
 							   int defStyleAttr, int defStyleRes, AttributeSet set,
 							   int[] inAttrs, int[] outValues, int[] outIndices) {
-		// if (defStyleRes == 0 && theme.containsKey(defStyleAttr))
-		// 	defStyleRes = theme.get(defStyleAttr).getData();
-		if (defStyleRes == 0 && set != null)
-			defStyleRes = set.getAttributeResourceValue(null, "style", 0);
-		Map<Integer,ValueItem> defStyle = loadStyle(defStyleRes);
-
 		ResXmlPullParser parser = (ResXmlPullParser)set;
 		outIndices[0] = 0;
 
@@ -854,31 +798,11 @@ public final class AssetManager {
 			outValues[d+AssetManager.STYLE_RESOURCE_ID] = 0;
 			outValues[d+AssetManager.STYLE_TYPE] = 0;
 			outValues[d+AssetManager.STYLE_DATA] = 0;
-			outValues[d+AssetManager.STYLE_ASSET_COOKIE] = 0;
+			outValues[d+AssetManager.STYLE_ASSET_COOKIE] = -1;
 			int resId = inAttrs[i];
-			EntryGroup entryGroup = null;
-			Entry entry = null;
 			ValueItem valueItem = null;
-			while (entry == null || "attr".equals(entry.getTypeName())) {
+			if (xmlCache.containsKey(resId)) {
-				valueItem = null;
+				valueItem = parser.getResXmlAttributeAt(xmlCache.get(resId));
-				if (xmlCache.containsKey(resId)) {
-					valueItem = parser.getResXmlAttributeAt(xmlCache.get(resId));
-				}
-				if (valueItem == null) {
-					valueItem = defStyle.get(resId);
-					if (valueItem != null && valueItem.getValueType() == ValueType.ATTRIBUTE && valueItem.getData() == resId)
-						valueItem = null;
-				}
-				if (valueItem != null && valueItem.getValueType() == ValueType.ATTRIBUTE) {
-					resId = valueItem.getData();
-					entryGroup = tableBlockSearch(resId);
-					if (entryGroup == null) {
-						break;
-					}
-					entry = entryGroup.pickOne();
-				} else {
-					break;
-				}
 			}
 			if (valueItem == null) {
 				if (theme != 0) {
@@ -894,34 +818,40 @@ public final class AssetManager {
 				}
 				continue;
 			}
-			if (valueItem.getValueType() == ValueType.REFERENCE) {
+			if (valueItem.getValueType() == ValueType.REFERENCE || valueItem.getValueType() == ValueType.ATTRIBUTE) {
+				int prev_resId = resId;
 				resId = valueItem.getData();
 				TypedValue value = new TypedValue();
-				int block = loadResourceValue(resId, (short)0, value, true);
+				int block = -1;
-				if (block >= 0) {
+				if (theme != 0)
-					outValues[d + AssetManager.STYLE_RESOURCE_ID] = value.resourceId;
+					block = loadThemeAttributeValue(theme, resId, value, true);
-					outValues[d + AssetManager.STYLE_TYPE] = value.type;
+				if (block < 0)
-					outValues[d + AssetManager.STYLE_DATA] = value.data;
+					block = loadResourceValue(resId, (short)0, value, true);
-					outValues[d + AssetManager.STYLE_ASSET_COOKIE] = block;
+				if (block < 0) {
-					outIndices[++outIndices[0]] = i;
+					block = -1;
-					continue;
+					if (resId == 0) {
+						value.resourceId = 0;
+						value.type = 0;
+						value.data = 0;
+					} else {
+						value.resourceId = prev_resId;
+						value.type = valueItem.getType();
+						value.data = valueItem.getData();
+					}
 				}
-				while (valueItem.getValueType() == ValueType.REFERENCE) {
+				outValues[d + AssetManager.STYLE_RESOURCE_ID] = value.resourceId;
-					resId = valueItem.getData();
+				outValues[d + AssetManager.STYLE_TYPE] = value.type;
-					if (resId == 0)
+				outValues[d + AssetManager.STYLE_DATA] = value.data;
-						break;
+				outValues[d + AssetManager.STYLE_ASSET_COOKIE] = block;
-					entry = tableBlockSearch(resId).pickOne();
+				outIndices[++outIndices[0]] = i;
-					if (entry == null || entry.getResValue() == null)
+				continue;
-						break;
+			} else {
-					valueItem = entry.getResValue();
+				outValues[d + AssetManager.STYLE_RESOURCE_ID] = 0;
-				}
+				outValues[d + AssetManager.STYLE_TYPE] = valueItem.getType();
-
+				outValues[d + AssetManager.STYLE_DATA] = valueItem.getData();
-				outValues[d + AssetManager.STYLE_RESOURCE_ID] = resId;
+				outValues[d + AssetManager.STYLE_ASSET_COOKIE] = -1;
+				outIndices[++outIndices[0]] = i;
 			}
-			outValues[d + AssetManager.STYLE_TYPE] = valueItem.getType();
-			outValues[d + AssetManager.STYLE_DATA] = valueItem.getData();
-			outValues[d + AssetManager.STYLE_ASSET_COOKIE] = getCookie(valueItem);
-			outIndices[++outIndices[0]] = i;
 		}
 		return true;
 	}
@@ -968,10 +898,16 @@ public final class AssetManager {
 	private native final String[] getArrayStringResource(int arrayRes);
 	private native final int[] getArrayStringInfo(int arrayRes);
 	/*package*/ final int[] getArrayIntResource(int arrayRes) {
-		ResValueMap children[] = tableBlockSearch(arrayRes).pickOne().getResValueMapArray().getChildes();
+		int n = getArraySize(arrayRes);
-		int values[] = new int[children.length];
+		int[] valueArray = new int[n * STYLE_NUM_ENTRIES];
-		for (int i = 0; i < children.length; i++) {
+		retrieveArray(arrayRes, valueArray);
-			values[i] = children[i].getData();
+		int[] values = new int[n];
+		TypedValue value = new TypedValue();
+		for (int i = 0; i < n; i++) {
+			value.data = valueArray[i*STYLE_NUM_ENTRIES + STYLE_DATA];
+			value.type = valueArray[i*STYLE_NUM_ENTRIES + STYLE_TYPE];
+			value.assetCookie = valueArray[i*STYLE_NUM_ENTRIES + STYLE_ASSET_COOKIE];
+			values[i] = value.data;
 		}
 		return values;
 	}