Make sure sideload wipeout doesn't interfere with the tests.

chrome/browser/extensions/extension_service.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/extensions/extension_service.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/basictypes.h"
@@ skipping 558 matching lines @@
   CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   DCHECK(!ready_);  // Can't redo init.
   DCHECK_EQ(extensions_.size(), 0u);
 
   component_loader_->LoadAll();
   extensions::InstalledLoader(this).LoadAllExtensions();
 
   // The Sideload Wipeout effort takes place during load (see above), so once
   // that is done the flag can be set so that we don't have to check again.
+#if defined(OS_WIN)

[Aaron Boodman, 2012/10/23 15:20:20]

See other comment about OS_WIN ifdef.

   if (FeatureSwitch::sideload_wipeout()->IsEnabled())
     extension_prefs_->SetSideloadWipeoutDone();
+#endif
 
   // If we are running in the import process, don't bother initializing the
   // extension service since this can interfere with the main browser process
   // that is already running an extension service for this profile.
   // TODO(aa): can we start up even less of ExtensionService?
   // http://crbug.com/107636
   if (!CommandLine::ForCurrentProcess()->HasSwitch(switches::kImport) &&
       !CommandLine::ForCurrentProcess()->HasSwitch(switches::kImportFromFile)) {
     if (g_browser_process->profile_manager() &&
         g_browser_process->profile_manager()->will_import()) {
@@ skipping 1586 matching lines @@
     extension_prefs_->SetExtensionState(extension->id(), Extension::DISABLED);
     extension_prefs_->SetDidExtensionEscalatePermissions(extension, true);
     extension_prefs_->AddDisableReason(
         extension->id(),
         static_cast<Extension::DisableReason>(disable_reasons));
   }
 }
 
 void ExtensionService::MaybeWipeout(
     const extensions::Extension* extension) {
+#if !defined(OS_WIN)

[Aaron Boodman, 2012/10/23 15:20:20]

Better to chokepoint this in the feature switch itself. See:
http://git.chromium.org/gitweb/?p=chromium.git;a=blob;f=chrome/common/extensi...

+  // We are targeting registry-installed extensions, which are Windows-specific,
+  // and extensions marked internal and not from the web store, which are mostly
+  // problematic on Windows.
+  return;
+#endif
+
   if (!FeatureSwitch::sideload_wipeout()->IsEnabled())
     return;
 
   bool done = extension_prefs_->GetSideloadWipeoutDone();
   if (done)
     return;
 
   int disable_reasons = extension_prefs_->GetDisableReasons(extension->id());
   if (disable_reasons == Extension::DISABLE_NONE) {
     Extension::Location location = extension->location();
@@ skipping 588 matching lines @@
     // enabling them.
     if (Extension::IsExternalLocation(extension->location()) &&
         !extension_prefs_->IsExternalExtensionAcknowledged(extension->id())) {
       return false;
     }
   }
 #endif
 
   return true;
 }