Refactor net::BackoffEntry to not require subclassing

components/domain_reliability/util.h

 // Copyright 2014 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.
 
 #ifndef COMPONENTS_DOMAIN_RELIABILITY_UTIL_H_
 #define COMPONENTS_DOMAIN_RELIABILITY_UTIL_H_
 
 #include <map>
 
 #include "base/callback_forward.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/time/clock.h"
+#include "base/time/tick_clock.h"
 #include "base/time/time.h"
 #include "base/tracked_objects.h"
 #include "components/domain_reliability/domain_reliability_export.h"
 #include "components/domain_reliability/uploader.h"
-#include "net/base/backoff_entry.h"
 #include "net/http/http_response_info.h"
 #include "net/url_request/url_request_status.h"
 
 namespace domain_reliability {
 
 // Attempts to convert a net error and an HTTP response code into the status
 // string that should be recorded in a beacon. Returns true if it could.
 //
 // N.B.: This functions as the whitelist of "safe" errors to report; network-
 //       local errors are purposefully not converted to avoid revealing
@@ skipping 16 matching lines @@
 // fills |status| with the result of a report upload.
 void GetUploadResultFromResponseDetails(
     int net_error,
     int http_response_code,
     base::TimeDelta retry_after,
     DomainReliabilityUploader::UploadResult* result);
 
 // Mockable wrapper around TimeTicks::Now and Timer. Mock version is in
 // test_util.h.
 // TODO(ttuttle): Rename to Time{Provider,Source,?}.
-class DOMAIN_RELIABILITY_EXPORT MockableTime {
+class DOMAIN_RELIABILITY_EXPORT MockableTime : public base::Clock,
+                                               public base::TickClock {
  public:
   // Mockable wrapper around (a subset of) base::Timer.
   class DOMAIN_RELIABILITY_EXPORT Timer {
    public:
     virtual ~Timer();
 
     virtual void Start(const tracked_objects::Location& posted_from,
                        base::TimeDelta delay,
                        const base::Closure& user_task) = 0;
     virtual void Stop() = 0;
     virtual bool IsRunning() = 0;
 
    protected:
     Timer();
   };
 
-  virtual ~MockableTime();
+  ~MockableTime() override;
 
-  // Returns base::Time::Now() or a mocked version thereof.
-  virtual base::Time Now() = 0;
-  // Returns base::TimeTicks::Now() or a mocked version thereof.
-  virtual base::TimeTicks NowTicks() = 0;
+  // Clock impl; returns base::Time::Now() or a mocked version thereof.
+  base::Time Now() override = 0;
+  // TickClock impl; returns base::TimeTicks::Now() or a mocked version thereof.
+  base::TimeTicks NowTicks() override = 0;
+
   // Returns a new Timer, or a mocked version thereof.
   virtual scoped_ptr<MockableTime::Timer> CreateTimer() = 0;
 
  protected:
   MockableTime();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockableTime);
 };
 
 // Implementation of MockableTime that passes through to
 // base::Time{,Ticks}::Now() and base::Timer.
 class DOMAIN_RELIABILITY_EXPORT ActualTime : public MockableTime {
  public:
   ActualTime();
 
   ~ActualTime() override;
 
   // MockableTime implementation:
   base::Time Now() override;
   base::TimeTicks NowTicks() override;
   scoped_ptr<MockableTime::Timer> CreateTimer() override;
 };
 
-// A subclass of BackoffEntry that uses a MockableTime to keep track of time.
-class MockableTimeBackoffEntry : public net::BackoffEntry {
- public:
-  MockableTimeBackoffEntry(const net::BackoffEntry::Policy* const policy,
-                           MockableTime* time);
-
-  ~MockableTimeBackoffEntry() override;
-
- protected:
-  base::TimeTicks ImplGetTimeNow() const override;
-
- private:
-  MockableTime* time_;
-};
-
 }  // namespace domain_reliability
 
 #endif  // COMPONENTS_DOMAIN_RELIABILITY_UTIL_H_