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

Unified Diff: chrome/browser/ui/views/profiles/profile_chooser_view_browsertest.cc

Issue 1226093005: Give ProfileChooserView signin button focus. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments. Created 5 years, 5 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/ui/views/profiles/profile_chooser_view.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/profiles/profile_chooser_view_browsertest.cc
diff --git a/chrome/browser/ui/views/profiles/profile_chooser_view_browsertest.cc b/chrome/browser/ui/views/profiles/profile_chooser_view_browsertest.cc
index 22ec640e2c9d9449ec6fe870128824935e71355b..b7550be879d184fad51ed67ffd6af4b15fcde7c6 100644
--- a/chrome/browser/ui/views/profiles/profile_chooser_view_browsertest.cc
+++ b/chrome/browser/ui/views/profiles/profile_chooser_view_browsertest.cc
@@ -168,6 +168,10 @@ class ProfileChooserViewExtensionsTest : public ExtensionBrowserTest {
return ProfileChooserView::profile_bubble_;
}
+ views::View* signin_current_profile_link() {
+ return ProfileChooserView::profile_bubble_->signin_current_profile_link_;
+ }
+
void ShowSigninView() {
DCHECK(current_profile_bubble());
DCHECK(current_profile_bubble()->avatar_menu_);
@@ -183,7 +187,13 @@ class ProfileChooserViewExtensionsTest : public ExtensionBrowserTest {
DISALLOW_COPY_AND_ASSIGN(ProfileChooserViewExtensionsTest);
};
-// crbug.com/502370
+IN_PROC_BROWSER_TEST_F(ProfileChooserViewExtensionsTest, SigninButtonHasFocus) {
+ ASSERT_TRUE(profiles::IsMultipleProfilesEnabled());
+ ASSERT_NO_FATAL_FAILURE(OpenProfileChooserView(browser()));
+
+ EXPECT_TRUE(signin_current_profile_link()->HasFocus());
+}
+
IN_PROC_BROWSER_TEST_F(ProfileChooserViewExtensionsTest, ContentAreaHasFocus) {
ASSERT_TRUE(profiles::IsMultipleProfilesEnabled());
« no previous file with comments | « chrome/browser/ui/views/profiles/profile_chooser_view.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698