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

Unified Diff: chrome/browser/ui/gtk/options/passwords_page_gtk.cc

Issue 6609008: Change other usages of .size() to .empty() when applicable. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Peter nits Created 9 years, 10 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
Index: chrome/browser/ui/gtk/options/passwords_page_gtk.cc
diff --git a/chrome/browser/ui/gtk/options/passwords_page_gtk.cc b/chrome/browser/ui/gtk/options/passwords_page_gtk.cc
index 6eb3b0b283da4a8c245c056dff07582ead17a633..e26cea09bd31f3734113d5c71c6585122ecda316 100644
--- a/chrome/browser/ui/gtk/options/passwords_page_gtk.cc
+++ b/chrome/browser/ui/gtk/options/passwords_page_gtk.cc
@@ -165,7 +165,7 @@ void PasswordsPageGtk::SetPasswordList(
UTF16ToUTF8(net::FormatUrl(result[i]->origin, languages)).c_str(),
COL_USERNAME, UTF16ToUTF8(result[i]->username_value).c_str(), -1);
}
- gtk_widget_set_sensitive(remove_all_button_, result.size() > 0);
+ gtk_widget_set_sensitive(remove_all_button_, !result.empty());
}
void PasswordsPageGtk::HidePassword() {
@@ -222,7 +222,7 @@ void PasswordsPageGtk::OnRemoveButtonClicked(GtkWidget* widget) {
delete password_list_[index];
password_list_.erase(password_list_.begin() + index);
- gtk_widget_set_sensitive(remove_all_button_, password_list_.size() > 0);
+ gtk_widget_set_sensitive(remove_all_button_, !password_list_.empty());
}
void PasswordsPageGtk::OnRemoveAllButtonClicked(GtkWidget* widget) {
« no previous file with comments | « chrome/browser/ui/gtk/options/passwords_exceptions_page_gtk.cc ('k') | chrome/browser/ui/gtk/rounded_window.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698