API: Clean up redundant and ineffective usages of SystemApi and TestApi

Everything that is marked SystemApi or TestApi, but not @hide is still
part of the public SDK, it is therefore not sound to have that combination.

In the future, specifing such a combination will be considered an error
to prevent inadvertently exposing SystemApi and TestApi as public API.

Bug: 115333477
Change-Id: Ibd5d6a22862fdbc1e20a1cb3925280f5a682edea
Merged-In: Ibd5d6a22862fdbc1e20a1cb3925280f5a682edea
Test: METALAVA_PREPEND_ARGS="--error UnhiddenSystemApi" m checkapi
Exempt-From-Owner-Approval: API cleanup
This commit is contained in:
Adrian Roos
2018-12-11 19:47:33 +01:00
parent c8dcfb8b4b
commit be53fbb8d5

View File

@@ -274,7 +274,6 @@ public class TrafficStats {
* Changes only take effect during subsequent calls to * Changes only take effect during subsequent calls to
* {@link #tagSocket(Socket)}. * {@link #tagSocket(Socket)}.
*/ */
@SystemApi
@SuppressLint("Doclava125") @SuppressLint("Doclava125")
public static void setThreadStatsUid(int uid) { public static void setThreadStatsUid(int uid) {
NetworkManagementSocketTagger.setThreadSocketStatsUid(uid); NetworkManagementSocketTagger.setThreadSocketStatsUid(uid);
@@ -313,7 +312,6 @@ public class TrafficStats {
* *
* @see #setThreadStatsUid(int) * @see #setThreadStatsUid(int)
*/ */
@SystemApi
@SuppressLint("Doclava125") @SuppressLint("Doclava125")
public static void clearThreadStatsUid() { public static void clearThreadStatsUid() {
NetworkManagementSocketTagger.setThreadSocketStatsUid(-1); NetworkManagementSocketTagger.setThreadSocketStatsUid(-1);