Refactor net::BackoffEntry to not require subclassing

components/password_manager/core/browser/affiliation_fetch_throttler.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/password_manager/core/browser/affiliation_fetch_throttler.h"
 
 #include <stdint.h>
 
 #include "base/logging.h"
 #include "base/rand_util.h"
 #include "base/thread_task_runner_handle.h"
 #include "base/time/tick_clock.h"
 #include "base/time/time.h"
 #include "components/password_manager/core/browser/affiliation_fetch_throttler_delegate.h"
 
 namespace password_manager {
 
-namespace {
-
-// Implementation of net::BackoffEntry that allows mocking its tick source.
-class BackoffEntryImpl : public net::BackoffEntry {
- public:
-  // |tick_clock| must outlive this instance.
-  explicit BackoffEntryImpl(const net::BackoffEntry::Policy* const policy,
-                            base::TickClock* tick_clock)
-      : BackoffEntry(policy), tick_clock_(tick_clock) {}
-  ~BackoffEntryImpl() override {}
-
- private:
-  // net::BackoffEntry:
-  base::TimeTicks ImplGetTimeNow() const override {
-    return tick_clock_->NowTicks();
-  }
-
-  base::TickClock* tick_clock_;
-
-  DISALLOW_COPY_AND_ASSIGN(BackoffEntryImpl);
-};
-
-}  // namespace
-
 // static
 const net::BackoffEntry::Policy AffiliationFetchThrottler::kBackoffPolicy = {
     // Number of initial errors (in sequence) to ignore before going into
     // exponential backoff.
     0,
 
     // Initial delay (in ms) once backoff starts.
     10 * 1000,  // 10 seconds
 
     // Factor by which the delay will be multiplied on each subsequent failure.
@@ skipping 23 matching lines @@
 AffiliationFetchThrottler::AffiliationFetchThrottler(
     AffiliationFetchThrottlerDelegate* delegate,
     const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
     base::TickClock* tick_clock)
     : delegate_(delegate),
       task_runner_(task_runner),
       tick_clock_(tick_clock),
       state_(IDLE),
       has_network_connectivity_(false),
       is_fetch_scheduled_(false),
-      exponential_backoff_(new BackoffEntryImpl(&kBackoffPolicy, tick_clock_)),
+      exponential_backoff_(new net::BackoffEntry(&kBackoffPolicy, tick_clock_)),
       weak_ptr_factory_(this) {
   DCHECK(delegate);
   // Start observing before querying the current connectivity state, so that if
   // the state changes concurrently in-between, it will not go unnoticed.
   net::NetworkChangeNotifier::AddConnectionTypeObserver(this);
   has_network_connectivity_ = !net::NetworkChangeNotifier::IsOffline();
 }
 
 AffiliationFetchThrottler::~AffiliationFetchThrottler() {
   net::NetworkChangeNotifier::RemoveConnectionTypeObserver(this);
@@ skipping 62 matching lines @@
                     (1 - base::RandDouble() * kBackoffPolicy.jitter_factor);
   exponential_backoff_->SetCustomReleaseTime(std::max(
       exponential_backoff_->GetReleaseTime(),
       tick_clock_->NowTicks() + base::TimeDelta::FromMillisecondsD(grace_ms)));
 
   if (state_ == FETCH_NEEDED)
     EnsureCallbackIsScheduled();
 }
 
 }  // namespace password_manager