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

Unified Diff: ui/views/controls/label_unittest.cc

Issue 108063004: Give up focus if the focused view becomes unfocusable (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years 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: ui/views/controls/label_unittest.cc
diff --git a/ui/views/controls/label_unittest.cc b/ui/views/controls/label_unittest.cc
index b64d98c5973f355a901c77806d31e9e54b9bb694..50f11f7442af8c4e3cb907b52420842c7cbd3f30 100644
--- a/ui/views/controls/label_unittest.cc
+++ b/ui/views/controls/label_unittest.cc
@@ -201,7 +201,7 @@ TEST(LabelTest, MultilineSmallAvailableWidthSizing) {
TEST(LabelTest, MultiLineSizing) {
Label label;
- label.set_focusable(false);
+ label.SetFocusable(false);
string16 test_text(
ASCIIToUTF16("A random string\nwith multiple lines\nand returns!"));
label.SetText(test_text);
@@ -322,7 +322,7 @@ TEST(LabelTest, AutoDetectDirectionality) {
TEST(LabelTest, DrawSingleLineString) {
Label label;
- label.set_focusable(false);
+ label.SetFocusable(false);
// Turn off mirroring so that we don't need to figure out if
// align right really means align left.
@@ -458,7 +458,7 @@ TEST(LabelTest, DrawSingleLineString) {
// multiline lables to not ellide in Linux only.
TEST(LabelTest, DrawMultiLineString) {
Label label;
- label.set_focusable(false);
+ label.SetFocusable(false);
// Turn off mirroring so that we don't need to figure out if
// align right really means align left.
@@ -599,7 +599,7 @@ TEST(LabelTest, DrawMultiLineString) {
TEST(LabelTest, DrawSingleLineStringInRTL) {
Label label;
- label.set_focusable(false);
+ label.SetFocusable(false);
std::string locale = l10n_util::GetApplicationLocale("");
base::i18n::SetICUDefaultLocale("he");
@@ -738,7 +738,7 @@ TEST(LabelTest, DrawSingleLineStringInRTL) {
// multiline lables to not ellide in Linux only.
TEST(LabelTest, DrawMultiLineStringInRTL) {
Label label;
- label.set_focusable(false);
+ label.SetFocusable(false);
// Test for RTL.
std::string locale = l10n_util::GetApplicationLocale("");

Powered by Google App Engine
This is Rietveld 408576698