am 2adbc7f7: Backport l53e91db7 from master.

Merge commit '2adbc7f7a4a4a827360f8a585e7306f07f2fd718' into eclair-plus-aosp

* commit '2adbc7f7a4a4a827360f8a585e7306f07f2fd718':
  Backport l53e91db7 from master.
This commit is contained in:
Robert Greenwalt
2010-01-13 17:09:40 -08:00
committed by Android Git Automerger

View File

@@ -111,6 +111,7 @@ public class ConnectivityService extends IConnectivityManager.Stub {
public int mType; public int mType;
public int mRadio; public int mRadio;
public int mPriority; public int mPriority;
public NetworkInfo.State mLastState;
public NetworkAttributes(String init) { public NetworkAttributes(String init) {
String fragments[] = init.split(","); String fragments[] = init.split(",");
mName = fragments[0].toLowerCase(); mName = fragments[0].toLowerCase();
@@ -131,6 +132,7 @@ public class ConnectivityService extends IConnectivityManager.Stub {
mType = ConnectivityManager.TYPE_MOBILE_HIPRI; mType = ConnectivityManager.TYPE_MOBILE_HIPRI;
} }
mPriority = Integer.parseInt(fragments[2]); mPriority = Integer.parseInt(fragments[2]);
mLastState = NetworkInfo.State.UNKNOWN;
} }
public boolean isDefault() { public boolean isDefault() {
return (mType == mRadio); return (mType == mRadio);
@@ -1214,9 +1216,22 @@ public class ConnectivityService extends IConnectivityManager.Stub {
switch (msg.what) { switch (msg.what) {
case NetworkStateTracker.EVENT_STATE_CHANGED: case NetworkStateTracker.EVENT_STATE_CHANGED:
info = (NetworkInfo) msg.obj; info = (NetworkInfo) msg.obj;
int type = info.getType();
NetworkInfo.State state = info.getState();
if (mNetAttributes[type].mLastState == state) {
if (DBG) {
// TODO - remove this after we validate the dropping doesn't break anything
Log.d(TAG, "Dropping ConnectivityChange for " +
info.getTypeName() + ": " +
state + "/" + info.getDetailedState());
}
return;
}
mNetAttributes[type].mLastState = state;
if (DBG) Log.d(TAG, "ConnectivityChange for " + if (DBG) Log.d(TAG, "ConnectivityChange for " +
info.getTypeName() + ": " + info.getTypeName() + ": " +
info.getState() + "/" + info.getDetailedState()); state + "/" + info.getDetailedState());
// Connectivity state changed: // Connectivity state changed:
// [31-13] Reserved for future use // [31-13] Reserved for future use
@@ -1234,10 +1249,9 @@ public class ConnectivityService extends IConnectivityManager.Stub {
if (info.getDetailedState() == if (info.getDetailedState() ==
NetworkInfo.DetailedState.FAILED) { NetworkInfo.DetailedState.FAILED) {
handleConnectionFailure(info); handleConnectionFailure(info);
} else if (info.getState() == } else if (state == NetworkInfo.State.DISCONNECTED) {
NetworkInfo.State.DISCONNECTED) {
handleDisconnect(info); handleDisconnect(info);
} else if (info.getState() == NetworkInfo.State.SUSPENDED) { } else if (state == NetworkInfo.State.SUSPENDED) {
// TODO: need to think this over. // TODO: need to think this over.
// the logic here is, handle SUSPENDED the same as // the logic here is, handle SUSPENDED the same as
// DISCONNECTED. The only difference being we are // DISCONNECTED. The only difference being we are
@@ -1246,7 +1260,7 @@ public class ConnectivityService extends IConnectivityManager.Stub {
// opportunity to handle DISCONNECTED and SUSPENDED // opportunity to handle DISCONNECTED and SUSPENDED
// differently, or not. // differently, or not.
handleDisconnect(info); handleDisconnect(info);
} else if (info.getState() == NetworkInfo.State.CONNECTED) { } else if (state == NetworkInfo.State.CONNECTED) {
handleConnect(info); handleConnect(info);
} }
break; break;