FBTF: Move virtual methods to implementation files.

chrome/browser/browsing_instance.h

 // Copyright (c) 2010 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_
 #pragma once
 
 #include "base/hash_tables.h"
-#include "base/logging.h"
 #include "base/ref_counted.h"
 #include "chrome/browser/profile.h"
 
 class GURL;
 class SiteInstance;
 
 ///////////////////////////////////////////////////////////////////////////////
 //
 // BrowsingInstance class
 //
@@ skipping 28 matching lines @@
 // Currently, the BrowsingInstance class is not visible outside of the
 // SiteInstance class.  To get a new SiteInstance that is part of the same
 // BrowsingInstance, use SiteInstance::GetRelatedSiteInstance.  Because of
 // this, BrowsingInstances and SiteInstances are tested together in
 // site_instance_unittest.cc.
 //
 ///////////////////////////////////////////////////////////////////////////////
 class BrowsingInstance : public base::RefCounted<BrowsingInstance> {
  public:
   // Create a new BrowsingInstance.
-  explicit BrowsingInstance(Profile* profile)
-      : profile_(profile) {
-  }
+  explicit BrowsingInstance(Profile* profile);
 
   // Returns whether the process-per-site model is in use (globally or just for
   // the given url), in which case we should ensure there is only one
   // SiteInstance per site for the entire profile, not just for this
   // BrowsingInstance.
   virtual bool ShouldUseProcessPerSite(const GURL& url);
 
   // Get the profile to which this BrowsingInstance belongs.
   Profile* profile() { return profile_; }
 
@@ skipping 13 matching lines @@
   // Removes the given SiteInstance from our map, after all references to it
   // have been deleted.  This means it is safe to create a new SiteInstance
   // if the user later visits a page from this site, within this
   // BrowsingInstance.
   void UnregisterSiteInstance(SiteInstance* site_instance);
 
  protected:
   friend class base::RefCounted<BrowsingInstance>;
 
   // Virtual to allow tests to extend it.
-  virtual ~BrowsingInstance() {
-    // We should only be deleted when all of the SiteInstances that refer to
-    // us are gone.
-    DCHECK(site_instance_map_.empty());
-  }
+  virtual ~BrowsingInstance();
 
  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 runtime Id to SiteInstanceMap, for use in the
   // process-per-site model.
   typedef base::hash_map<ProfileId, SiteInstanceMap> ProfileSiteInstanceMap;
@@ skipping 18 matching lines @@
   // 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_COPY_AND_ASSIGN(BrowsingInstance);
 };
 
 #endif  // CHROME_BROWSER_BROWSING_INSTANCE_H_