Captive portal service: Don't use a retry delay after getting a new result.

chrome/browser/captive_portal/captive_portal_service.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.
 
 #include "chrome/browser/captive_portal/captive_portal_service.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ skipping 243 matching lines @@
 
   if (last_check_time_.is_null() || new_result != last_detection_result_) {
     first_check_time_with_same_result_ = now;
     num_checks_with_same_result_ = 1;
 
     // Reset the backoff entry both to update the default time and clear
     // previous failures.
     ResetBackoffEntry(new_result);
 
     backoff_entry_->SetCustomReleaseTime(now + retry_after_delta);
-    backoff_entry_->InformOfRequest(true);
+    // The BackoffEntry is not informed of this request, so there's no delay
+    // before the next request.  This allows for faster login when a captive
+    // portal is first detected.  It can also help when moving between captive
+    // portals.
   } else {
     DCHECK_LE(1, num_checks_with_same_result_);
     ++num_checks_with_same_result_;
 
     // Requests that have the same Result as the last one are considered
     // "failures", to trigger backoff.
+    backoff_entry_->SetCustomReleaseTime(now + retry_after_delta);
     backoff_entry_->InformOfRequest(false);
   }
 
   last_check_time_ = now;
 
   OnResult(new_result);
 }
 
 void CaptivePortalService::Observe(
     int type,
@@ skipping 134 matching lines @@
 
 bool CaptivePortalService::FetchingURL() const {
   return url_fetcher_.get() != NULL;
 }
 
 bool CaptivePortalService::TimerRunning() const {
   return check_captive_portal_timer_.IsRunning();
 }
 
 }  // namespace captive_portal