Stop using renderer specific host ids in ResourceDispatcher. This allows it ...

chrome/browser/site_instance_unittest.cc

 // Copyright (c) 2006-2008 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 "base/string16.h"
 #include "chrome/browser/renderer_host/browser_render_process_host.h"
 #include "chrome/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/tab_contents/navigation_entry.h"
 #include "chrome/browser/tab_contents/web_contents.h"
 #include "chrome/common/render_messages.h"
@@ skipping 159 matching lines @@
   EXPECT_EQ(-1, instance.get()->max_page_id());
 
   // Make sure max_page_id_ is monotonically increasing.
   instance.get()->UpdateMaxPageID(3);
   instance.get()->UpdateMaxPageID(1);
   EXPECT_EQ(3, instance.get()->max_page_id());
 }
 
 // Test to ensure GetProcess returns and creates processes correctly.
 TEST_F(SiteInstanceTest, GetProcess) {
-  // Ensure that GetProcess returns the process based on its host id.
+  // Ensure that GetProcess returns a process.
   scoped_ptr<TestingProfile> profile(new TestingProfile());
-  scoped_ptr<BrowserRenderProcessHost> host1(
-      new BrowserRenderProcessHost(profile.get()));
+  scoped_ptr<RenderProcessHost> host1;
   scoped_refptr<SiteInstance> instance(
       SiteInstance::CreateSiteInstance(profile.get()));
-  instance.get()->set_process_host_id(host1.get()->host_id());
-  EXPECT_EQ(host1.get(), instance.get()->GetProcess());
+  host1.reset(instance.get()->GetProcess());
+  EXPECT_TRUE(host1.get() != NULL);
 
-  // Ensure that GetProcess creates a new process if no host id is set.
+  // Ensure that GetProcess creates a new process.
   scoped_refptr<SiteInstance> instance2(
       SiteInstance::CreateSiteInstance(profile.get()));
   scoped_ptr<RenderProcessHost> host2(instance2.get()->GetProcess());
   EXPECT_TRUE(host2.get() != NULL);
   EXPECT_NE(host1.get(), host2.get());
 }
 
 // Test to ensure SetSite and site() work properly.
 TEST_F(SiteInstanceTest, SetSite) {
   scoped_refptr<SiteInstance> instance(SiteInstance::CreateSiteInstance(NULL));
@@ skipping 189 matching lines @@
       GURL("http://www.yahoo.com")));  // different BI, but same profile
 
   // Should be able to see that we don't have SiteInstances.
   EXPECT_FALSE(browsing_instance->HasSiteInstance(
       GURL("https://www.google.com")));  // not visited before
   EXPECT_FALSE(browsing_instance3->HasSiteInstance(
       GURL("http://www.yahoo.com")));  // different BI, different profile
 
   // browsing_instances will be deleted when their SiteInstances are deleted
 }