Adding a unique runtime Id to Profile objects, that can be used as the key...

chrome/browser/browsing_instance.h

 // 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.
 
 #ifndef CHROME_BROWSER_BROWSING_INSTANCE_H__
 #define CHROME_BROWSER_BROWSING_INSTANCE_H__
 
 #include "base/hash_tables.h"
 #include "base/logging.h"
 #include "base/ref_counted.h"
+#include "chrome/browser/profile.h"
 
 class GURL;
-class Profile;
 class SiteInstance;
 
 ///////////////////////////////////////////////////////////////////////////////
 //
 // BrowsingInstance class
 //
 // A browsing instance corresponds to the notion of a "unit of related browsing
 // contexts" in the HTML 5 spec.  Intuitively, it represents a collection of
 // tabs and frames that can have script connections to each other.  In that
 // sense, it reflects the user interface, and not the contents of the tabs and
@@ skipping 69 matching lines @@
   // if the user later visits a page from this site, within this
   // BrowsingInstance.
   void UnregisterSiteInstance(SiteInstance* site_instance);
 
  private:
   // Map of site to SiteInstance, to ensure we only have one SiteInstance per
   // site.  The site string should be the possibly_invalid_spec() of a GURL
   // obtained with SiteInstance::GetSiteForURL.
   typedef base::hash_map<std::string, SiteInstance*> SiteInstanceMap;
 
-  // Map of Profile to SiteInstanceMap, for use in the process-per-site model.
-  typedef base::hash_map<Profile*, SiteInstanceMap> ProfileSiteInstanceMap;
+  // Map of Profile runtime Id to SiteInstanceMap, for use in the
+  // process-per-site model.
+  typedef base::hash_map<ProfileId, SiteInstanceMap> ProfileSiteInstanceMap;
 
   // Returns a pointer to the relevant SiteInstanceMap for this object.  If the
   // process-per-site model is in use, or if process-per-site-instance is in
   // use and |url| matches a site for which we always use one process (e.g.,
   // the new tab page), then this returns the SiteInstanceMap for the entire
   // profile.  If not, this returns the BrowsingInstance's own private
   // SiteInstanceMap.
   SiteInstanceMap* GetSiteInstanceMap(Profile* profile, const GURL& url);
 
   // Common profile to which all SiteInstances in this BrowsingInstance
   // must belong.
   Profile* const profile_;
 
   // Map of site to SiteInstance, to ensure we only have one SiteInstance per
   // site.  The site string should be the possibly_invalid_spec() of a GURL
   // obtained with SiteInstance::GetSiteForURL.  Note that this map may not
   // contain every active SiteInstance, because a race exists where two
   // SiteInstances can be assigned to the same site.  This is ok in rare cases.
   // This field is only used if we are not using process-per-site.
   SiteInstanceMap site_instance_map_;
 
   // Global map of Profile to SiteInstanceMap, for process-per-site.
   static ProfileSiteInstanceMap profile_site_instance_map_;
 
   DISALLOW_EVIL_CONSTRUCTORS(BrowsingInstance);
 };
 
 #endif  //  CHROME_BROWSER_BROWSING_INSTANCE_H__