am 9b650702: Merge "Fix state transition verification." into klp-dev

* commit '9b650702113f8a17e09940c3802de71a9fc62ff1':
  Fix state transition verification.
This commit is contained in:
Xia Wang
2014-03-18 22:16:26 +00:00
committed by Android Git Automerger

View File

@@ -101,9 +101,10 @@ public class NetworkState {
}
/*
* Transition from CONNECTED -> DISCONNECTED:
* CONNECTED->DISCONNECTING->DISCONNECTED
* return false if any state transition is not valid and save a message in mReason
* Verifies state transition from CONNECTED->...-> DISCONNECTED.
*
* returns false if initial state or target state is not correct, or if there is
* any transition from DISCONNECTING/DISCONNECTED -> CONNECTED.
*/
public boolean transitToDisconnection () {
mReason = "states: " + printStates();
@@ -120,13 +121,13 @@ public class NetworkState {
for (int i = 1; i < mStateDepository.size() - 1; i++) {
State preState = mStateDepository.get(i-1);
State curState = mStateDepository.get(i);
if ((preState == State.CONNECTED) && ((curState == State.DISCONNECTING) ||
if (preState == curState) {
continue;
} else if ((preState == State.CONNECTED) && ((curState == State.DISCONNECTING) ||
(curState == State.DISCONNECTED))) {
continue;
} else if ((preState == State.DISCONNECTING) && (curState == State.DISCONNECTED)) {
continue;
} else if ((preState == State.DISCONNECTED) && (curState == State.DISCONNECTED)) {
continue;
} else {
mReason += " Transition state from " + preState.toString() + " to " +
curState.toString() + " is not valid.";
@@ -136,7 +137,12 @@ public class NetworkState {
return true;
}
// DISCONNECTED->CONNECTING->CONNECTED
/*
* Verifies state transition from DISCONNECTED->...-> CONNECTED.
*
* returns false if initial state or target state is not correct, or if there is
* any transition from CONNECED -> DISCONNECTED.
*/
public boolean transitToConnection() {
mReason = "states: " + printStates();
if (mStateDepository.get(0) != State.DISCONNECTED) {
@@ -152,14 +158,15 @@ public class NetworkState {
for (int i = 1; i < mStateDepository.size(); i++) {
State preState = mStateDepository.get(i-1);
State curState = mStateDepository.get(i);
if (preState == curState) {
continue;
}
if ((preState == State.DISCONNECTED) && ((curState == State.CONNECTING) ||
(curState == State.CONNECTED) || (curState == State.DISCONNECTED))) {
(curState == State.CONNECTED))) {
continue;
} else if ((preState == State.CONNECTING) && (curState == State.CONNECTED)) {
continue;
} else if ((preState == State.CONNECTED) && (curState == State.CONNECTED)) {
continue;
} else {
} else if ((preState == State.CONNECTING) && (curState == State.CONNECTED)) {
continue;
} else {
mReason += " Transition state from " + preState.toString() + " to " +
curState.toString() + " is not valid.";
return false;