Alter settings enforcement default to GROUP_NO_ENFORCEMENT in absence of Finch seed.

chrome/browser/extensions/extension_startup_browsertest.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 <vector>
 
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/files/file_path.h"
 #include "base/path_service.h"
@@ skipping 45 matching lines @@
       // extension settings to be loaded from Preferences as if they had been
       // authenticated correctly before they were handed to the ExtensionSystem.
       command_line->AppendSwitchASCII(
           switches::kForceFieldTrials,
           base::StringPrintf(
               "%s/%s/",
               chrome_prefs::internals::kSettingsEnforcementTrialName,
               chrome_prefs::internals::kSettingsEnforcementGroupNoEnforcement));
 #if defined(OFFICIAL_BUILD) && defined(OS_WIN)
       // In Windows official builds, it is not possible to disable settings
       // authentication.

[Bernhard Bauer, 2014/06/02 21:13:32]

Does this comment need to be updated?

[robertshield, 2014/06/02 21:16:43]

I believe the comment remains accurate as it describes the need for the
|unauthenticated_load_allowed_| override to exist, rather than the value it is
set to. 

[Bernhard Bauer, 2014/06/02 21:30:46]

Hm, it is already initialized to true in line 47, so this whole line isn't
necessary (and then the variable could be removed completely, now that I think
about it). 

Or do you want to keep this CL small, for merges?

[robertshield, 2014/06/02 22:30:06]

As discussed, would prefer to keep this small for merges. This code will be
revisited very shortly, will clean up these tests if possible then.

-      unauthenticated_load_allowed_ = false;
+      unauthenticated_load_allowed_ = true;
 #endif
     } else {
       base::FilePath::StringType paths = JoinString(load_extensions_, ',');
       command_line->AppendSwitchNative(switches::kLoadExtension,
                                        paths);
       command_line->AppendSwitch(switches::kDisableExtensionsFileAccessCheck);
     }
   }
 
   virtual bool SetUpUserDataDirectory() OVERRIDE {
@@ skipping 215 matching lines @@
         .AppendASCII("extensions")
         .AppendASCII("app2");
     load_extensions_.push_back(fourth_extension_path.value());
   }
 };
 
 IN_PROC_BROWSER_TEST_F(ExtensionsLoadMultipleTest, Test) {
   WaitForServicesToStart(4, true);
   TestInjection(true, true);
 }