From a8b76eafc419eda662689b9f88ef9dc5fe4af4c0 Mon Sep 17 00:00:00 2001 From: Taesu Lee Date: Wed, 5 Feb 2020 13:06:34 +0900 Subject: [PATCH] Allow acceptable media types only for sharing Test: Manual Change-Id: I20d5f8abf2ba7b7b620aa2a4ec1e0a43091f98c2 Signed-off-by: Taesu Lee --- .../android/messaging/datamodel/data/MessagePartData.java | 5 +++++ .../messaging/ui/conversationlist/ShareIntentActivity.java | 4 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/com/android/messaging/datamodel/data/MessagePartData.java b/src/com/android/messaging/datamodel/data/MessagePartData.java index 5846c97..ea24876 100644 --- a/src/com/android/messaging/datamodel/data/MessagePartData.java +++ b/src/com/android/messaging/datamodel/data/MessagePartData.java @@ -341,6 +341,11 @@ public class MessagePartData implements Parcelable { return mHeight; } + public static boolean isSupportedMediaType(final String contentType) { + return ContentType.isVCardType(contentType) + || Arrays.asList(ACCEPTABLE_GALLERY_MEDIA_TYPES).contains(contentType); + } + /** * * @return true if this part can only exist by itself, with no other attachments diff --git a/src/com/android/messaging/ui/conversationlist/ShareIntentActivity.java b/src/com/android/messaging/ui/conversationlist/ShareIntentActivity.java index 25d5ea3..412177e 100644 --- a/src/com/android/messaging/ui/conversationlist/ShareIntentActivity.java +++ b/src/com/android/messaging/ui/conversationlist/ShareIntentActivity.java @@ -103,7 +103,7 @@ public class ShareIntentActivity extends BaseBugleActivity implements } mDraftMessage = sharedText != null ? MessageData.createSharedMessage(sharedText) : null; - } else if (ContentType.isMediaType(contentType)) { + } else if (PendingAttachmentData.isSupportedMediaType(contentType)) { if (contentUri != null) { mDraftMessage = MessageData.createSharedMessage(null); addSharedPartToDraft(contentType, contentUri); @@ -139,7 +139,7 @@ public class ShareIntentActivity extends BaseBugleActivity implements } strBuffer.append(sharedText); } - } else if (ContentType.isMediaType(actualContentType)) { + } else if (PendingAttachmentData.isSupportedMediaType(actualContentType)) { uriMap.put(uri, actualContentType); } else { // Unsupported content type.