am 442b08bc: Merge "Cleanup all functions overridden from DCT in GDCT" into honeycomb-LTE

* commit '442b08bc7c934f601a111bf97ea96842d69cdc23':
  Cleanup all functions overridden from DCT in GDCT
This commit is contained in:
Wink Saville
2011-04-09 07:57:35 -07:00
committed by Android Git Automerger
2 changed files with 30 additions and 40 deletions

View File

@@ -949,10 +949,6 @@ public abstract class DataConnectionTracker extends Handler {
}
}
public synchronized boolean getDataEnabled() {
return mDataEnabled;
}
public void cleanUpAllConnections(String cause) {
Message msg = obtainMessage(EVENT_CLEAN_UP_ALL_CONNECTIONS);
msg.obj = cause;

View File

@@ -67,7 +67,6 @@ import java.net.InetSocketAddress;
import java.net.UnknownHostException;
import java.util.ArrayList;
import java.util.concurrent.ConcurrentHashMap;
import java.util.Iterator;
import java.util.Map;
import java.util.HashMap;
@@ -296,25 +295,29 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
@Override
protected LinkProperties getLinkProperties(String apnType) {
ApnContext apnContext = mApnContexts.get(apnType);
if (apnContext != null && apnContext.getDataConnection() != null) {
if (DBG) log("get active pdp is not null, return link properites for " + apnType);
return apnContext.getDataConnection().getLinkProperties();
} else {
if (DBG) log("return new LinkProperties");
return new LinkProperties();
if (apnContext != null) {
DataConnection dataConnection = apnContext.getDataConnection();
if (dataConnection != null) {
if (DBG) log("get active pdp is not null, return link properites for " + apnType);
return dataConnection.getLinkProperties();
}
}
if (DBG) log("return new LinkProperties");
return new LinkProperties();
}
@Override
protected LinkCapabilities getLinkCapabilities(String apnType) {
ApnContext apnContext = mApnContexts.get(apnType);
if (apnContext!=null && apnContext.getDataConnection() != null) {
if (DBG) log("get active pdp is not null, return link Capabilities for " + apnType);
return apnContext.getDataConnection().getLinkCapabilities();
} else {
if (DBG) log("return new LinkCapabilities");
return new LinkCapabilities();
if (apnContext!=null) {
DataConnection dataConnection = apnContext.getDataConnection();
if (dataConnection != null) {
if (DBG) log("get active pdp is not null, return link Capabilities for " + apnType);
return dataConnection.getLinkCapabilities();
}
}
if (DBG) log("return new LinkCapabilities");
return new LinkCapabilities();
}
@Override
@@ -323,9 +326,7 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
if (DBG) log("get all active apn types");
ArrayList<String> result = new ArrayList<String>();
Iterator<ApnContext> it = mApnContexts.values().iterator();
while (it.hasNext()) {
ApnContext apnContext = it.next();
for (ApnContext apnContext : mApnContexts.values()) {
if (apnContext.isReady()) {
result.add(apnContext.getApnType());
}
@@ -351,11 +352,14 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
}
// Return active apn of specific apn type
public synchronized String getActiveApnString(String apnType) {
public String getActiveApnString(String apnType) {
if (DBG) log( "get active apn string for type:" + apnType);
ApnContext apnContext = mApnContexts.get(apnType);
if (apnContext != null && apnContext.getApnSetting() != null) {
return apnContext.getApnSetting().apn;
if (apnContext != null) {
ApnSetting apnSetting = apnContext.getApnSetting();
if (apnSetting != null) {
return apnSetting.apn;
}
}
return null;
}
@@ -367,7 +371,7 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
// Return state of specific apn type
@Override
public synchronized State getState(String apnType) {
public State getState(String apnType) {
ApnContext apnContext = mApnContexts.get(apnType);
if (apnContext != null) {
return apnContext.getState();
@@ -378,9 +382,7 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
// Return state of overall
public State getOverallState() {
boolean isConnecting = false;
Iterator<ApnContext> it = mApnContexts.values().iterator();
while (it.hasNext()) {
ApnContext apnContext = it.next();
for (ApnContext apnContext : mApnContexts.values()) {
if (apnContext.getState() == State.CONNECTED ||
apnContext.getState() == State.DISCONNECTING) {
if (DBG) log("overall state is CONNECTED");
@@ -713,10 +715,8 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
protected void cleanUpAllConnections(boolean tearDown, String reason) {
if (DBG) log("Clean up all connections due to " + reason);
Iterator<ApnContext> it = mApnContexts.values().iterator();
while (it.hasNext()) {
ApnContext apnContext = it.next();
apnContext.setReason(reason);
for (ApnContext apnContext : mApnContexts.values()) {
apnContext.setReason(reason);
cleanUpConnection(tearDown, apnContext);
}
@@ -970,9 +970,7 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
return;
}
Iterator<ApnContext> it = mApnContexts.values().iterator();
while (it.hasNext()) {
ApnContext apnContext = it.next();
for (ApnContext apnContext : mApnContexts.values()) {
onDataStateChanged(dataCallStates, explicitPoll, apnContext);
}
}
@@ -1655,9 +1653,7 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
}
protected boolean isConnected() {
Iterator<ApnContext> it = mApnContexts.values().iterator();
while (it.hasNext()) {
ApnContext apnContext = it.next();
for (ApnContext apnContext : mApnContexts.values()) {
if (apnContext.getState() == State.CONNECTED) {
return true;
}
@@ -1668,9 +1664,7 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
@Override
protected void notifyDataConnection(String reason) {
if (DBG) log("notify all enabled connection for:" + reason);
Iterator<ApnContext> it = mApnContexts.values().iterator();
while (it.hasNext()) {
ApnContext apnContext = it.next();
for (ApnContext apnContext : mApnContexts.values()) {
if (apnContext.isReady()) {
if (DBG) log("notify for type:"+apnContext.getApnType());
mPhone.notifyDataConnection(reason != null ? reason : apnContext.getReason(),