Merge "do not process content uri in call Intents" (df396d06) · Commits · e / os / android_packages_services_Telecomm

src/com/android/server/telecom/NewOutgoingCallIntentBroadcaster.java

+12 −2

Original line number	Diff line number	Diff line
		@@ -314,8 +314,18 @@ public class NewOutgoingCallIntentBroadcaster {
		}

		private String getNumberFromCallIntent(Intent intent) {
		String number;
		number = mPhoneNumberUtilsAdapter.getNumberFromIntent(intent, mContext);
		String number = null;

		Uri uri = intent.getData();
		if (uri != null) {
		String scheme = uri.getScheme();
		if (scheme != null) {
		if (scheme.equals("tel") \|\| scheme.equals("sip")) {
		number = uri.getSchemeSpecificPart();
		}
		}
		}

		if (TextUtils.isEmpty(number)) {
		Log.w(this, "Empty number obtained from the call intent.");
		return null;

+13 −0

Original line number	Diff line number	Diff line
		@@ -214,6 +214,19 @@ public class NewOutgoingCallIntentBroadcasterTest extends TelecomTestCase {
		verifyNoCallPlaced();
		}

		@Test
		public void testNoCallsPlacedWithContentUri() {
		Uri handle = Uri.parse("content://com.android.contacts/data/1");
		Intent intent = new Intent(Intent.ACTION_CALL, handle);

		int result = processIntent(intent, true).disconnectCause;

		assertEquals(DisconnectCause.NO_PHONE_NUMBER_SUPPLIED, result);
		verify(mContext, never()).getContentResolver();
		verifyNoBroadcastSent();
		verifyNoCallPlaced();
		}

		@SmallTest
		@Test
		public void testEmergencyCallWithNonDefaultDialer() {