Extends IBusObject.

chromeos/dbus/ibus/ibus_object.cc

Index: chromeos/dbus/ibus/ibus_object.cc
diff --git a/chromeos/dbus/ibus/ibus_object.cc b/chromeos/dbus/ibus/ibus_object.cc
index e81fcee67f4f81f80063b6dfe44bbdba2d203b73..b8599e5a70d711263e3ead8a915dbcf973fe40b6 100644
--- a/chromeos/dbus/ibus/ibus_object.cc
+++ b/chromeos/dbus/ibus/ibus_object.cc
@@ -5,10 +5,18 @@
 #include "chromeos/dbus/ibus/ibus_object.h"
 
 #include "base/logging.h"
+#include "chromeos/dbus/ibus/ibus_text.h"
 #include "dbus/message.h"
 
 namespace chromeos {
 
+// TODO(nona): Remove after complete libibus removal.

[satorux1, 2012/05/21 19:58:14]

We should put the code in the file in 'ibus' namespace too

[Seigo Nonaka, 2012/05/21 20:12:00]

Done.

+using chromeos::ibus::IBusText;
+using chromeos::ibus::PopIBusText;
+using chromeos::ibus::PopStringFromIBusText;
+using chromeos::ibus::AppendIBusText;
+using chromeos::ibus::AppendStringAsIBusText;
+
 ///////////////////////////////////////////////////////////////////////////////
 // IBusObjectReader
 IBusObjectReader::IBusObjectReader(const std::string& type_name,
@@ -110,6 +118,19 @@ bool IBusObjectReader::PopArray(dbus::MessageReader* reader) {
   return IsValid() && contents_reader_->PopArray(reader);
 }
 
+bool IBusObjectReader::PopIBusText(IBusText* text) {
+  DCHECK_NE(IBUS_OBJECT_NOT_CHECKED, check_result_);
+  DCHECK(contents_reader_.get());
+  return IsValid() && chromeos::PopIBusText(contents_reader_.get(), text);
+}
+
+bool IBusObjectReader::PopStringFromIBusText(std::string* text) {
+  DCHECK_NE(IBUS_OBJECT_NOT_CHECKED, check_result_);
+  DCHECK(contents_reader_.get());
+  return IsValid() && chromeos::PopStringFromIBusText(contents_reader_.get(),
+                                                      text);
+}
+
 bool IBusObjectReader::HasMoreData() {
   DCHECK_NE(IBUS_OBJECT_NOT_CHECKED, check_result_);
   DCHECK(contents_reader_.get());
@@ -128,16 +149,6 @@ bool IBusObjectReader::IsValid() const {
   return check_result_ == IBUS_OBJECT_VALID;
 }
 
-dbus::MessageReader* IBusObjectReader::GetContentsReader() {
-  DCHECK_NE(IBUS_OBJECT_NOT_CHECKED, check_result_);
-  DCHECK(contents_reader_.get());
-  if (IsValid()) {
-    return contents_reader_.get();
-  } else {
-    return NULL;
-  }
-}
-
 ///////////////////////////////////////////////////////////////////////////////
 //  IBusObjectWriter
 IBusObjectWriter::IBusObjectWriter(const std::string& type_name,
@@ -179,6 +190,16 @@ void IBusObjectWriter::OpenArray(const std::string& signature,
   contents_writer_->OpenArray(signature, writer);
 }
 
+void IBusObjectWriter::AppendIBusText(const IBusText& text) {
+  DCHECK(IsInitialized());
+  chromeos::AppendIBusText(text, contents_writer_.get());
+}
+
+void IBusObjectWriter::AppendStringAsIBusText(const std::string& text) {
+  DCHECK(IsInitialized());
+  chromeos::AppendStringAsIBusText(text, contents_writer_.get());
+}
+
 void IBusObjectWriter::CloseContainer(dbus::MessageWriter* writer) {
   DCHECK(IsInitialized());
   contents_writer_->CloseContainer(writer);
@@ -230,13 +251,4 @@ bool IBusObjectWriter::IsInitialized() const {
   return contents_writer_.get() != NULL;
 }
 
-dbus::MessageWriter* IBusObjectWriter::GetContentsWriter() {
-  DCHECK(contents_writer_.get());
-  if (IsInitialized()) {
-    return contents_writer_.get();
-  } else {
-    return NULL;
-  }
-}
-
 }  // namespace chromeos