Do not load extension system in the Profile import process.

chrome/browser/extensions/component_loader.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_COMPONENT_LOADER_H_
 #define CHROME_BROWSER_EXTENSIONS_COMPONENT_LOADER_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 14 matching lines @@
  public:
   ComponentLoader(ExtensionServiceInterface* extension_service,
                   PrefService* prefs,
                   PrefService* local_state);
   virtual ~ComponentLoader();
 
   size_t registered_extensions_count() const {
     return component_extensions_.size();
   }
 
-  // Loads any registered component extensions.
-  void LoadAll();
+  // Loads registered component extensions. Extensions with background pages
+  // will be deferred until LoadDeferred if the argument is true.

[Yoyo Zhou, 2012/12/19 21:50:38]

LoadAllDeferred

[tapted, 2012/12/19 22:58:24]

Done.

+  void LoadAll(bool defer_extensions_with_background_pages);

[Yoyo Zhou, 2012/12/19 21:50:38]

Reading LoadAll(false) in the code isn't very self-descriptive. I'd prefer this
to be 2 functions, like LoadAllNow and LoadAllDeferringBackgroundPages (this
name isn't any good so feel free to improve it).

[tapted, 2012/12/19 22:58:24]

Done. I went with
- BulkLoadAll
- BulkLoadDeferBackgroundPages
- BulkLoadDeferred

(I also toyed with an enum in an earlier iteration, since true/false was giving
me an itch too).

+
+  // Load component extensions deferred in the last call to LoadAll(true).
+  void LoadAllDeferred();
 
   // Clear the list of all registered extensions and unloads them from the
   // extension service.
   void RemoveAll();
 
   // Registers and possibly loads a component extension. If ExtensionService
   // has been initialized, the extension is loaded; otherwise, the load is
   // deferred until LoadAll is called. The ID of the added extension is
   // returned.
   //
@@ skipping 57 matching lines @@
     FilePath root_directory;
 
     // The component extension's ID.
     std::string extension_id;
   };
 
   std::string Add(const DictionaryValue* parsed_manifest,
                   const FilePath& root_directory);
 
   // Loads a registered component extension.
-  const Extension* Load(const ComponentExtensionInfo& info);
+  void Load(const ComponentExtensionInfo& info,

[tapted, 2012/12/19 13:19:14]

(note: return type changed to void - nothing used it, and it raises fewer
questions this way)

+            bool defer_if_has_background_page);
 
   void AddDefaultComponentExtensionsWithBackgroundPages(
       bool skip_session_components);
   void AddFileManagerExtension();
 
 #if defined(OS_CHROMEOS)
   void AddGaiaAuthExtension();
 #endif
 
   // Add the enterprise webstore extension, or reload it if already loaded.
   void AddOrReloadEnterpriseWebStore();
 
   void AddChromeApp();
 
   // Unloads |component| from the memory.
   void UnloadComponent(ComponentExtensionInfo* component);
 
   PrefService* prefs_;
   PrefService* local_state_;
 
   ExtensionServiceInterface* extension_service_;
 
   // List of registered component extensions (see Extension::Location).
   typedef std::vector<ComponentExtensionInfo> RegisteredComponentExtensions;
   RegisteredComponentExtensions component_extensions_;
 
   PrefChangeRegistrar pref_change_registrar_;
 
+  // Extensions with background pages deferred by LoadAll(true).
+  typedef std::vector<scoped_refptr<const Extension> > DeferredAtLoadExtensions;
+  DeferredAtLoadExtensions deferred_at_load_extensions;
+
   DISALLOW_COPY_AND_ASSIGN(ComponentLoader);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_COMPONENT_LOADER_H_