Test extension reloading behavior.

chrome/browser/chrome_content_browser_client.cc

 // Copyright (c) 2012 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/chrome_content_browser_client.h"
 
 #include <set>
 #include <utility>
 #include <vector>
 
@@ skipping 880 matching lines @@
   if (!extensions::BackgroundInfo::HasBackgroundPage(extension))
     return false;
 
   std::set<int> process_ids;
   size_t max_process_count =
       content::RenderProcessHost::GetMaxRendererProcessCount();
 
   // Go through all profiles to ensure we have total count of extension
   // processes containing background pages, otherwise one profile can
   // starve the other.
-  std::vector<Profile*> profiles = g_browser_process->profile_manager()->
-      GetLoadedProfiles();
+  ProfileManager* profile_manager = g_browser_process->profile_manager();
+  // Tests may not set up a profile manager.

[Matt Perry, 2013/04/04 20:54:26]

Can we fix that instead?

[Paweł Hajdan Jr., 2013/04/04 21:30:05]

Probably, see TestingBrowserProcess::SetProfileManager.

[Jeffrey Yasskin, 2013/04/05 13:14:01]

Some of the uses of that function re-set it to NULL when they're done, and other
don't. I'll reset it in this test, but it could probably use a better API.

+  if (profile_manager == NULL)
+    return false;
+
+  std::vector<Profile*> profiles = profile_manager->GetLoadedProfiles();
   for (size_t i = 0; i < profiles.size(); ++i) {
     ExtensionProcessManager* epm =
         extensions::ExtensionSystem::Get(profiles[i])->process_manager();
     for (ExtensionProcessManager::const_iterator iter =
              epm->background_hosts().begin();
          iter != epm->background_hosts().end(); ++iter) {
       const extensions::ExtensionHost* host = *iter;
       process_ids.insert(host->render_process_host()->GetID());
     }
   }
@@ skipping 1267 matching lines @@
     io_thread_application_locale_ = locale;
 }
 
 void ChromeContentBrowserClient::SetApplicationLocaleOnIOThread(
     const std::string& locale) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   io_thread_application_locale_ = locale;
 }
 
 }  // namespace chrome