Merge "Move EventLogTags.writeDeviceIdle calls outside synchronized block." into nyc-mr1-dev

This commit is contained in:
TreeHugger Robot
2016-09-09 02:15:38 +00:00
committed by Android (Google) Code Review
2 changed files with 23 additions and 23 deletions

View File

@@ -2217,23 +2217,23 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub {
@Override
public void setDeviceIdleMode(boolean enabled) {
mContext.enforceCallingOrSelfPermission(MANAGE_NETWORK_POLICY, TAG);
Trace.traceBegin(Trace.TRACE_TAG_NETWORK, "setDeviceIdleMode");
try {
synchronized (mUidRulesFirstLock) {
if (mDeviceIdleMode != enabled) {
mDeviceIdleMode = enabled;
if (mSystemReady) {
// Device idle change means we need to rebuild rules for all
// known apps, so do a global refresh.
updateRulesForRestrictPowerUL();
}
if (enabled) {
EventLogTags.writeDeviceIdleOnPhase("net");
} else {
EventLogTags.writeDeviceIdleOffPhase("net");
}
if (mDeviceIdleMode == enabled) {
return;
}
mDeviceIdleMode = enabled;
if (mSystemReady) {
// Device idle change means we need to rebuild rules for all
// known apps, so do a global refresh.
updateRulesForRestrictPowerUL();
}
}
if (enabled) {
EventLogTags.writeDeviceIdleOnPhase("net");
} else {
EventLogTags.writeDeviceIdleOffPhase("net");
}
} finally {
Trace.traceEnd(Trace.TRACE_TAG_NETWORK);

View File

@@ -2538,18 +2538,18 @@ public final class PowerManagerService extends SystemService
boolean setDeviceIdleModeInternal(boolean enabled) {
synchronized (mLock) {
if (mDeviceIdleMode != enabled) {
mDeviceIdleMode = enabled;
updateWakeLockDisabledStatesLocked();
if (enabled) {
EventLogTags.writeDeviceIdleOnPhase("power");
} else {
EventLogTags.writeDeviceIdleOffPhase("power");
}
return true;
if (mDeviceIdleMode == enabled) {
return false;
}
return false;
mDeviceIdleMode = enabled;
updateWakeLockDisabledStatesLocked();
}
if (enabled) {
EventLogTags.writeDeviceIdleOnPhase("power");
} else {
EventLogTags.writeDeviceIdleOffPhase("power");
}
return true;
}
boolean setLightDeviceIdleModeInternal(boolean enabled) {