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

Unified Diff: chrome/browser/chromeos/login/wizard_accessibility_handler.cc

Issue 6609008: Change other usages of .size() to .empty() when applicable. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Up 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/chromeos/login/wizard_accessibility_handler.cc
diff --git a/chrome/browser/chromeos/login/wizard_accessibility_handler.cc b/chrome/browser/chromeos/login/wizard_accessibility_handler.cc
index 16bb99b6f778970ac4c319fa80c7a5a9e5ca143a..539b0bd762876c1106158f9faa747fe755040349 100644
--- a/chrome/browser/chromeos/login/wizard_accessibility_handler.cc
+++ b/chrome/browser/chromeos/login/wizard_accessibility_handler.cc
@@ -430,13 +430,13 @@ void WizardAccessibilityHandler::DescribeTextContentsChanged(
new_indices[prefix_char_len],
new_indices[new_suffix_char_start] - new_indices[prefix_char_len]);
- if (inserted.size() > 0 && deleted.size() > 0) {
+ if (!inserted.empty() && !deleted.empty()) {
Peter Kasting 2011/03/03 19:35:53 Nit: It seems like this first conditional + arm ca
// Replace one substring with another, speak inserted text.
AppendUtterance(inserted, out_spoken_description);
- } else if (inserted.size() > 0) {
+ } else if (!inserted.empty()) {
// Speak inserted text.
AppendUtterance(inserted, out_spoken_description);
- } else if (deleted.size() > 0) {
+ } else if (!deleted.empty()) {
// Speak deleted text.
AppendUtterance(deleted, out_spoken_description);
}

Powered by Google App Engine
This is Rietveld 408576698