Snap for 6324712 from 7a9c03e529da5f484ca863d5e0ae1cdca3dabe7d to qt-qpr3-release (0586184e) · Commits · e / os / android_frameworks_base

core/java/android/net/Uri.java

+34 −27

Original line number	Diff line number	Diff line
		@@ -1987,17 +1987,26 @@ public abstract class Uri implements Parcelable, Comparable<Uri> {
		* Enum which indicates which representation of a given part we have.
		*/
		static class Representation {
		static final int BOTH = 0;
		static final int ENCODED = 1;
		static final int DECODED = 2;
		}

		volatile String encoded;
		volatile String decoded;
		private final int mCanonicalRepresentation;

		AbstractPart(String encoded, String decoded) {
		if (encoded != NOT_CACHED) {
		this.mCanonicalRepresentation = Representation.ENCODED;
		this.encoded = encoded;
		this.decoded = NOT_CACHED;
		} else if (decoded != NOT_CACHED) {
		this.mCanonicalRepresentation = Representation.DECODED;
		this.encoded = NOT_CACHED;
		this.decoded = decoded;
		} else {
		throw new IllegalArgumentException("Neither encoded nor decoded");
		}
		}

		abstract String getEncoded();
		@@ -2009,25 +2018,21 @@ public abstract class Uri implements Parcelable, Comparable<Uri> {
		}

		final void writeTo(Parcel parcel) {
		@SuppressWarnings("StringEquality")
		boolean hasEncoded = encoded != NOT_CACHED;

		@SuppressWarnings("StringEquality")
		boolean hasDecoded = decoded != NOT_CACHED;

		if (hasEncoded && hasDecoded) {
		parcel.writeInt(Representation.BOTH);
		parcel.writeString(encoded);
		parcel.writeString(decoded);
		} else if (hasEncoded) {
		parcel.writeInt(Representation.ENCODED);
		parcel.writeString(encoded);
		} else if (hasDecoded) {
		parcel.writeInt(Representation.DECODED);
		parcel.writeString(decoded);
		final String canonicalValue;
		if (mCanonicalRepresentation == Representation.ENCODED) {
		canonicalValue = encoded;
		} else if (mCanonicalRepresentation == Representation.DECODED) {
		canonicalValue = decoded;
		} else {
		throw new IllegalArgumentException("Neither encoded nor decoded");
		throw new IllegalArgumentException("Unknown representation: "
		+ mCanonicalRepresentation);
		}
		if (canonicalValue == NOT_CACHED) {
		throw new AssertionError("Canonical value not cached ("
		+ mCanonicalRepresentation + ")");
		}
		parcel.writeInt(mCanonicalRepresentation);
		parcel.writeString(canonicalValue);
		}
		}

		@@ -2059,13 +2064,12 @@ public abstract class Uri implements Parcelable, Comparable<Uri> {

		static Part readFrom(Parcel parcel) {
		int representation = parcel.readInt();
		String value = parcel.readString();
		switch (representation) {
		case Representation.BOTH:
		return from(parcel.readString(), parcel.readString());
		case Representation.ENCODED:
		return fromEncoded(parcel.readString());
		return fromEncoded(value);
		case Representation.DECODED:
		return fromDecoded(parcel.readString());
		return fromDecoded(value);
		default:
		throw new IllegalArgumentException("Unknown representation: "
		+ representation);
		@@ -2127,6 +2131,11 @@ public abstract class Uri implements Parcelable, Comparable<Uri> {
		private static class EmptyPart extends Part {
		public EmptyPart(String value) {
		super(value, value);
		if (value != null && !value.isEmpty()) {
		throw new IllegalArgumentException("Expected empty value, got: " + value);
		}
		// Avoid having to re-calculate the non-canonical value.
		encoded = decoded = value;
		}

		@Override
		@@ -2245,14 +2254,12 @@ public abstract class Uri implements Parcelable, Comparable<Uri> {
		static PathPart readFrom(Parcel parcel) {
		int representation = parcel.readInt();
		switch (representation) {
		case Representation.BOTH:
		return from(parcel.readString(), parcel.readString());
		case Representation.ENCODED:
		return fromEncoded(parcel.readString());
		case Representation.DECODED:
		return fromDecoded(parcel.readString());
		default:
		throw new IllegalArgumentException("Bad representation: " + representation);
		throw new IllegalArgumentException("Unknown representation: " + representation);
		}
		}

core/tests/coretests/src/android/net/UriTest.java

+60 −0

Original line number	Diff line number	Diff line
		@@ -24,6 +24,9 @@ import androidx.test.filters.SmallTest;
		import junit.framework.TestCase;

		import java.io.File;
		import java.lang.reflect.Constructor;
		import java.lang.reflect.InvocationTargetException;
		import java.lang.reflect.Method;
		import java.util.Arrays;
		import java.util.Iterator;
		import java.util.List;
		@@ -816,6 +819,63 @@ public class UriTest extends TestCase {
		Uri.parse("content://com.example/path%2Fpath")));
		}


		/**
		* Check that calling Part(String, String) with inconsistent Strings does not lead
		* to the Part's encoded vs. decoded values being inconsistent.
		*/
		public void testPart_consistentEncodedVsDecoded() throws Exception {
		Object authority = createPart(Class.forName("android.net.Uri$Part"), "a.com", "b.com");
		Object path = createPart(Class.forName("android.net.Uri$PathPart"), "/foo/a", "/foo/b");
		Uri uri = makeHierarchicalUri(authority, path);
		// In these cases, decoding/encoding the encoded/decoded representation yields the same
		// String, so we can just assert equality.
		// assertEquals(uri.getPath(), uri.getEncodedPath());
		assertEquals(uri.getAuthority(), uri.getEncodedAuthority());

		// When both encoded and decoded strings are given, the encoded one is preferred.
		assertEquals("a.com", uri.getAuthority());
		assertEquals("/foo/a", uri.getPath());
		}

		private Object createPart(Class partClass, String encoded, String decoded) throws Exception {
		Constructor partConstructor = partClass.getDeclaredConstructor(String.class, String.class);
		partConstructor.setAccessible(true);
		return partConstructor.newInstance(encoded, decoded);
		}

		private static Uri makeHierarchicalUri(Object authority, Object path) throws Exception {
		Class hierarchicalUriClass = Class.forName("android.net.Uri$HierarchicalUri");
		Constructor hierarchicalUriConstructor = hierarchicalUriClass.getDeclaredConstructors()[0];
		hierarchicalUriConstructor.setAccessible(true);
		return (Uri) hierarchicalUriConstructor.newInstance("https", authority, path, null, null);
		}

		/** Attempting to unparcel a legacy parcel format of Uri.{,Path}Part should fail. */
		public void testUnparcelLegacyPart_fails() throws Exception {
		assertUnparcelLegacyPart_fails(Class.forName("android.net.Uri$Part"));
		assertUnparcelLegacyPart_fails(Class.forName("android.net.Uri$PathPart"));
		}

		private static void assertUnparcelLegacyPart_fails(Class partClass) throws Exception {
		Parcel parcel = Parcel.obtain();
		parcel.writeInt(0 /* BOTH */);
		parcel.writeString("encoded");
		parcel.writeString("decoded");
		parcel.setDataPosition(0);

		Method readFromMethod = partClass.getDeclaredMethod("readFrom", Parcel.class);
		readFromMethod.setAccessible(true);
		try {
		readFromMethod.invoke(null, parcel);
		fail();
		} catch (InvocationTargetException expected) {
		Throwable targetException = expected.getTargetException();
		// Check that the exception was thrown for the correct reason.
		assertEquals("Unknown representation: 0", targetException.getMessage());
		}
		}

		public void testToSafeString() {
		checkToSafeString("tel:xxxxxx", "tel:Google");
		checkToSafeString("tel:xxxxxxxxxx", "tel:1234567890");

libs/androidfw/ResourceTypes.cpp

+1 −2

Original line number	Diff line number	Diff line
		@@ -6902,9 +6902,8 @@ std::unique_ptr<DynamicRefTable> DynamicRefTable::clone() const {
		status_t DynamicRefTable::load(const ResTable_lib_header* const header)
		{
		const uint32_t entryCount = dtohl(header->count);
		const uint32_t sizeOfEntries = sizeof(ResTable_lib_entry) * entryCount;
		const uint32_t expectedSize = dtohl(header->header.size) - dtohl(header->header.headerSize);
		if (sizeOfEntries > expectedSize) {
		if (entryCount > (expectedSize / sizeof(ResTable_lib_entry))) {
		ALOGE("ResTable_lib_header size %u is too small to fit %u entries (x %u).",
		expectedSize, entryCount, (uint32_t)sizeof(ResTable_lib_entry));
		return UNKNOWN_ERROR;

packages/SettingsLib/res/values-or/strings.xml

+1 −1

Original line number	Diff line number	Diff line
		@@ -412,7 +412,7 @@
		<string name="battery_info_status_full" msgid="4443168946046847468">"ଚାର୍ଜ ସମ୍ପୂର୍ଣ୍ଣ"</string>
		<string name="disabled_by_admin_summary_text" msgid="5343911767402923057">"ଆଡ୍‌ମିନ୍‌ ଦ୍ୱାରା ନିୟନ୍ତ୍ରିତ"</string>
		<string name="disabled" msgid="8017887509554714950">"ଅକ୍ଷମ ହୋଇଛି"</string>
		<string name="external_source_trusted" msgid="1146522036773132905">"ଅନୁମୋଦିତ"</string>
		<string name="external_source_trusted" msgid="1146522036773132905">"ଅନୁମତି ଦିଆଯାଇଛି"</string>
		<string name="external_source_untrusted" msgid="5037891688911672227">"ଅନୁମତି ନାହିଁ"</string>
		<string name="install_other_apps" msgid="3232595082023199454">"ଅଜଣା ଆପ୍‌ ଇନଷ୍ଟଲ୍‌ କରନ୍ତୁ"</string>
		<string name="home" msgid="973834627243661438">"ସେଟିଙ୍ଗ ହୋମ୍‌"</string>