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

Unified Diff: chrome/browser/language_order_table_model.cc

Issue 174043: Gtk languages options page part 1: accept-languages configuration. (Closed)
Patch Set: rebase Created 11 years, 4 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 | « chrome/browser/language_order_table_model.h ('k') | chrome/browser/views/options/languages_page_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/language_order_table_model.cc
diff --git a/chrome/browser/language_order_table_model.cc b/chrome/browser/language_order_table_model.cc
index 50799d24b2df538d5e9192efa544e1bd82960dbd..e5bd403e78cec6d4ab624502b29de73bd5500591 100644
--- a/chrome/browser/language_order_table_model.cc
+++ b/chrome/browser/language_order_table_model.cc
@@ -32,17 +32,18 @@ std::wstring LanguageOrderTableModel::GetText(int row, int column_id) {
true));
}
-void LanguageOrderTableModel::Add(const std::string& language) {
+bool LanguageOrderTableModel::Add(const std::string& language) {
if (language.empty())
- return;
+ return false;
// Check for selecting duplicated language.
for (std::vector<std::string>::const_iterator cit = languages_.begin();
cit != languages_.end(); ++cit)
if (*cit == language)
- return;
+ return false;
languages_.push_back(language);
if (observer_)
observer_->OnItemsAdded(RowCount() - 1, 1);
+ return true;
}
void LanguageOrderTableModel::Remove(int index) {
« no previous file with comments | « chrome/browser/language_order_table_model.h ('k') | chrome/browser/views/options/languages_page_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698