Ensure ResourceScheduler starts requests even if the tab is closed.

content/browser/loader/resource_scheduler_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.
 
 #include "content/browser/loader/resource_scheduler.h"
 
 #include "base/memory/scoped_vector.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 2195 matching lines @@
                                                    rvh2->GetRoutingID()));
   // Clean up.
   web_contents_1.reset();
   web_contents_2.reset();
   base::RunLoop().RunUntilIdle();
 
   browser_context.reset();
   render_process_host_factory.reset();
 }
 
+// Async revalidations which are not started when the tab is closed must be
+// started at some point, or they will hang around forever and prevent other
+// async revalidations to the same URL from being issued.
+TEST_F(ResourceSchedulerTest, RequestStartedAfterClientDeleted) {
+  scheduler_.OnClientCreated(kChildId2, kRouteId2, false, false);
+  scoped_ptr<TestRequest> high(NewRequestWithChildAndRoute(
+      "http://host/high", net::HIGHEST, kChildId2, kRouteId2));
+  scoped_ptr<TestRequest> lowest1(NewRequestWithChildAndRoute(
+      "http://host/lowest", net::LOWEST, kChildId2, kRouteId2));
+  scoped_ptr<TestRequest> lowest2(NewRequestWithChildAndRoute(
+      "http://host/lowest", net::LOWEST, kChildId2, kRouteId2));
+  EXPECT_FALSE(lowest2->started());
+  scheduler_.OnClientDeleted(kChildId2, kRouteId2);
+  high.reset();
+  lowest1.reset();
+  EXPECT_TRUE(lowest2->started());
+}
+
+// The ResourceScheduler::Client destructor calls
+// LoadAnyStartablePendingRequests(), which may start some pending requests.
+// This test is to verify that requests will be started at some point
+// even if they were not started by the destructor.
+TEST_F(ResourceSchedulerTest, RequestStartedAfterClientDeletedManyDelayable) {
+  scheduler_.OnClientCreated(kChildId2, kRouteId2, false, false);
+  scoped_ptr<TestRequest> high(NewRequestWithChildAndRoute(
+      "http://host/high", net::HIGHEST, kChildId2, kRouteId2));
+  const int kMaxNumDelayableRequestsPerClient = 10;
+  ScopedVector<TestRequest> delayable_requests;
+  for (int i = 0; i < kMaxNumDelayableRequestsPerClient + 1; ++i) {
+    delayable_requests.push_back(NewRequestWithChildAndRoute(
+        "http://host/lowest", net::LOWEST, kChildId2, kRouteId2));
+  }
+  scoped_ptr<TestRequest> lowest(NewRequestWithChildAndRoute(
+      "http://host/lowest", net::LOWEST, kChildId2, kRouteId2));
+  EXPECT_FALSE(lowest->started());
+  scheduler_.OnClientDeleted(kChildId2, kRouteId2);
+  high.reset();
+  delayable_requests.clear();
+  EXPECT_TRUE(lowest->started());
+}
+
 }  // unnamed namespace
 
 }  // namespace content