Fix UI-thread blocking issue in SetImeConfig

chrome/browser/chromeos/cros/input_method_library.h

 // Copyright (c) 2010 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_CROS_INPUT_METHOD_LIBRARY_H_
 #define CHROME_BROWSER_CHROMEOS_CROS_INPUT_METHOD_LIBRARY_H_
 #pragma once
 
 #include <string>
 #include <utility>
@@ skipping 66 matching lines @@
   virtual void SetImePropertyActivated(const std::string& key,
                                        bool activated) = 0;
 
   // Returns true if the input method specified by |input_method_id| is active.
   virtual bool InputMethodIsActivated(const std::string& input_method_id) = 0;
 
   // Get a configuration of ibus-daemon or IBus engines and stores it on
   // |out_value|. Returns true if |out_value| is successfully updated.
   // When you would like to retrieve 'panel/custom_font', |section| should
   // be "panel", and |config_name| should be "custom_font".
-  virtual bool GetImeConfig(
-      const char* section, const char* config_name,
-      ImeConfigValue* out_value) = 0;
+  virtual bool GetImeConfig(const std::string& section,
+                            const std::string& config_name,
+                            ImeConfigValue* out_value) = 0;
 
   // Updates a configuration of ibus-daemon or IBus engines with |value|.
   // Returns true if the configuration (and all pending configurations, if any)
   // are processed. If ibus-daemon is not running, this function just queues
   // the request and returns false.
   // You can specify |section| and |config_name| arguments in the same way
   // as GetImeConfig() above.
   // Notice: This function might call the Observer::ActiveInputMethodsChanged()
   // callback function immediately, before returning from the SetImeConfig
   // function. See also http://crosbug.com/5217.
-  virtual bool SetImeConfig(const char* section,
-                            const char* config_name,
+  virtual bool SetImeConfig(const std::string& section,
+                            const std::string& config_name,
                             const ImeConfigValue& value) = 0;
 
   // Sets the IME state to enabled, and launches its processes if needed.
   virtual void StartInputMethodProcesses() = 0;
 
   // Disables the IME, and kills the processes if they are running.
   virtual void StopInputMethodProcesses() = 0;
 
   // Controls whether the IME process is started when preload engines are
   // specificed, or defered until a non-default method is activated.
@@ skipping 10 matching lines @@
   virtual const ImePropertyList& current_ime_properties() const = 0;
 
   // Factory function, creates a new instance and returns ownership.
   // For normal usage, access the singleton via CrosLibrary::Get().
   static InputMethodLibrary* GetImpl(bool stub);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_CROS_INPUT_METHOD_LIBRARY_H_