[automerger] RESTRICT AUTOMERGE: Messaging ignores file URIs shared via intent am: 0ee4c0e33b am: a04af950ce am: b40c396122
Change-Id: Ib4958d367e4e2297154af56b477121f1c8420746
This commit is contained in:
@@ -34,6 +34,8 @@ import com.android.messaging.util.Assert;
|
|||||||
import com.android.messaging.util.ContentType;
|
import com.android.messaging.util.ContentType;
|
||||||
import com.android.messaging.util.LogUtil;
|
import com.android.messaging.util.LogUtil;
|
||||||
import com.android.messaging.util.MediaMetadataRetrieverWrapper;
|
import com.android.messaging.util.MediaMetadataRetrieverWrapper;
|
||||||
|
import com.android.messaging.util.FileUtil;
|
||||||
|
import com.android.messaging.util.UriUtil;
|
||||||
|
|
||||||
import java.io.IOException;
|
import java.io.IOException;
|
||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
@@ -74,6 +76,12 @@ public class ShareIntentActivity extends BaseBugleActivity implements
|
|||||||
final String action = intent.getAction();
|
final String action = intent.getAction();
|
||||||
if (Intent.ACTION_SEND.equals(action)) {
|
if (Intent.ACTION_SEND.equals(action)) {
|
||||||
final Uri contentUri = (Uri) intent.getParcelableExtra(Intent.EXTRA_STREAM);
|
final Uri contentUri = (Uri) intent.getParcelableExtra(Intent.EXTRA_STREAM);
|
||||||
|
if (UriUtil.isFileUri(contentUri)) {
|
||||||
|
LogUtil.i(
|
||||||
|
LogUtil.BUGLE_TAG,
|
||||||
|
"Ignoring attachment from file URI which are no longer supported.");
|
||||||
|
return;
|
||||||
|
}
|
||||||
final String contentType = extractContentType(contentUri, intent.getType());
|
final String contentType = extractContentType(contentUri, intent.getType());
|
||||||
if (LogUtil.isLoggable(LogUtil.BUGLE_TAG, LogUtil.DEBUG)) {
|
if (LogUtil.isLoggable(LogUtil.BUGLE_TAG, LogUtil.DEBUG)) {
|
||||||
LogUtil.d(LogUtil.BUGLE_TAG, String.format(
|
LogUtil.d(LogUtil.BUGLE_TAG, String.format(
|
||||||
@@ -111,6 +119,12 @@ public class ShareIntentActivity extends BaseBugleActivity implements
|
|||||||
if (imageUris != null && imageUris.size() > 0) {
|
if (imageUris != null && imageUris.size() > 0) {
|
||||||
mDraftMessage = MessageData.createSharedMessage(null);
|
mDraftMessage = MessageData.createSharedMessage(null);
|
||||||
for (final Uri imageUri : imageUris) {
|
for (final Uri imageUri : imageUris) {
|
||||||
|
if (UriUtil.isFileUri(imageUri)) {
|
||||||
|
LogUtil.i(
|
||||||
|
LogUtil.BUGLE_TAG,
|
||||||
|
"Ignoring attachment from file URI which are no longer supported.");
|
||||||
|
continue;
|
||||||
|
}
|
||||||
final String actualContentType = extractContentType(imageUri, contentType);
|
final String actualContentType = extractContentType(imageUri, contentType);
|
||||||
addSharedImagePartToDraft(actualContentType, imageUri);
|
addSharedImagePartToDraft(actualContentType, imageUri);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -22,12 +22,15 @@ import android.app.FragmentManager;
|
|||||||
import android.app.FragmentTransaction;
|
import android.app.FragmentTransaction;
|
||||||
import android.content.Context;
|
import android.content.Context;
|
||||||
import android.content.DialogInterface;
|
import android.content.DialogInterface;
|
||||||
|
import android.content.Intent;
|
||||||
import android.media.MediaPlayer;
|
import android.media.MediaPlayer;
|
||||||
|
import android.net.Uri;
|
||||||
import android.os.Environment;
|
import android.os.Environment;
|
||||||
import android.telephony.SmsMessage;
|
import android.telephony.SmsMessage;
|
||||||
import android.text.TextUtils;
|
import android.text.TextUtils;
|
||||||
import android.widget.ArrayAdapter;
|
import android.widget.ArrayAdapter;
|
||||||
|
|
||||||
|
import com.android.messaging.Factory;
|
||||||
import com.android.messaging.R;
|
import com.android.messaging.R;
|
||||||
import com.android.messaging.datamodel.SyncManager;
|
import com.android.messaging.datamodel.SyncManager;
|
||||||
import com.android.messaging.datamodel.action.DumpDatabaseAction;
|
import com.android.messaging.datamodel.action.DumpDatabaseAction;
|
||||||
@@ -179,6 +182,13 @@ public class DebugUtils {
|
|||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
|
arrayAdapter.add(new DebugAction("Test sharing a file URI") {
|
||||||
|
@Override
|
||||||
|
public void run() {
|
||||||
|
shareFileUri();
|
||||||
|
}
|
||||||
|
});
|
||||||
|
|
||||||
builder.setAdapter(arrayAdapter,
|
builder.setAdapter(arrayAdapter,
|
||||||
new android.content.DialogInterface.OnClickListener() {
|
new android.content.DialogInterface.OnClickListener() {
|
||||||
@Override
|
@Override
|
||||||
@@ -422,4 +432,17 @@ public class DebugUtils {
|
|||||||
public static boolean debugClassZeroSmsEnabled() {
|
public static boolean debugClassZeroSmsEnabled() {
|
||||||
return sDebugClassZeroSms;
|
return sDebugClassZeroSms;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/** Shares a ringtone file via file URI. */
|
||||||
|
private static void shareFileUri() {
|
||||||
|
final String packageName = "com.android.messaging";
|
||||||
|
final String fileName = "/system/media/audio/ringtones/Andromeda.ogg";
|
||||||
|
|
||||||
|
Intent intent = new Intent(Intent.ACTION_SEND);
|
||||||
|
intent.setPackage(packageName);
|
||||||
|
intent.putExtra(Intent.EXTRA_STREAM, Uri.parse("file://" + fileName));
|
||||||
|
intent.setType("image/*");
|
||||||
|
intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
|
||||||
|
Factory.get().getApplicationContext().startActivity(intent);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user