Profile shouldn't own background page stuff.

chrome/browser/tab_contents/render_view_host_delegate_helper.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/tab_contents/render_view_host_delegate_helper.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/background_contents_service.h"
+#include "chrome/browser/background_contents_service_factory.h"
 #include "chrome/browser/character_encoding.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/gpu_data_manager.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/prefs/scoped_user_pref_update.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/tab_contents/background_contents.h"
 #include "chrome/browser/user_style_sheet_watcher.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
@@ skipping 34 matching lines @@
       extensions_service->GetExtensionByWebExtent(opener_url);
   if (!extension)
     return NULL;
 
   // If the extension manifest specifies a background page, then don't allow one
   // to be created here.
   if (extension->background_url().is_valid())
     return NULL;
 
   // Only allow a single background contents per app.
-  if (!profile->GetBackgroundContentsService() ||
-      profile->GetBackgroundContentsService()->GetAppBackgroundContents(
+  BackgroundContentsService* service =
+      BackgroundContentsServiceFactory::GetForProfile(profile);
+  if (!service || service->GetAppBackgroundContents(
           ASCIIToUTF16(extension->id())))
     return NULL;
 
   // Ensure that we're trying to open this from the extension's process.
   ExtensionProcessManager* process_manager =
       profile->GetExtensionProcessManager();
   if (!site->GetProcess() || !process_manager ||
       site->GetProcess() != process_manager->GetExtensionProcess(opener_url))
     return NULL;
 
   // Passed all the checks, so this should be created as a BackgroundContents.
-  return profile->GetBackgroundContentsService()->CreateBackgroundContents(
-      site, route_id, profile, frame_name, ASCIIToUTF16(extension->id()));
+  return service->CreateBackgroundContents(site, route_id, profile, frame_name,
+                                           ASCIIToUTF16(extension->id()));
 }
 
 TabContents* RenderViewHostDelegateViewHelper::CreateNewWindow(
     int route_id,
     Profile* profile,
     SiteInstance* site,
     WebUI::TypeID webui_type,
     RenderViewHostDelegate* opener,
     WindowContainerType window_container_type,
     const string16& frame_name) {
@@ skipping 283 matching lines @@
   DictionaryPrefUpdate update(profile->GetPrefs(),
                               prefs::kWebKitInspectorSettings);
   DictionaryValue* inspector_settings = update.Get();
   inspector_settings->SetWithoutPathExpansion(key,
                                               Value::CreateStringValue(value));
 }
 
 void RenderViewHostDelegateHelper::ClearInspectorSettings(Profile* profile) {
   profile->GetPrefs()->ClearPref(prefs::kWebKitInspectorSettings);
 }