Merge change 415 into donut
* changes: code cleanup
This commit is contained in:
@@ -64,7 +64,7 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
private static final String LOG_TAG = "CDMA";
|
private static final String LOG_TAG = "CDMA";
|
||||||
private static final boolean DBG = true;
|
private static final boolean DBG = true;
|
||||||
|
|
||||||
//***** Instance Variables
|
private CDMAPhone mCdmaPhone;
|
||||||
|
|
||||||
// Indicates baseband will not auto-attach
|
// Indicates baseband will not auto-attach
|
||||||
private boolean noAutoAttach = false;
|
private boolean noAutoAttach = false;
|
||||||
@@ -83,20 +83,19 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
private CdmaDataConnection mActiveDataConnection;
|
private CdmaDataConnection mActiveDataConnection;
|
||||||
|
|
||||||
/** Defined cdma connection profiles */
|
/** Defined cdma connection profiles */
|
||||||
private static int EXTERNAL_NETWORK_DEFAULT_ID = 0;
|
private static final int EXTERNAL_NETWORK_DEFAULT_ID = 0;
|
||||||
private static int EXTERNAL_NETWORK_NUM_TYPES = 1;
|
private static final int EXTERNAL_NETWORK_NUM_TYPES = 1;
|
||||||
|
|
||||||
private boolean[] dataEnabled = new boolean[EXTERNAL_NETWORK_NUM_TYPES];
|
private boolean[] dataEnabled = new boolean[EXTERNAL_NETWORK_NUM_TYPES];
|
||||||
|
|
||||||
//***** Constants
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Pool size of CdmaDataConnection objects.
|
* Pool size of CdmaDataConnection objects.
|
||||||
*/
|
*/
|
||||||
private static final int DATA_CONNECTION_POOL_SIZE = 1;
|
private static final int DATA_CONNECTION_POOL_SIZE = 1;
|
||||||
|
|
||||||
private static final int POLL_CONNECTION_MILLIS = 5 * 1000;
|
private static final int POLL_CONNECTION_MILLIS = 5 * 1000;
|
||||||
private static final String INTENT_RECONNECT_ALARM = "com.android.internal.telephony.cdma-reconnect";
|
private static final String INTENT_RECONNECT_ALARM =
|
||||||
|
"com.android.internal.telephony.cdma-reconnect";
|
||||||
private static final String INTENT_RECONNECT_ALARM_EXTRA_REASON = "reason";
|
private static final String INTENT_RECONNECT_ALARM_EXTRA_REASON = "reason";
|
||||||
|
|
||||||
// Possibly promoate to base class, the only difference is
|
// Possibly promoate to base class, the only difference is
|
||||||
@@ -146,6 +145,7 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
|
|
||||||
CdmaDataConnectionTracker(CDMAPhone p) {
|
CdmaDataConnectionTracker(CDMAPhone p) {
|
||||||
super(p);
|
super(p);
|
||||||
|
mCdmaPhone = p;
|
||||||
|
|
||||||
p.mCM.registerForAvailable (this, EVENT_RADIO_AVAILABLE, null);
|
p.mCM.registerForAvailable (this, EVENT_RADIO_AVAILABLE, null);
|
||||||
p.mCM.registerForOffOrNotAvailable(this, EVENT_RADIO_OFF_OR_NOT_AVAILABLE, null);
|
p.mCM.registerForOffOrNotAvailable(this, EVENT_RADIO_OFF_OR_NOT_AVAILABLE, null);
|
||||||
@@ -187,15 +187,15 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
//Unregister from all events
|
//Unregister from all events
|
||||||
phone.mCM.unregisterForAvailable(this);
|
phone.mCM.unregisterForAvailable(this);
|
||||||
phone.mCM.unregisterForOffOrNotAvailable(this);
|
phone.mCM.unregisterForOffOrNotAvailable(this);
|
||||||
((CDMAPhone) phone).mRuimRecords.unregisterForRecordsLoaded(this);
|
mCdmaPhone.mRuimRecords.unregisterForRecordsLoaded(this);
|
||||||
phone.mCM.unregisterForNVReady(this);
|
phone.mCM.unregisterForNVReady(this);
|
||||||
phone.mCM.unregisterForDataStateChanged(this);
|
phone.mCM.unregisterForDataStateChanged(this);
|
||||||
((CDMAPhone) phone).mCT.unregisterForVoiceCallEnded(this);
|
mCdmaPhone.mCT.unregisterForVoiceCallEnded(this);
|
||||||
((CDMAPhone) phone).mCT.unregisterForVoiceCallStarted(this);
|
mCdmaPhone.mCT.unregisterForVoiceCallStarted(this);
|
||||||
((CDMAPhone) phone).mSST.unregisterForCdmaDataConnectionAttached(this);
|
mCdmaPhone.mSST.unregisterForCdmaDataConnectionAttached(this);
|
||||||
((CDMAPhone) phone).mSST.unregisterForCdmaDataConnectionDetached(this);
|
mCdmaPhone.mSST.unregisterForCdmaDataConnectionDetached(this);
|
||||||
((CDMAPhone) phone).mSST.unregisterForRoamingOn(this);
|
mCdmaPhone.mSST.unregisterForRoamingOn(this);
|
||||||
((CDMAPhone) phone).mSST.unregisterForRoamingOff(this);
|
mCdmaPhone.mSST.unregisterForRoamingOff(this);
|
||||||
|
|
||||||
phone.getContext().unregisterReceiver(this.mIntentReceiver);
|
phone.getContext().unregisterReceiver(this.mIntentReceiver);
|
||||||
destroyAllDataConnectionList();
|
destroyAllDataConnectionList();
|
||||||
@@ -277,10 +277,10 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
public boolean isDataConnectionAsDesired() {
|
public boolean isDataConnectionAsDesired() {
|
||||||
boolean roaming = phone.getServiceState().getRoaming();
|
boolean roaming = phone.getServiceState().getRoaming();
|
||||||
|
|
||||||
if ( ((phone.mCM.getRadioState() == CommandsInterface.RadioState.NV_READY) ||
|
if (((phone.mCM.getRadioState() == CommandsInterface.RadioState.NV_READY) ||
|
||||||
((CDMAPhone) phone).mRuimRecords.getRecordsLoaded()) &&
|
mCdmaPhone.mRuimRecords.getRecordsLoaded()) &&
|
||||||
((CDMAPhone) phone).mSST.getCurrentCdmaDataConnectionState() ==
|
(mCdmaPhone.mSST.getCurrentCdmaDataConnectionState() ==
|
||||||
ServiceState.STATE_IN_SERVICE &&
|
ServiceState.STATE_IN_SERVICE) &&
|
||||||
(!roaming || getDataOnRoamingEnabled()) &&
|
(!roaming || getDataOnRoamingEnabled()) &&
|
||||||
!mIsWifiConnected ) {
|
!mIsWifiConnected ) {
|
||||||
return (state == State.CONNECTED);
|
return (state == State.CONNECTED);
|
||||||
@@ -353,7 +353,7 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
int psState = ((CDMAPhone) phone).mSST.getCurrentCdmaDataConnectionState();
|
int psState = mCdmaPhone.mSST.getCurrentCdmaDataConnectionState();
|
||||||
boolean roaming = phone.getServiceState().getRoaming();
|
boolean roaming = phone.getServiceState().getRoaming();
|
||||||
|
|
||||||
if ((state == State.IDLE || state == State.SCANNING)
|
if ((state == State.IDLE || state == State.SCANNING)
|
||||||
@@ -361,9 +361,9 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
psState == ServiceState.RADIO_TECHNOLOGY_EVDO_0 ||
|
psState == ServiceState.RADIO_TECHNOLOGY_EVDO_0 ||
|
||||||
psState == ServiceState.RADIO_TECHNOLOGY_EVDO_A)
|
psState == ServiceState.RADIO_TECHNOLOGY_EVDO_A)
|
||||||
&& ((phone.mCM.getRadioState() == CommandsInterface.RadioState.NV_READY) ||
|
&& ((phone.mCM.getRadioState() == CommandsInterface.RadioState.NV_READY) ||
|
||||||
((CDMAPhone) phone).mRuimRecords.getRecordsLoaded())
|
mCdmaPhone.mRuimRecords.getRecordsLoaded())
|
||||||
&& (((CDMAPhone) phone).mSST.isConcurrentVoiceAndData() ||
|
&& (mCdmaPhone.mSST.isConcurrentVoiceAndData() ||
|
||||||
phone.getState() == Phone.State.IDLE )
|
phone.getState() == Phone.State.IDLE )
|
||||||
&& isDataAllowed()) {
|
&& isDataAllowed()) {
|
||||||
|
|
||||||
return setupData(reason);
|
return setupData(reason);
|
||||||
@@ -374,8 +374,8 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
" dataState=" + state +
|
" dataState=" + state +
|
||||||
" PS state=" + psState +
|
" PS state=" + psState +
|
||||||
" radio state=" + phone.mCM.getRadioState() +
|
" radio state=" + phone.mCM.getRadioState() +
|
||||||
" ruim=" + ((CDMAPhone) phone).mRuimRecords.getRecordsLoaded() +
|
" ruim=" + mCdmaPhone.mRuimRecords.getRecordsLoaded() +
|
||||||
" concurrentVoice&Data=" + ((CDMAPhone) phone).mSST.isConcurrentVoiceAndData() +
|
" concurrentVoice&Data=" + mCdmaPhone.mSST.isConcurrentVoiceAndData() +
|
||||||
" phoneState=" + phone.getState() +
|
" phoneState=" + phone.getState() +
|
||||||
" dataEnabled=" + getAnyDataEnabled() +
|
" dataEnabled=" + getAnyDataEnabled() +
|
||||||
" roaming=" + roaming +
|
" roaming=" + roaming +
|
||||||
@@ -781,7 +781,7 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
* @override com.android.internal.telephony.DataConnectionTracker
|
* @override com.android.internal.telephony.DataConnectionTracker
|
||||||
*/
|
*/
|
||||||
protected void onVoiceCallStarted() {
|
protected void onVoiceCallStarted() {
|
||||||
if (state == State.CONNECTED && !((CDMAPhone) phone).mSST.isConcurrentVoiceAndData()) {
|
if (state == State.CONNECTED && !mCdmaPhone.mSST.isConcurrentVoiceAndData()) {
|
||||||
stopNetStatPoll();
|
stopNetStatPoll();
|
||||||
phone.notifyDataConnection(Phone.REASON_VOICE_CALL_STARTED);
|
phone.notifyDataConnection(Phone.REASON_VOICE_CALL_STARTED);
|
||||||
}
|
}
|
||||||
@@ -792,7 +792,7 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
*/
|
*/
|
||||||
protected void onVoiceCallEnded() {
|
protected void onVoiceCallEnded() {
|
||||||
if (state == State.CONNECTED) {
|
if (state == State.CONNECTED) {
|
||||||
if (!((CDMAPhone) phone).mSST.isConcurrentVoiceAndData()) {
|
if (!mCdmaPhone.mSST.isConcurrentVoiceAndData()) {
|
||||||
startNetStatPoll();
|
startNetStatPoll();
|
||||||
phone.notifyDataConnection(Phone.REASON_VOICE_CALL_ENDED);
|
phone.notifyDataConnection(Phone.REASON_VOICE_CALL_ENDED);
|
||||||
} else {
|
} else {
|
||||||
@@ -818,7 +818,7 @@ public final class CdmaDataConnectionTracker extends DataConnectionTracker {
|
|||||||
CdmaDataConnection dataConn;
|
CdmaDataConnection dataConn;
|
||||||
|
|
||||||
for (int i = 0; i < DATA_CONNECTION_POOL_SIZE; i++) {
|
for (int i = 0; i < DATA_CONNECTION_POOL_SIZE; i++) {
|
||||||
dataConn = new CdmaDataConnection(((CDMAPhone) phone));
|
dataConn = new CdmaDataConnection(mCdmaPhone);
|
||||||
dataConnectionList.add(dataConn);
|
dataConnectionList.add(dataConn);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user