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

Commit f95c8654 authored by Jeremy Joslin's avatar Jeremy Joslin
Browse files

Adding a new internal method to get all valid scorers.

New NetworkScoreManager method to get all valid scorers from the
NetworkScoreService. The implementation is stubbed out for the time
being.

Test: make
Bug: 35095406
Change-Id: Iece2cc10a0a9377f2d82385d172b06660bb9b130
parent b09ac24c
Loading
Loading
Loading
Loading
+9 −0
Original line number Diff line number Diff line
@@ -132,5 +132,14 @@ interface INetworkScoreService
    oneway void requestRecommendationAsync(in RecommendationRequest request,
                                           in RemoteCallback remoteCallback);

    /**
     * Returns metadata about the active scorer or <code>null</code> if there is no active scorer.
     */
    NetworkScorerAppManager.NetworkScorerAppData getActiveScorer();

    /**
     * Returns the list of available scorer apps. The list will be empty if there are
     * no valid scorers.
     */
    List<NetworkScorerAppManager.NetworkScorerAppData> getAllValidScorers();
}
+15 −0
Original line number Diff line number Diff line
@@ -37,6 +37,7 @@ import com.android.internal.util.Preconditions;

import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.util.List;
import java.util.concurrent.CompletableFuture;

/**
@@ -199,6 +200,20 @@ public class NetworkScoreManager {
        }
    }

    /**
     * Returns the list of available scorer apps. The list will be empty if there are
     * no valid scorers.
     *
     * @hide
     */
    public List<NetworkScorerAppData> getAllValidScorers() {
        try {
            return mService.getAllValidScorers();
        } catch (RemoteException e) {
            throw e.rethrowFromSystemServer();
        }
    }

    /**
     * Update network scores.
     *
+8 −0
Original line number Diff line number Diff line
@@ -126,6 +126,14 @@ public class NetworkScorerAppManager {
        }
    }

    /**
     * Returns the list of available scorer apps. The list will be empty if there are
     * no valid scorers.
     */
    public List<NetworkScorerAppData> getAllValidScorers() {
        return Collections.emptyList();
    }

    /**
     * @return A {@link NetworkScorerAppData} instance containing information about the
     *         best configured network recommendation provider installed or {@code null}
+9 −0
Original line number Diff line number Diff line
@@ -718,6 +718,15 @@ public class NetworkScoreService extends INetworkScoreService.Stub {
        return null;
    }

    /**
     * Returns the list of available scorer apps. The list will be empty if there are
     * no valid scorers.
     */
    @Override
    public List<NetworkScorerAppData> getAllValidScorers() {
        return mNetworkScorerAppManager.getAllValidScorers();
    }

    @Override
    public void disableScoring() {
        // Only the active scorer or the system should be allowed to disable scoring.