Merge "IpManager: always log immediate provisioning errors"

am: 98f2decdad

Change-Id: I6d745a4d6e9b932c16fe54fdabe191962389882d
This commit is contained in:
Hugo Benichi
2017-06-26 11:57:17 +00:00
committed by android-build-merger

View File

@@ -714,7 +714,6 @@ public class IpManager extends StateMachine {
return shouldLog;
}
// TODO: Migrate all Log.e(...) to logError(...).
private void logError(String fmt, Object... args) {
final String msg = "ERROR " + String.format(fmt, args);
Log.e(mTag, msg);
@@ -1035,7 +1034,7 @@ public class IpManager extends StateMachine {
}
private void doImmediateProvisioningFailure(int failureType) {
if (DBG) { Log.e(mTag, "onProvisioningFailure(): " + failureType); }
logError("onProvisioningFailure(): %s", failureType);
recordMetric(failureType);
mCallback.onProvisioningFailure(new LinkProperties(mLinkProperties));
}
@@ -1170,7 +1169,7 @@ public class IpManager extends StateMachine {
case DhcpClient.CMD_ON_QUIT:
// Everything is already stopped.
Log.e(mTag, "Unexpected CMD_ON_QUIT (already stopped).");
logError("Unexpected CMD_ON_QUIT (already stopped).");
break;
default:
@@ -1376,7 +1375,7 @@ public class IpManager extends StateMachine {
break;
case CMD_START:
Log.e(mTag, "ALERT: START received in StartedState. Please fix caller.");
logError("ALERT: START received in StartedState. Please fix caller.");
break;
case CMD_CONFIRM:
@@ -1475,13 +1474,13 @@ public class IpManager extends StateMachine {
handleIPv4Failure();
break;
default:
Log.e(mTag, "Unknown CMD_POST_DHCP_ACTION status:" + msg.arg1);
logError("Unknown CMD_POST_DHCP_ACTION status: %s", msg.arg1);
}
break;
case DhcpClient.CMD_ON_QUIT:
// DHCPv4 quit early for some reason.
Log.e(mTag, "Unexpected CMD_ON_QUIT.");
logError("Unexpected CMD_ON_QUIT.");
mDhcpClient = null;
break;