Switch to direct use of OwnerSettingsServiceChromeOS::Set() in tests.

chrome/browser/chromeos/app_mode/kiosk_app_manager.h

 // Copyright 2013 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_CHROMEOS_APP_MODE_KIOSK_APP_MANAGER_H_
 #define CHROME_BROWSER_CHROMEOS_APP_MODE_KIOSK_APP_MANAGER_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 20 matching lines @@
 class Extension;
 class ExternalLoader;
 }
 
 namespace chromeos {
 
 class KioskAppData;
 class KioskAppExternalLoader;
 class KioskAppManagerObserver;
 class KioskExternalUpdater;
+class OwnerSettingsServiceChromeOS;
 
 // KioskAppManager manages cached app data.
 class KioskAppManager : public KioskAppDataDelegate,
                         public ExternalCache::Delegate {
  public:
   enum ConsumerKioskAutoLaunchStatus {
     // Consumer kiosk mode auto-launch feature can be enabled on this machine.
     CONSUMER_KIOSK_AUTO_LAUNCH_CONFIGURABLE,
     // Consumer kiosk auto-launch feature is enabled on this machine.
     CONSUMER_KIOSK_AUTO_LAUNCH_ENABLED,
@@ skipping 63 matching lines @@
   void EnableConsumerKioskAutoLaunch(
       const EnableKioskAutoLaunchCallback& callback);
 
   // Returns true if this device is consumer kiosk auto launch enabled.
   bool IsConsumerKioskDeviceWithAutoLaunch();
 
   // Returns auto launcher app id or an empty string if there is none.
   std::string GetAutoLaunchApp() const;
 
   // Sets |app_id| as the app to auto launch at start up.
-  void SetAutoLaunchApp(const std::string& app_id);
+  void SetAutoLaunchApp(const std::string& app_id,
+                        OwnerSettingsServiceChromeOS* service);

[Mattias Nissler (ping if slow), 2015/03/30 14:45:45]

Any thoughts regarding my earlier comment to move the write path out of
KioskAppManager?

[Ivan Podogov, 2015/03/31 08:39:59]

Yes, it is possible, although both of the browsertests will need either to be
rewritten from scratch, or moved to
chrome/browser/ui/webui/extensions/chromeos...

 
   // Returns true if there is a pending auto-launch request.
   bool IsAutoLaunchRequested() const;
 
   // Returns true if owner/policy enabled auto launch.
   bool IsAutoLaunchEnabled() const;
 
   // Enable auto launch setter.
   void SetEnableAutoLaunch(bool value);
 
   // Adds/removes a kiosk app by id. When removed, all locally cached data
   // will be removed as well.
-  void AddApp(const std::string& app_id);
-  void RemoveApp(const std::string& app_id);
+  void AddApp(const std::string& app_id, OwnerSettingsServiceChromeOS* service);
+  void RemoveApp(const std::string& app_id,
+                 OwnerSettingsServiceChromeOS* service);
 
   // Gets info of all apps that have no meta data load error.
   void GetApps(Apps* apps) const;
 
   // Gets app data for the given app id. Returns true if |app_id| is known and
   // |app| is populated. Otherwise, return false.
   bool GetApp(const std::string& app_id, App* app) const;
 
   // Gets whether the bailout shortcut is disabled.
   bool GetDisableBailoutShortcut() const;
@@ skipping 139 matching lines @@
   // The extension external loader for installing kiosk app.
   bool external_loader_created_;
   base::WeakPtr<KioskAppExternalLoader> external_loader_;
 
   DISALLOW_COPY_AND_ASSIGN(KioskAppManager);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_APP_MODE_KIOSK_APP_MANAGER_H_