summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYe Wen <ywen@google.com>2015-02-27 14:21:18 -0800
committerYe Wen <ywen@google.com>2015-03-26 21:08:29 +0000
commita7e39b3c8ef72dc25debbdab91629fcf342c2b15 (patch)
treef57a6795e3d0e5f55357110618c343077d9809b2
parent189322a58f0b9825b515cadd4c270d870ad39da2 (diff)
downloadMms-a7e39b3c8ef72dc25debbdab91629fcf342c2b15.tar.gz
Remove airplane mode and mobile data check for MMS request execution
The check prevents MMS from being sent if wifi or other channel can be used to send MMS. When there is an API in telephony allowing the check of available MMS channels, the check should be added back b/19519883 b/19950603 Change-Id: Ie90a4f3eabf3155159dd307bd613f669cda88ac6 (cherry picked from commit e03feaba8ca6d25163ff19e433a3a3c460e834e8)
-rw-r--r--src/com/android/mms/service/MmsRequest.java20
1 files changed, 2 insertions, 18 deletions
diff --git a/src/com/android/mms/service/MmsRequest.java b/src/com/android/mms/service/MmsRequest.java
index 00aea57..27cb968 100644
--- a/src/com/android/mms/service/MmsRequest.java
+++ b/src/com/android/mms/service/MmsRequest.java
@@ -107,21 +107,6 @@ public abstract class MmsRequest {
return mMmsConfig != null;
}
- private static boolean inAirplaneMode(final Context context) {
- return Settings.System.getInt(context.getContentResolver(),
- Settings.Global.AIRPLANE_MODE_ON, 0) != 0;
- }
-
- private static boolean isMobileDataEnabled(final Context context, final int subId) {
- final TelephonyManager telephonyManager =
- (TelephonyManager) context.getSystemService(Context.TELEPHONY_SERVICE);
- return telephonyManager.getDataEnabled(subId);
- }
-
- private static boolean isDataNetworkAvailable(final Context context, final int subId) {
- return !inAirplaneMode(context) && isMobileDataEnabled(context, subId);
- }
-
/**
* Execute the request
*
@@ -132,15 +117,14 @@ public abstract class MmsRequest {
int result = SmsManager.MMS_ERROR_UNSPECIFIED;
int httpStatusCode = 0;
byte[] response = null;
+ // TODO: add mms data channel check back to fast fail if no way to send mms,
+ // when telephony provides such API.
if (!ensureMmsConfigLoaded()) { // Check mms config
Log.e(MmsService.TAG, "MmsRequest: mms config is not loaded yet");
result = SmsManager.MMS_ERROR_CONFIGURATION_ERROR;
} else if (!prepareForHttpRequest()) { // Prepare request, like reading pdu data from user
Log.e(MmsService.TAG, "MmsRequest: failed to prepare for request");
result = SmsManager.MMS_ERROR_IO_ERROR;
- } else if (!isDataNetworkAvailable(context, mSubId)) {
- Log.e(MmsService.TAG, "MmsRequest: in airplane mode or mobile data disabled");
- result = SmsManager.MMS_ERROR_NO_DATA_NETWORK;
} else { // Execute
long retryDelaySecs = 2;
// Try multiple times of MMS HTTP request