Extends IBusText I/O function.

chromeos/dbus/ibus/ibus_text.cc

Index: chromeos/dbus/ibus/ibus_text.cc
diff --git a/chromeos/dbus/ibus/ibus_text.cc b/chromeos/dbus/ibus/ibus_text.cc
index b2eae1590657cf6d8d98749d0e1e0f87e82f6788..c46c5e1a77fc5b4ff3485cad81802f694a26d4c6 100644
--- a/chromeos/dbus/ibus/ibus_text.cc
+++ b/chromeos/dbus/ibus/ibus_text.cc
@@ -93,6 +93,13 @@ void AppendIBusText(const IBusText& ibus_text, dbus::MessageWriter* writer) {
   ibus_text_writer.CloseAll();
 }
 
+void CHROMEOS_EXPORT AppendStringAsIBusText(const std::string& text,
+                                            dbus::MessageWriter* writer) {
+  IBusText ibus_text;
+  ibus_text.set_text(text);
+  AppendIBusText(ibus_text, writer);
+}
+
 bool PopIBusText(dbus::MessageReader* reader, IBusText* ibus_text) {
   IBusObjectReader ibus_text_reader("IBusText", reader);
   if (!ibus_text_reader.Init())
@@ -153,6 +160,14 @@ bool PopIBusText(dbus::MessageReader* reader, IBusText* ibus_text) {
   return true;
 }
 
+bool CHROMEOS_EXPORT PopStringFromIBusText(dbus::MessageReader* reader,
+                                           std::string* text) {
+  IBusText ibus_text;
+  bool result = PopIBusText(reader, &ibus_text);
+  text->assign(ibus_text.text());

[satorux1, 2012/05/18 01:11:06]

Should we do this when |result| is false?

Maybe:

if (PopIBusText(reader, &ibus_text)) {
  *text = ibus_text.text()
  return true;
}
return false;

[Seigo Nonaka, 2012/05/18 01:18:43]

Done
On 2012/05/18 01:11:06, satorux1 wrote:

+  return result;
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 // IBusText
 IBusText::IBusText()