ResourceScheduler should use renderer notifications instead of MRUCache to track renderers.

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/message_loop.h"
 #include "content/browser/browser_thread_impl.h"
 #include "content/browser/loader/resource_dispatcher_host_impl.h"
 #include "content/browser/loader/resource_request_info_impl.h"
@@ skipping 62 matching lines @@
   }
 
   scoped_ptr<TestRequest> request_to_cancel_;
 };
 
 class ResourceSchedulerTest : public testing::Test {
  protected:
   ResourceSchedulerTest()
       : message_loop_(MessageLoop::TYPE_IO),
         ui_thread_(BrowserThread::UI, &message_loop_) {
-    scheduler_.OnNavigate(kChildId, kRouteId);
+    scheduler_.OnClientCreated(kChildId, kRouteId);
   }
 
   virtual ~ResourceSchedulerTest() {
+    scheduler_.OnClientDeleted(kChildId, kRouteId);
   }
 
   scoped_ptr<net::URLRequest> NewURLRequest(const char* url,
                                             net::RequestPriority priority,
                                             int route_id = kRouteId) {
     scoped_ptr<net::URLRequest> url_request(
         context_.CreateRequest(GURL(url), NULL));
     url_request->set_priority(priority);
     ResourceRequestInfo::AllocateForTesting(
         url_request.get(), ResourceType::SUB_RESOURCE, NULL, kChildId,
@@ skipping 64 matching lines @@
   scoped_ptr<TestRequest> low(NewRequest("http://host/low", net::LOWEST));
   EXPECT_TRUE(high1->started());
   EXPECT_TRUE(high2->started());
   EXPECT_FALSE(low->started());
   high1.reset();
   EXPECT_FALSE(low->started());
   high2.reset();
   EXPECT_TRUE(low->started());
 }
 
-TEST_F(ResourceSchedulerTest, EvictedClientsIssuePendingRequests) {
-  ScopedVector<TestRequest> low_requests;
-  ScopedVector<TestRequest> high_requests;
-  for (int i = 0; i < 6; i++) {
-    scheduler_.OnNavigate(kChildId, kRouteId + i);
-    high_requests.push_back(
-        NewRequest("http://host/i", net::HIGHEST, kRouteId + i));
-    low_requests.push_back(
-        NewRequest("http://host/i", net::LOWEST, kRouteId + i));
-    EXPECT_FALSE(low_requests[i]->started());
-  }
-  EXPECT_TRUE(low_requests[0]->started());
-}
-
 TEST_F(ResourceSchedulerTest, CancelOtherRequestsWhileResuming) {
   scoped_ptr<TestRequest> high(NewRequest("http://host/high", net::HIGHEST));
   scoped_ptr<TestRequest> low1(NewRequest("http://host/low1", net::LOWEST));
 
   scoped_ptr<net::URLRequest> url_request(
       NewURLRequest("http://host/low2", net::LOWEST));
   scoped_ptr<ResourceThrottle> throttle(scheduler_.ScheduleRequest(
       kChildId, kRouteId, url_request.get()));
   scoped_ptr<CancelingTestRequest> low2(new CancelingTestRequest(
       throttle.Pass(), url_request.Pass()));
   low2->Start();
 
   scoped_ptr<TestRequest> low3(NewRequest("http://host/low3", net::LOWEST));
   low2->set_request_to_cancel(low3.Pass());
   scoped_ptr<TestRequest> low4(NewRequest("http://host/low4", net::LOWEST));
 
   EXPECT_TRUE(high->started());
   EXPECT_FALSE(low2->started());
   high.reset();
   EXPECT_TRUE(low1->started());
   EXPECT_TRUE(low2->started());
   EXPECT_TRUE(low4->started());
 }
 
 }  // unnamed namespace
 
 }  // namespace content