commit 6a3c07e2c59ace87d491c15682f631d3fc403beb
author Sudheer Shanka <sudheersai@google.com> Tue Sep 11 15:21:22 2018 -0700
committer Sudheer Shanka <sudheersai@google.com> Tue Sep 11 19:14:50 2018 -0700
tree fafccbc4c678f536177481fb104b4b06c76156c8
parent 76f4a95675d8d7f322d147d1809fb61b30393b0b

Register NPMS$UidObserver to listen for only necessary procstate changes.

Right now, NPMS listens for all the procstate changes. This will slows
down processing of procstate changes which actually matter and also
fills up the netpolicy history pretty quick.

Bug: 112485285
Test: cts-tradefed run singleCommand cts-dev -m CtsHostsideNetworkTests -t \
      com.android.cts.net.HostsideRestrictBackgroundNetworkTests
Test: atest services/tests/servicestests/src/com/android/server/NetworkPolicyManagerServiceTest.java

Change-Id: Iaaef8ea4ee16f5b69cafc1e7e5908c6ff6fe1ed3

services/core/java/com/android/server/net/NetworkPolicyManagerService.java

diff --git a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java
index 76f9695..48e09d7 100644
--- a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java
+++ b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java
@@ -796,7 +796,7 @@
             try {
                 mActivityManager.registerUidObserver(mUidObserver,
                         ActivityManager.UID_OBSERVER_PROCSTATE|ActivityManager.UID_OBSERVER_GONE,
-                        ActivityManager.PROCESS_STATE_UNKNOWN, null);
+                        NetworkPolicyManager.FOREGROUND_THRESHOLD_STATE, "android");
                 mNetworkManager.registerObserver(mAlertObserver);
             } catch (RemoteException e) {
                 // ignored; both services live in system_server