Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Be careful sharing connections when dun is in play" into ics-mr1

Browse Source
This commit is contained in:
Robert Greenwalt
2011-12-02 16:52:29 -08:00
committed by Android (Google) Code Review
parent 2f1e1e4ab9 7cb88dda5b
commit 2a7b5ab16a
2 changed files with 22 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
telephony/java/com/android/internal/telephony/ApnSetting.java
View File

@@ -189,4 +189,11 @@ public class ApnSetting {
}
return false;
}
// TODO - if we have this function we should also have hashCode.
// Also should handle changes in type order and perhaps case-insensitivity
public boolean equals(Object o) {
if (o instanceof ApnSetting == false) return false;
return (this.toString().equals(o.toString()));
}
}

16
telephony/java/com/android/internal/telephony/gsm/GsmDataConnectionTracker.java
View File

@@ -1720,11 +1720,25 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
private DataConnection checkForConnectionForApnContext(ApnContext apnContext) {
// Loop through all apnContexts looking for one with a conn that satisfies this apnType
String apnType = apnContext.getApnType();
ApnSetting dunSetting = null;
if (Phone.APN_TYPE_DUN.equals(apnType)) {
dunSetting = fetchDunApn();
}
for (ApnContext c : mApnContexts.values()) {
DataConnection conn = c.getDataConnection();
if (conn != null) {
ApnSetting apnSetting = c.getApnSetting();
if (apnSetting != null && apnSetting.canHandleType(apnType)) {
if (dunSetting != null) {
if (dunSetting.equals(apnSetting)) {
if (DBG) {
log("checkForConnectionForApnContext: apnContext=" + apnContext +
" found conn=" + conn);
}
return conn;
}
} else if (apnSetting != null && apnSetting.canHandleType(apnType)) {
if (DBG) {
log("checkForConnectionForApnContext: apnContext=" + apnContext +
" found conn=" + conn);