sql::MetaTable.next_download_id, DownloadManager::GetNextId()

chrome/browser/profiles/profile_io_data.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/profile_io_data.h"
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 173 matching lines @@
     return profile->GetPrerenderManager();
   return NULL;
 }
 
 }  // namespace
 
 void ProfileIOData::InitializeProfileParams(Profile* profile) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   PrefService* pref_service = profile->GetPrefs();
 
+  next_download_id_thunk_ = profile->GetDownloadManager()->GetNextIdThunk();

[Randy Smith (Not in Mondays), 2011/07/28 21:03:16]

Why is this in ProfileIOData as well as ResourceContext?  Is it used from
ProfileIOData, and if so, can a resource_context_ indirection be substituted?

[benjhayden, 2011/08/03 17:44:46]

Yes, next_download_id_thunk_ is used in LazyInitialize(), which runs on the IO
thread so it can't GetDownloadManager() there.
Nothing accesses next_download_id_thunk_ through ProfileIOData's public
interface, so I removed that public method.

+
   scoped_ptr<ProfileParams> params(new ProfileParams);
   params->is_incognito = profile->IsOffTheRecord();
   params->clear_local_state_on_exit =
       pref_service->GetBoolean(prefs::kClearSiteDataOnExit);
 
   params->appcache_service = profile->GetAppCacheService();
 
   // Set up Accept-Language and Accept-Charset header values
   params->accept_language = net::HttpUtil::GenerateAcceptLanguageHeader(
       pref_service->GetString(prefs::kAcceptLanguages));
@@ skipping 292 matching lines @@
   resource_context_.set_database_tracker(database_tracker_);
   resource_context_.set_appcache_service(appcache_service_);
   resource_context_.set_blob_storage_context(blob_storage_context_);
   resource_context_.set_file_system_context(file_system_context_);
   resource_context_.set_quota_manager(quota_manager_);
   resource_context_.set_host_zoom_map(host_zoom_map_);
   resource_context_.set_prerender_manager_getter(prerender_manager_getter_);
   resource_context_.SetUserData(NULL, const_cast<ProfileIOData*>(this));
   resource_context_.set_media_observer(
       io_thread_globals->media.media_internals.get());
+  resource_context_.SetUserData(reinterpret_cast<void*>(
+      BASE_HASH_NAMESPACE::hash<std::string>()("next_download_id_thunk")),
+      &next_download_id_thunk_);

[Randy Smith (Not in Mondays), 2011/07/28 21:03:16]

I presume that the whole nasty reinterpret_cast<> business is because we can't
include download_manager.h in the ResourceContext declaration because that's in
content?  If so, put a TODO here to pull it back out once download_manager.h is
moved into content.

(What I'd *like* to say is that you should just include download_manager.h in
the ResourceContext header file, and jam can remove it when he does the move,
but that probably requires pulling him into this review and modifying the
exceptions file, and it probably just makes more sense to clean up once he's
done.)

[benjhayden, 2011/08/03 17:44:46]

Done.

 
   LazyInitializeInternal(profile_params_.get());
 
   profile_params_.reset();
   initialized_ = true;
 }
 
 void ProfileIOData::ApplyProfileParamsToContext(
     ChromeURLRequestContext* context) const {
   context->set_is_incognito(profile_params_->is_incognito);
@@ skipping 14 matching lines @@
       BrowserThread::IO, FROM_HERE,
       base::Bind(
           &ResourceDispatcherHost::CancelRequestsForContext,
           base::Unretained(g_browser_process->resource_dispatcher_host()),
           &resource_context_));
   bool posted = BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
                                         new ReleaseTask<ProfileIOData>(this));
   if (!posted)
     Release();
 }