Review fix

brave · Aug 15, 2022 · 2a84b32 · 2a84b32
1 parent ef4ecfc
commit 2a84b32
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/browser/ipfs/ipfs_dns_resolver_impl.cc b/browser/ipfs/ipfs_dns_resolver_impl.cc
@@ -5,14 +5,15 @@
 
 #include "brave/browser/ipfs/ipfs_dns_resolver_impl.h"
 
+#include "base/time/time.h"
 #include "chrome/browser/net/secure_dns_config.h"
 #include "chrome/browser/net/system_network_context_manager.h"
 #include "content/public/browser/network_service_instance.h"
 #include "services/network/public/mojom/network_service.mojom.h"
 
 namespace ipfs {
 
-const int kThrottleDeltaMs = 1000;
+constexpr base::TimeDelta kRetryDelay = base::Seconds(1);
 
 mojo::Remote<network::mojom::DnsConfigChangeManager>
 GetDnsConfigChangeManager() {
@@ -43,11 +44,12 @@ IpfsDnsResolverImpl::~IpfsDnsResolverImpl() {}
 void IpfsDnsResolverImpl::OnDnsConfigChangeManagerConnectionError() {
   DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_);
   receiver_.reset();
+  // Throttle network service reconnect to prevent possible battery drain.
   base::SequencedTaskRunnerHandle::Get()->PostDelayedTask(
       FROM_HERE,
       base::BindOnce(&IpfsDnsResolverImpl::SetupDnsConfigChangeNotifications,
                      weak_ptr_factory_.GetWeakPtr()),
-      base::Milliseconds(kThrottleDeltaMs));
+      kRetryDelay);
 }
 
 void IpfsDnsResolverImpl::OnDnsConfigChanged() {