Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(120)

Unified Diff: chromeos/dbus/ibus/ibus_lookup_table.cc

Issue 11783085: Extends IBusLookupTable to handle IsEqual/CopyFrom. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix comment Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chromeos/dbus/ibus/ibus_lookup_table.h ('k') | chromeos/dbus/ibus/ibus_lookup_table_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/dbus/ibus/ibus_lookup_table.cc
diff --git a/chromeos/dbus/ibus/ibus_lookup_table.cc b/chromeos/dbus/ibus/ibus_lookup_table.cc
index eba15108d3ff6caec558aac014f3e738126b5796..e44d046611b2d16dade9a8352624f1cbd99f712c 100644
--- a/chromeos/dbus/ibus/ibus_lookup_table.cc
+++ b/chromeos/dbus/ibus/ibus_lookup_table.cc
@@ -184,13 +184,45 @@ IBusLookupTable::IBusLookupTable()
: page_size_(kDefaultPageSize),
cursor_position_(0),
is_cursor_visible_(true),
- orientation_(IBUS_LOOKUP_TABLE_ORIENTATION_HORIZONTAL),
+ orientation_(HORIZONTAL),
show_window_at_composition_(false) {
}
IBusLookupTable::~IBusLookupTable() {
}
+bool IBusLookupTable::IsEqual(const IBusLookupTable& table) const {
+ if (page_size_ != table.page_size_ ||
+ cursor_position_ != table.cursor_position_ ||
+ is_cursor_visible_ != table.is_cursor_visible_ ||
+ orientation_ != table.orientation_ ||
+ show_window_at_composition_ != table.show_window_at_composition_ ||
+ candidates_.size() != table.candidates_.size())
+ return false;
+
+ for (size_t i = 0; i < candidates_.size(); ++i) {
+ const Entry& left = candidates_[i];
+ const Entry& right = table.candidates_[i];
+ if (left.value != right.value ||
+ left.label != right.label ||
+ left.annotation != right.annotation ||
+ left.description_title != right.description_title ||
+ left.description_body != right.description_body)
+ return false;
+ }
+ return true;
+}
+
+void IBusLookupTable::CopyFrom(const IBusLookupTable& table) {
+ page_size_ = table.page_size_;
+ cursor_position_ = table.cursor_position_;
+ is_cursor_visible_ = table.is_cursor_visible_;
+ orientation_ = table.orientation_;
+ show_window_at_composition_ = table.show_window_at_composition_;
+ candidates_.clear();
+ candidates_ = table.candidates_;
+}
+
IBusLookupTable::Entry::Entry() {
}
« no previous file with comments | « chromeos/dbus/ibus/ibus_lookup_table.h ('k') | chromeos/dbus/ibus/ibus_lookup_table_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698