diff options
| author | Chiachang Wang <chiachangwang@google.com> | 2019-10-24 00:48:27 -0700 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2019-10-24 00:48:27 -0700 |
| commit | 0ec9d5bb84e8394ca2c2ead47646aa2d431a87af (patch) | |
| tree | 1dfb10b345581fca32b443b43f5c12dac98e29f1 /src | |
| parent | 3a0cd647c7d03465c36aa330468bc6dd91e4dd59 (diff) | |
| parent | ed5f519883cd4257881ca8f46961e8d4189feed8 (diff) | |
Keep only one CMD_EVALUATE_PRIVATE_DNS in the queue
am: ed5f519883
Change-Id: I91aaddb89b5439269af121c8c8d51e496abbaf0a
Diffstat (limited to 'src')
| -rw-r--r-- | src/com/android/server/connectivity/NetworkMonitor.java | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/com/android/server/connectivity/NetworkMonitor.java b/src/com/android/server/connectivity/NetworkMonitor.java index 7cabb1e..e08170a 100644 --- a/src/com/android/server/connectivity/NetworkMonitor.java +++ b/src/com/android/server/connectivity/NetworkMonitor.java @@ -681,6 +681,8 @@ public class NetworkMonitor extends StateMachine { // no resolved IP addresses, IPs unreachable, // port 853 unreachable, port 853 is not running a // DNS-over-TLS server, et cetera). + // Cancel any outstanding CMD_EVALUATE_PRIVATE_DNS. + removeMessages(CMD_EVALUATE_PRIVATE_DNS); sendMessage(CMD_EVALUATE_PRIVATE_DNS); break; } @@ -1042,6 +1044,11 @@ public class NetworkMonitor extends StateMachine { // All good! transitionTo(mValidatedState); break; + case CMD_PRIVATE_DNS_SETTINGS_CHANGED: + // When settings change the reevaluation timer must be reset. + mPrivateDnsReevalDelayMs = INITIAL_REEVALUATE_DELAY_MS; + // Let the message bubble up and be handled by parent states as usual. + return NOT_HANDLED; default: return NOT_HANDLED; } @@ -1094,10 +1101,6 @@ public class NetworkMonitor extends StateMachine { // transitioning back to EvaluatingState, to perhaps give ourselves // the opportunity to (re)detect a captive portal or something. // - // TODO: distinguish between CMD_EVALUATE_PRIVATE_DNS messages that are caused by server - // lookup failures (which should continue to do exponential backoff) and - // CMD_EVALUATE_PRIVATE_DNS messages that are caused by user reconfiguration (which - // should be processed immediately. sendMessageDelayed(CMD_EVALUATE_PRIVATE_DNS, mPrivateDnsReevalDelayMs); mPrivateDnsReevalDelayMs *= 2; if (mPrivateDnsReevalDelayMs > MAX_REEVALUATE_DELAY_MS) { |