Chromium Code Reviews| 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); |
| } |