LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Dump only NORMAL priority information if no priority assigned

Browse Source 
The legacy design of "dumpsys connectivity" will only dump
information with NORMAL priority. It was updated to provide
both NORMAL and HIGH priority information in order to support
dump in bugreport. However, it will also affect the result
using dumpsys connectivity.

Update design to dump NORMAL priority only to align with legacy
design.

Bug: 188387185
Test: adb shell dumpsys connectivity
Test: adb bugreport and check the result in bugreport
Change-Id: I6825c5038e48e3060c0c3ad1512bd584ef6d10a7
This commit is contained in:
Chiachang Wang
2021-05-17 16:57:15 +08:00
parent ca98d3d814
commit 05fbb45936
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
service/src/com/android/server/ConnectivityService.java
View File

@@ -1045,14 +1045,10 @@ public class ConnectivityService extends IConnectivityManager.Stub
} else {
// ConnectivityService publishes binder service using publishBinderService() with
// no priority assigned will be treated as NORMAL priority. Dumpsys does not send
// "--dump-priority" arguments to the service. Thus, dump both NORMAL and HIGH to
// align the legacy design.
// "--dump-priority" arguments to the service. Thus, dump NORMAL only to align the
// legacy output for dumpsys connectivity.
// TODO: Integrate into signal dump.
dumpNormal(fd, pw, args);
pw.println();
pw.println("DUMP OF SERVICE HIGH connectivity");
pw.println();
dumpHigh(fd, pw);
}
}
}