Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 92621a58 authored by Treehugger Robot's avatar Treehugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Clean up log spam" into main

parents 7972d436 ddee2819
Loading
Loading
Loading
Loading
+20 −34
Original line number Diff line number Diff line
@@ -247,37 +247,27 @@ public class NetworkTypeController extends StateMachine {
    private final ConnectivityManager.NetworkCallback mNetworkCallback =
            new ConnectivityManager.NetworkCallback() {
                @Override
                public void onAvailable(Network network) {
                    log("On Available: " + network);
                    if (network != null) {
                public void onAvailable(@NonNull Network network) {
                    if (mConnectivityManager != null) {
                        NetworkCapabilities capabilities =
                                mConnectivityManager.getNetworkCapabilities(network);
                            updateBandwidthConstrainedStatus(capabilities);
                        } else {
                            log("network is null");
                        if (capabilities != null) {
                            updateBandwidthConstrainedStatus(
                                    isBandwidthConstrainedCapabilitySupported(capabilities));
                        }
                    }
                }

                @Override
                public void onCapabilitiesChanged(Network network,
                        NetworkCapabilities networkCapabilities) {
                    log("onCapabilitiesChanged: " + network);
                    if (network != null) {
                        updateBandwidthConstrainedStatus(networkCapabilities);
                    } else {
                        log("network is null");
                    }
                public void onCapabilitiesChanged(@NonNull Network network,
                        @NonNull NetworkCapabilities networkCapabilities) {
                    updateBandwidthConstrainedStatus(
                            isBandwidthConstrainedCapabilitySupported(networkCapabilities));
                }

                @Override
                public void onLost(Network network) {
                    log("Network Lost");
                    if (mIsSatelliteConstrainedData) {
                        mIsSatelliteConstrainedData = false;
                        mDisplayInfoController.updateTelephonyDisplayInfo();
                    }
                public void onLost(@NonNull Network network) {
                    updateBandwidthConstrainedStatus(false);
                }
            };

@@ -296,17 +286,13 @@ public class NetworkTypeController extends StateMachine {
        }
    }

    private void updateBandwidthConstrainedStatus(NetworkCapabilities capabilities) {
        if (capabilities != null) {
            boolean isConstrained = isBandwidthConstrainedCapabilitySupported(capabilities);
    private void updateBandwidthConstrainedStatus(boolean isConstrained) {
        if (isConstrained != mIsSatelliteConstrainedData) {
            mIsSatelliteConstrainedData = isConstrained;
                log("Update network type because satellite constrained data status changed to "
            log("Reset timers because satellite constrained data status changed to "
                    + mIsSatelliteConstrainedData);
                mDisplayInfoController.updateTelephonyDisplayInfo();
            }
        } else {
            log("capabilities is null");
            resetAllTimers();
            transitionToCurrentState();
        }
    }