Merge "[SP18.1] add dependency object to OffloadController"
This commit is contained in:
@@ -130,8 +130,20 @@ public class OffloadController {
|
||||
private int mNatUpdateCallbacksReceived;
|
||||
private int mNatUpdateNetlinkErrors;
|
||||
|
||||
@NonNull
|
||||
private final Dependencies mDeps;
|
||||
|
||||
// TODO: Put more parameters in constructor into dependency object.
|
||||
static class Dependencies {
|
||||
int getPerformPollInterval() {
|
||||
// TODO: Consider make this configurable.
|
||||
return DEFAULT_PERFORM_POLL_INTERVAL_MS;
|
||||
}
|
||||
}
|
||||
|
||||
public OffloadController(Handler h, OffloadHardwareInterface hwi,
|
||||
ContentResolver contentResolver, NetworkStatsManager nsm, SharedLog log) {
|
||||
ContentResolver contentResolver, NetworkStatsManager nsm, SharedLog log,
|
||||
@NonNull Dependencies deps) {
|
||||
mHandler = h;
|
||||
mHwInterface = hwi;
|
||||
mContentResolver = contentResolver;
|
||||
@@ -147,6 +159,7 @@ public class OffloadController {
|
||||
provider = null;
|
||||
}
|
||||
mStatsProvider = provider;
|
||||
mDeps = deps;
|
||||
}
|
||||
|
||||
/** Start hardware offload. */
|
||||
@@ -439,7 +452,7 @@ public class OffloadController {
|
||||
if (mHandler.hasCallbacks(mScheduledPollingTask)) {
|
||||
mHandler.removeCallbacks(mScheduledPollingTask);
|
||||
}
|
||||
mHandler.postDelayed(mScheduledPollingTask, DEFAULT_PERFORM_POLL_INTERVAL_MS);
|
||||
mHandler.postDelayed(mScheduledPollingTask, mDeps.getPerformPollInterval());
|
||||
}
|
||||
|
||||
private boolean isPollingStatsNeeded() {
|
||||
|
||||
@@ -273,7 +273,7 @@ public class Tethering {
|
||||
mHandler = mTetherMasterSM.getHandler();
|
||||
mOffloadController = new OffloadController(mHandler,
|
||||
mDeps.getOffloadHardwareInterface(mHandler, mLog), mContext.getContentResolver(),
|
||||
statsManager, mLog);
|
||||
statsManager, mLog, new OffloadController.Dependencies());
|
||||
mUpstreamNetworkMonitor = mDeps.getUpstreamNetworkMonitor(mContext, mTetherMasterSM, mLog,
|
||||
TetherMasterSM.EVENT_UPSTREAM_CALLBACK);
|
||||
mForwardedDownstreams = new LinkedHashSet<>();
|
||||
|
||||
Reference in New Issue
Block a user