Updates to NetworkFactory for the new NetworkAgent constructor
Bug: 138306002
Bug: 139268426
Test: atest FrameworksNetTests FrameworksWifiTests FrameworksTelephonyTests
Merged-In: If327cf0ffd917f9cb358f9ed3779da58fce40ef8
Change-Id: If327cf0ffd917f9cb358f9ed3779da58fce40ef8
(clean cherry-pick from internal branch)
diff --git a/staticlibs/src_servicescommon/android/net/NetworkFactory.java b/staticlibs/src_servicescommon/android/net/NetworkFactory.java
index 3758bb5..47054f5 100644
--- a/staticlibs/src_servicescommon/android/net/NetworkFactory.java
+++ b/staticlibs/src_servicescommon/android/net/NetworkFactory.java
@@ -108,7 +108,6 @@
private int mRefCount = 0;
private Messenger mMessenger = null;
private NetworkProvider mProvider = null;
- private int mProviderId;
public NetworkFactory(Looper looper, Context context, String logTag,
NetworkCapabilities filter) {
@@ -139,7 +138,7 @@
};
mMessenger = new Messenger(this);
- mProviderId = ((ConnectivityManager) mContext.getSystemService(
+ ((ConnectivityManager) mContext.getSystemService(
Context.CONNECTIVITY_SERVICE)).registerNetworkProvider(mProvider);
}
@@ -292,8 +291,8 @@
log(" n.requests = " + n.requested);
log(" n.score = " + n.score);
log(" mScore = " + mScore);
- log(" n.providerId = " + n.providerId);
- log(" mProviderId = " + mProviderId);
+ log(" request.providerId = " + n.providerId);
+ log(" mProvider.id = " + mProvider.getProviderId());
}
if (shouldNeedNetworkFor(n)) {
if (VDBG) log(" needNetworkFor");
@@ -314,7 +313,7 @@
// If the score of this request is higher or equal to that of this factory and some
// other factory is responsible for it, then this factory should not track the request
// because it has no hope of satisfying it.
- && (n.score < mScore || n.providerId == mProviderId)
+ && (n.score < mScore || n.providerId == mProvider.getProviderId())
// If this factory can't satisfy the capability needs of this request, then it
// should not be tracked.
&& n.request.satisfiedBy(mCapabilityFilter)
@@ -332,7 +331,7 @@
// assigned to the factory
// - This factory can't satisfy the capability needs of the request
// - The concrete implementation of the factory rejects the request
- && ((n.score > mScore && n.providerId != mProviderId)
+ && ((n.score > mScore && n.providerId != mProvider.getProviderId())
|| !n.request.satisfiedBy(mCapabilityFilter)
|| !acceptRequest(n.request, n.score));
}
@@ -402,11 +401,11 @@
/* TODO: delete when all callers have migrated to NetworkProvider IDs. */
public int getSerialNumber() {
- return mProviderId;
+ return mProvider.getProviderId();
}
- public int getProviderId() {
- return mProviderId;
+ public NetworkProvider getProvider() {
+ return mProvider;
}
protected void log(String s) {
@@ -422,8 +421,8 @@
@Override
public String toString() {
- StringBuilder sb = new StringBuilder("{").append(LOG_TAG).append(" - mProviderId=")
- .append(mProviderId).append(", ScoreFilter=")
+ StringBuilder sb = new StringBuilder("{").append(LOG_TAG).append(" - providerId=")
+ .append(mProvider.getProviderId()).append(", ScoreFilter=")
.append(mScore).append(", Filter=").append(mCapabilityFilter).append(", requests=")
.append(mNetworkRequests.size()).append(", refCount=").append(mRefCount)
.append("}");