| Index: net/base/backoff_entry_unittest.cc
|
| diff --git a/net/base/backoff_entry_unittest.cc b/net/base/backoff_entry_unittest.cc
|
| index 9a9f4cfca0a232024182784f87d5d4a9b19f579a..1b025647cd42e2bb4317cbb542c59d4f70e97935 100644
|
| --- a/net/base/backoff_entry_unittest.cc
|
| +++ b/net/base/backoff_entry_unittest.cc
|
| @@ -11,13 +11,12 @@ using base::TimeDelta;
|
| using base::TimeTicks;
|
| using net::BackoffEntry;
|
|
|
| -BackoffEntry::Policy base_policy = { 0, 1000, 2.0, 0.0, 20000, 2000, false };
|
| +BackoffEntry::Policy base_policy = {0, 1000, 2.0, 0.0, 20000, 2000, false};
|
|
|
| class TestBackoffEntry : public BackoffEntry {
|
| public:
|
| explicit TestBackoffEntry(const Policy* const policy)
|
| - : BackoffEntry(policy),
|
| - now_(TimeTicks()) {
|
| + : BackoffEntry(policy), now_(TimeTicks()) {
|
| // Work around initialization in constructor not picking up
|
| // fake time.
|
| SetCustomReleaseTime(TimeTicks());
|
| @@ -25,13 +24,9 @@ class TestBackoffEntry : public BackoffEntry {
|
|
|
| virtual ~TestBackoffEntry() {}
|
|
|
| - virtual TimeTicks ImplGetTimeNow() const OVERRIDE {
|
| - return now_;
|
| - }
|
| + virtual TimeTicks ImplGetTimeNow() const OVERRIDE { return now_; }
|
|
|
| - void set_now(const TimeTicks& now) {
|
| - now_ = now;
|
| - }
|
| + void set_now(const TimeTicks& now) { now_ = now; }
|
|
|
| private:
|
| TimeTicks now_;
|
| @@ -68,20 +63,20 @@ TEST(BackoffEntryTest, CanDiscard) {
|
| EXPECT_FALSE(entry.CanDiscard());
|
|
|
| // Test the case where there are errors but we can time out.
|
| - entry.set_now(
|
| - entry.GetReleaseTime() + TimeDelta::FromMilliseconds(1));
|
| + entry.set_now(entry.GetReleaseTime() + TimeDelta::FromMilliseconds(1));
|
| EXPECT_FALSE(entry.CanDiscard());
|
| - entry.set_now(entry.GetReleaseTime() + TimeDelta::FromMilliseconds(
|
| - base_policy.maximum_backoff_ms + 1));
|
| + entry.set_now(
|
| + entry.GetReleaseTime() +
|
| + TimeDelta::FromMilliseconds(base_policy.maximum_backoff_ms + 1));
|
| EXPECT_TRUE(entry.CanDiscard());
|
|
|
| // Test the final case (no errors, dependent only on specified lifetime).
|
| - entry.set_now(entry.GetReleaseTime() + TimeDelta::FromMilliseconds(
|
| - base_policy.entry_lifetime_ms - 1));
|
| + entry.set_now(entry.GetReleaseTime() +
|
| + TimeDelta::FromMilliseconds(base_policy.entry_lifetime_ms - 1));
|
| entry.InformOfRequest(true);
|
| EXPECT_FALSE(entry.CanDiscard());
|
| - entry.set_now(entry.GetReleaseTime() + TimeDelta::FromMilliseconds(
|
| - base_policy.entry_lifetime_ms));
|
| + entry.set_now(entry.GetReleaseTime() +
|
| + TimeDelta::FromMilliseconds(base_policy.entry_lifetime_ms));
|
| EXPECT_TRUE(entry.CanDiscard());
|
| }
|
|
|
| @@ -158,8 +153,8 @@ TEST(BackoffEntryTest, ReleaseTimeCalculation) {
|
| entry.InformOfRequest(false);
|
| entry.InformOfRequest(false);
|
| result = entry.GetReleaseTime();
|
| - EXPECT_EQ(
|
| - entry.ImplGetTimeNow() + TimeDelta::FromMilliseconds(20000), result);
|
| + EXPECT_EQ(entry.ImplGetTimeNow() + TimeDelta::FromMilliseconds(20000),
|
| + result);
|
| }
|
|
|
| TEST(BackoffEntryTest, ReleaseTimeCalculationAlwaysDelay) {
|
| @@ -209,10 +204,10 @@ TEST(BackoffEntryTest, ReleaseTimeCalculationWithJitter) {
|
| entry.InformOfRequest(false);
|
| entry.InformOfRequest(false);
|
| TimeTicks result = entry.GetReleaseTime();
|
| - EXPECT_LE(
|
| - entry.ImplGetTimeNow() + TimeDelta::FromMilliseconds(3200), result);
|
| - EXPECT_GE(
|
| - entry.ImplGetTimeNow() + TimeDelta::FromMilliseconds(4000), result);
|
| + EXPECT_LE(entry.ImplGetTimeNow() + TimeDelta::FromMilliseconds(3200),
|
| + result);
|
| + EXPECT_GE(entry.ImplGetTimeNow() + TimeDelta::FromMilliseconds(4000),
|
| + result);
|
| }
|
| }
|
|
|
|
|