Make extensions developer mode adhere to policy

chrome/browser/extensions/extension_service_test_base.h

 // Copyright 2014 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_EXTENSION_SERVICE_TEST_BASE_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_TEST_BASE_H_
 
 #include <stddef.h>
 
 #include <memory>
@@ skipping 17 matching lines @@
 #endif
 
 class ExtensionService;
 class Profile;
 class TestingProfile;
 
 namespace content {
 class BrowserContext;
 }
 
+namespace sync_preferences {
+class TestingPrefServiceSyncable;
+}
+
 namespace extensions {
 
 class ExtensionRegistry;
 class ManagementPolicy;
 
 // A unittest infrastructure which creates an ExtensionService. Whenever
 // possible, use this instead of creating a browsertest.
 // Note: Before adding methods to this class, please, please, please think about
 // whether they should go here or in a more specific subclass. Lots of things
 // need an ExtensionService, but they don't all need to know how you want yours
 // to be initialized.
 class ExtensionServiceTestBase : public testing::Test {
  public:
   struct ExtensionServiceInitParams {
     base::FilePath profile_path;
     base::FilePath pref_file;
     base::FilePath extensions_install_dir;
     bool autoupdate_enabled;    // defaults to false.
     bool is_first_run;          // defaults to true.
-    bool profile_is_supervised; // defaults to false.
+    bool profile_is_supervised;  // defaults to false.

[Andrew T Wilson (Slow), 2016/12/04 14:54:54]

I think intent is for comments to line up, so can you make them continue to line
up (remove extra space, or add space to previous ones)

[pmarko, 2016/12/07 16:12:11]

Done.

 
     // Though you could use this constructor, you probably want to use
     // CreateDefaultInitParams(), and then make a change or two.
     ExtensionServiceInitParams();
     ExtensionServiceInitParams(const ExtensionServiceInitParams& other);
   };
 
   // Public because parameterized test cases need it to be, or else the compiler
   // barfs.
   static void SetUpTestCase();  // faux-verride (static override).
@@ skipping 43 matching lines @@
   void ValidateStringPref(const std::string& extension_id,
                           const std::string& pref_path,
                           const std::string& expected_val);
 
   // TODO(rdevlin.cronin): Pull out more methods from ExtensionServiceTest that
   // are commonly used and/or reimplemented. For instance, methods to install
   // extensions from various locations, etc.
 
   content::BrowserContext* browser_context();
   Profile* profile();
+  sync_preferences::TestingPrefServiceSyncable* testing_pref_service();
   ExtensionService* service() { return service_; }
   ExtensionRegistry* registry() { return registry_; }
   const base::FilePath& extensions_install_dir() const {
     return extensions_install_dir_;
   }
   const base::FilePath& data_dir() const { return data_dir_; }
   const base::ScopedTempDir& temp_dir() const { return temp_dir_; }
 
  private:
   // Must be declared before anything that may make use of the
@@ skipping 44 matching lines @@
   chromeos::ScopedTestCrosSettings test_cros_settings_;
   chromeos::ScopedTestUserManager test_user_manager_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionServiceTestBase);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_SERVICE_TEST_BASE_H_