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

Add comment / logging in NetworkMonitor callbacks

Browse Source 
Add a comment explaining the ordering of messages sent to the tracker
and connectivity diagnostics handlers.

Add a Slog.wtf call in case the deprecated notifyNetworkTested callback
is called.

Bug: 153500847
Test: atest ConnectivityServiceTest
Merged-In: I2dbfc9bf7b2f785ea4594851bd354e9fd0fc0bd1
Change-Id: I2dbfc9bf7b2f785ea4594851bd354e9fd0fc0bd1
This commit is contained in:
Remi NGUYEN VAN
2020-04-27 08:08:57 +00:00
parent cc9f73cad3
commit 3d3c9f7b76
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
services/core/java/com/android/server/ConnectivityService.java
View File

@@ -3011,10 +3011,13 @@ public class ConnectivityService extends IConnectivityManager.Stub
// Legacy version of notifyNetworkTestedWithExtras. // Legacy version of notifyNetworkTestedWithExtras.
// Would only be called if the system has a NetworkStack module older than the // Would only be called if the system has a NetworkStack module older than the
// framework, which does not happen in practice. // framework, which does not happen in practice.
Slog.wtf(TAG, "Deprecated notifyNetworkTested called: no action taken");
} }
@Override @Override
public void notifyNetworkTestedWithExtras(NetworkTestResultParcelable p) { public void notifyNetworkTestedWithExtras(NetworkTestResultParcelable p) {
// Notify mTrackerHandler and mConnectivityDiagnosticsHandler of the event. Both use
// the same looper so messages will be processed in sequence.
final Message msg = mTrackerHandler.obtainMessage( final Message msg = mTrackerHandler.obtainMessage(
EVENT_NETWORK_TESTED, EVENT_NETWORK_TESTED,
new NetworkTestedResults( new NetworkTestedResults(