Extends DBusThreadManager to connect ibus-bus.

chrome/browser/chromeos/input_method/ibus_controller_impl.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_INPUT_METHOD_IBUS_CONTROLLER_IMPL_H_
 #define CHROME_BROWSER_CHROMEOS_INPUT_METHOD_IBUS_CONTROLLER_IMPL_H_
 #pragma once
 
 #include <gio/gio.h>  // GAsyncResult and related types.
 #include <glib-object.h>
@@ skipping 97 matching lines @@
   void ConnectBusSignals();
 
   // Starts listening to the "focus-in", "register-properties", and
   // "update-property" D-Bus signals from ibus-daemon.
   void ConnectPanelServiceSignals();
 
   // Starts ibus-daemon if it's not yet started and |should_launch_daemon_| is
   // true.
   bool MaybeLaunchIBusDaemon();
 
+  // Starts ibus-daemon.
+  void LaunchIBusDaemon(const std::string& ibus_address);
+
   // Launches an input method procsess specified by the given command
   // line. On success, returns true and stores the process handle in
   // |process_handle|. Otherwise, returns false, and the contents of
   // |process_handle| is untouched. |watch_func| will be called when the
   // process terminates.
   bool LaunchProcess(const std::string& command_line,
                      base::ProcessHandle* process_handle,
                      GChildWatchFunc watch_func);
 
+  // Called after the connection to ibus-daemon is established.
+  void IBusDaemonInitializationDone();
+
   // A callback function that will be called when ibus_config_set_value_async()
   // request is finished.
   static void SetInputMethodConfigCallback(GObject* source_object,
                                            GAsyncResult* res,
                                            gpointer user_data);
 
   // Called when the input method process is shut down.
   static void OnIBusDaemonExit(GPid pid,
                                gint status,
                                IBusControllerImpl* controller);
@@ skipping 13 matching lines @@
   // Current input context path.
   std::string current_input_context_path_;
 
   // The input method ID which is currently selected. The ID is sent to the
   // daemon when |ibus_| and |ibus_config_| connections are both established.
   std::string current_input_method_id_;
 
   // An object which knows all valid input methods and layout IDs.
   InputMethodWhitelist whitelist_;
 
+  base::WeakPtrFactory<IBusControllerImpl> weak_ptr_factory_;

[Yusuke Sato, 2012/04/27 02:12:13]

add comment. please explain why this is necessary.

[Seigo Nonaka, 2012/05/01 00:19:54]

Done.

+
+  // true if ibus-daemon is initializing. The initializing steps are as follows.
+  // 1. Starts watching ibus address file.
+  // 2. Launches ibus-daemon and ibus-daemon write ibus address file if ready.
+  // 3. Gets notification of changing address file, then creates connection.
+  // 4. Deletes watcher.
+  bool is_initializing_ibus_daemon_;

[Yusuke Sato, 2012/04/27 02:12:13]

I don't want to have two similar boolean flags here, should_launch_daemon_ and
is_initializing_ibus_daemon_. Can you consolidate them? Having one enum variable
e.g. DaemonStatus or something like that would be sufficient.

[Seigo Nonaka, 2012/05/01 00:19:54]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(IBusControllerImpl);
 };
 
 }  // namespace input_method
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_INPUT_METHOD_IBUS_CONTROLLER_IMPL_H_