Introduce a delegate to avoid hardcoding "chrome-extension" in net/.

net/url_request/url_request_throttler_simulation_unittest.cc

 // Copyright (c) 2012 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.
 
 // The tests in this file attempt to verify the following through simulation:
 // a) That a server experiencing overload will actually benefit from the
 //    anti-DDoS throttling logic, i.e. that its traffic spike will subside
 //    and be distributed over a longer period of time;
 // b) That "well-behaved" clients of a server under DDoS attack actually
 //    benefit from the anti-DDoS throttling logic; and
 // c) That the approximate increase in "perceived downtime" introduced by
 //    anti-DDoS throttling for various different actual downtimes is what
 //    we expect it to be.
 
 #include <cmath>
 #include <limits>
 #include <vector>
 
 #include "base/environment.h"
 #include "base/memory/scoped_vector.h"
 #include "base/rand_util.h"
 #include "base/time.h"
+#include "net/url_request/url_request_test_util.h"
 #include "net/url_request/url_request_throttler_manager.h"
 #include "net/url_request/url_request_throttler_test_support.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using base::TimeDelta;
 using base::TimeTicks;
 
 namespace net {
 namespace {
 
@@ skipping 119 matching lines @@
     requests_per_tick_.push_back(num_current_tick_queries_);
     num_current_tick_queries_ = 0;
   }
 
   // This is called by Requester. It returns the response code from
   // the server.
   int HandleRequest() {
     ++num_current_tick_queries_;
     if (!start_downtime_.is_null() &&
         start_downtime_ < now_ && now_ < end_downtime_) {
-      // TODO(joi): For the simulation measuring the increase in perceived
+      // For the simulation measuring the increase in perceived
       // downtime, it might be interesting to count separately the
       // queries seen by the server (assuming a front-end reverse proxy
       // is what actually serves up the 503s in this case) so that we could
       // visualize the traffic spike seen by the server when it comes up,
       // which would in many situations be ameliorated by the anti-DDoS
       // throttling.
       return 503;
     }
 
     if ((num_overloaded_ticks_remaining_ > 0 ||
@@ skipping 107 matching lines @@
   int num_current_tick_queries_;
   int num_overloaded_ticks_;
   int max_experienced_queries_per_tick_;
   std::vector<int> requests_per_tick_;
 
   DISALLOW_COPY_AND_ASSIGN(Server);
 };
 
 class TestingURLRequestThrottlerManager : public URLRequestThrottlerManager {
  public:
-  TestingURLRequestThrottlerManager() : URLRequestThrottlerManager() {
+  TestingURLRequestThrottlerManager() : URLRequestThrottlerManager(&delegate_) {
   }
+
+ private:
+  TestURLRequestThrottlerManagerDelegate delegate_;
 };
 
 // Mock throttler entry used by Requester class.
 class MockURLRequestThrottlerEntry : public URLRequestThrottlerEntry {
  public:
   explicit MockURLRequestThrottlerEntry(
       URLRequestThrottlerManager* manager)
       : URLRequestThrottlerEntry(manager, ""),
         mock_backoff_entry_(&backoff_policy_) {
   }
@@ skipping 118 matching lines @@
     TimeDelta current_jitter = TimeDelta::FromMilliseconds(
         request_jitter_.InMilliseconds() * base::RandDouble());
     if (base::RandInt(0, 1)) {
       effective_delay -= current_jitter;
     } else {
       effective_delay += current_jitter;
     }
 
     if (throttler_entry_->fake_now() - time_of_last_attempt_ >
         effective_delay) {
-      if (!throttler_entry_->ShouldRejectRequest(0)) {
+      if (!throttler_entry_->ShouldRejectRequest(NULL, 0)) {
         int status_code = server_->HandleRequest();
         MockURLRequestThrottlerHeaderAdapter response_headers(status_code);
         throttler_entry_->UpdateWithResponse("", &response_headers);
 
         if (status_code == 200) {
           if (results_)
             results_->AddSuccess();
 
           if (last_attempt_was_failure_) {
             last_downtime_duration_ =
@@ skipping 304 matching lines @@
     trials[i].PrintTrialDescription();
     trials[i].stats.ReportTrialResult(increase_ratio);
   }
 
   VerboseOut("Average increase ratio was %.4f\n", average_increase_ratio);
   VerboseOut("Maximum increase ratio was %.4f\n", max_increase_ratio);
 }
 
 }  // namespace
 }  // namespace net