DownloadId

chrome/browser/profiles/off_the_record_profile_impl.cc

 // Copyright (c) 2011 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 "chrome/browser/profiles/off_the_record_profile_impl.h"
 
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
@@ skipping 97 matching lines @@
   ChromePluginServiceFilter::GetInstance()->RegisterResourceContext(
     PluginPrefs::GetForProfile(this), &GetResourceContext());
 
   BrowserThread::PostTask(
     BrowserThread::IO, FROM_HERE,
     NewRunnableFunction(
       &NotifyOTRProfileCreatedOnIOThread, profile_, this));
 }
 
 OffTheRecordProfileImpl::~OffTheRecordProfileImpl() {
+  // Shutdown the DownloadManager here in the dtor as ProfileImpl does to
+  // guarantee that it happens before the last scoped_refptr<DM> is reaped.
+  // DownloadManager is lazily created, so check before accessing it.
+  if (download_manager_.get()) {
+    // Drop our download manager so we forget about all the downloads made
+    // in incognito mode.
+    download_manager_->Shutdown();
+    download_manager_ = NULL;
+  }
+
   NotificationService::current()->Notify(
     chrome::NOTIFICATION_PROFILE_DESTROYED, Source<Profile>(this),
     NotificationService::NoDetails());
 
   // There shouldn't be any browser window associated with this profile.
   CHECK(!BrowserList::FindBrowserWithProfile(this));
 
   ChromePluginServiceFilter::GetInstance()->UnregisterResourceContext(
     &GetResourceContext());
 
@@ skipping 404 matching lines @@
 void OffTheRecordProfileImpl::SetupChromeOSEnterpriseExtensionObserver() {
   profile_->SetupChromeOSEnterpriseExtensionObserver();
 }
 
 void OffTheRecordProfileImpl::InitChromeOSPreferences() {
   // The incognito profile shouldn't have Chrome OS's preferences.
   // The preferences are associated with the regular user profile.
 }
 #endif  // defined(OS_CHROMEOS)
 
-void OffTheRecordProfileImpl::ExitedOffTheRecordMode() {
-  // DownloadManager is lazily created, so check before accessing it.
-  if (download_manager_.get()) {
-    // Drop our download manager so we forget about all the downloads made
-    // in incognito mode.
-    download_manager_->Shutdown();
-    download_manager_ = NULL;
-  }
-}
-
 void OffTheRecordProfileImpl::OnBrowserAdded(const Browser* browser) {
 }
 
 void OffTheRecordProfileImpl::OnBrowserRemoved(const Browser* browser) {
-  if (BrowserList::GetBrowserCount(this) == 0)
-    ExitedOffTheRecordMode();
 }
 
 ChromeBlobStorageContext* OffTheRecordProfileImpl::GetBlobStorageContext() {
   if (!blob_storage_context_) {
     blob_storage_context_ = new ChromeBlobStorageContext();
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         NewRunnableMethod(
             blob_storage_context_.get(),
             &ChromeBlobStorageContext::InitializeOnIOThread));
@@ skipping 134 matching lines @@
   OffTheRecordProfileImpl* profile = NULL;
 #if defined(OS_CHROMEOS)
   if (Profile::IsGuestSession())
     profile = new GuestSessionProfile(this);
 #endif
   if (!profile)
     profile = new OffTheRecordProfileImpl(this);
   profile->Init();
   return profile;
 }