Enable extensions by default.

chrome/browser/renderer_host/browser_render_process_host.cc

Index: chrome/browser/renderer_host/browser_render_process_host.cc
diff --git a/chrome/browser/renderer_host/browser_render_process_host.cc b/chrome/browser/renderer_host/browser_render_process_host.cc
index a0b64c1fe0c93b78c90e4c7ebcd5ecf586ec5306..688fa824f5b63c5f9ed83191c0e17ff4bab8b579 100644
--- a/chrome/browser/renderer_host/browser_render_process_host.cc
+++ b/chrome/browser/renderer_host/browser_render_process_host.cc
@@ -358,25 +358,6 @@ bool BrowserRenderProcessHost::Init() {
     }
   }
 
-  // Tell the renderer to enable extensions if there are any extensions loaded.

[Erik does not do reviews, 2009/08/28 23:38:55]

do we need to explicitly pass along extensions-disabled to the renderer?

-  //
-  // NOTE: This is subtly different than just passing along whether
-  // --enable-extenisons is present in the browser process. For example, there
-  // is also an extensions.enabled preference, and there may be various special
-  // cases about whether to allow extensions to load.
-  //
-  // This introduces a race condition where the first renderer never gets
-  // extensions enabled, so we also set the flag if extensions_enabled(). This
-  // isn't perfect though, because of the special cases above.
-  //
-  // TODO(aa): We need to get rid of the need to pass this flag at all. It is
-  // only used in one place in the renderer.
-  if (profile()->GetExtensionsService()) {
-    if (profile()->GetExtensionsService()->extensions()->size() > 0 ||
-        profile()->GetExtensionsService()->extensions_enabled())
-      cmd_line.AppendSwitch(switches::kEnableExtensions);
-  }
-
   // Pass on the browser locale.
   const std::string locale = g_browser_process->GetApplicationLocale();
   cmd_line.AppendSwitchWithValue(switches::kLang, ASCIIToWide(locale));