Remove Network, NetworkRequest metrics from jobscheduler
These metrics are deprecated so remove them from jobscheduler.
Also remove dumpDebug method from Network, NetworkRequest and
NetworkCapabilities because there's no caller anymore.
This change also for connectivity mainline module. These
three classes are inculded in framework-connectivity so
external module cannot have dependencies on its hidden API.
With this change, the dependencies can be removed.
Bug: 178777253
Test: FrameworksNetTests
JobStoreTest
adb shell dumpsys jobscheduler --proto
CtsIncidentHostTestCases:JobSchedulerIncidentTest
Change-Id: Ie0c540303ba06b8fba029d2b98ae753afb08c963
This commit is contained in:
@@ -27,7 +27,6 @@ import android.os.Parcelable;
|
||||
import android.system.ErrnoException;
|
||||
import android.system.Os;
|
||||
import android.system.OsConstants;
|
||||
import android.util.proto.ProtoOutputStream;
|
||||
|
||||
import com.android.internal.annotations.GuardedBy;
|
||||
|
||||
@@ -526,11 +525,4 @@ public class Network implements Parcelable {
|
||||
public String toString() {
|
||||
return Integer.toString(netId);
|
||||
}
|
||||
|
||||
/** @hide */
|
||||
public void dumpDebug(ProtoOutputStream proto, long fieldId) {
|
||||
final long token = proto.start(fieldId);
|
||||
proto.write(NetworkProto.NET_ID, netId);
|
||||
proto.end(token);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -35,7 +35,6 @@ import android.os.Process;
|
||||
import android.text.TextUtils;
|
||||
import android.util.ArraySet;
|
||||
import android.util.Range;
|
||||
import android.util.proto.ProtoOutputStream;
|
||||
|
||||
import com.android.internal.annotations.VisibleForTesting;
|
||||
import com.android.net.module.util.CollectionUtils;
|
||||
@@ -2068,34 +2067,6 @@ public final class NetworkCapabilities implements Parcelable {
|
||||
}
|
||||
}
|
||||
|
||||
/** @hide */
|
||||
public void dumpDebug(@NonNull ProtoOutputStream proto, long fieldId) {
|
||||
final long token = proto.start(fieldId);
|
||||
|
||||
for (int transport : getTransportTypes()) {
|
||||
proto.write(NetworkCapabilitiesProto.TRANSPORTS, transport);
|
||||
}
|
||||
|
||||
for (int capability : getCapabilities()) {
|
||||
proto.write(NetworkCapabilitiesProto.CAPABILITIES, capability);
|
||||
}
|
||||
|
||||
proto.write(NetworkCapabilitiesProto.LINK_UP_BANDWIDTH_KBPS, mLinkUpBandwidthKbps);
|
||||
proto.write(NetworkCapabilitiesProto.LINK_DOWN_BANDWIDTH_KBPS, mLinkDownBandwidthKbps);
|
||||
|
||||
if (mNetworkSpecifier != null) {
|
||||
proto.write(NetworkCapabilitiesProto.NETWORK_SPECIFIER, mNetworkSpecifier.toString());
|
||||
}
|
||||
if (mTransportInfo != null) {
|
||||
// TODO b/120653863: write transport-specific info to proto?
|
||||
}
|
||||
|
||||
proto.write(NetworkCapabilitiesProto.CAN_REPORT_SIGNAL_STRENGTH, hasSignalStrength());
|
||||
proto.write(NetworkCapabilitiesProto.SIGNAL_STRENGTH, mSignalStrength);
|
||||
|
||||
proto.end(token);
|
||||
}
|
||||
|
||||
/**
|
||||
* @hide
|
||||
*/
|
||||
|
||||
@@ -47,7 +47,6 @@ import android.os.Parcelable;
|
||||
import android.os.Process;
|
||||
import android.text.TextUtils;
|
||||
import android.util.Range;
|
||||
import android.util.proto.ProtoOutputStream;
|
||||
|
||||
import java.util.Arrays;
|
||||
import java.util.List;
|
||||
@@ -647,18 +646,6 @@ public class NetworkRequest implements Parcelable {
|
||||
}
|
||||
}
|
||||
|
||||
/** @hide */
|
||||
public void dumpDebug(ProtoOutputStream proto, long fieldId) {
|
||||
final long token = proto.start(fieldId);
|
||||
|
||||
proto.write(NetworkRequestProto.TYPE, typeToProtoEnum(type));
|
||||
proto.write(NetworkRequestProto.REQUEST_ID, requestId);
|
||||
proto.write(NetworkRequestProto.LEGACY_TYPE, legacyType);
|
||||
networkCapabilities.dumpDebug(proto, NetworkRequestProto.NETWORK_CAPABILITIES);
|
||||
|
||||
proto.end(token);
|
||||
}
|
||||
|
||||
public boolean equals(@Nullable Object obj) {
|
||||
if (obj instanceof NetworkRequest == false) return false;
|
||||
NetworkRequest that = (NetworkRequest)obj;
|
||||
|
||||
Reference in New Issue
Block a user