Chromium Code Reviews| Index: chrome/browser/ui/views/passwords/manage_passwords_bubble_view_browsertest.cc |
| diff --git a/chrome/browser/ui/views/passwords/manage_passwords_bubble_view_browsertest.cc b/chrome/browser/ui/views/passwords/manage_passwords_bubble_view_browsertest.cc |
| index d8e550d12cc6cc2b8998f77fb142209e4bbfd86c..7058b77faa7a77d04d1f4b9113b275630667bef4 100644 |
| --- a/chrome/browser/ui/views/passwords/manage_passwords_bubble_view_browsertest.cc |
| +++ b/chrome/browser/ui/views/passwords/manage_passwords_bubble_view_browsertest.cc |
| @@ -7,6 +7,7 @@ |
| #include "base/metrics/histogram_samples.h" |
| #include "chrome/browser/ui/browser.h" |
| #include "chrome/browser/ui/passwords/manage_passwords_test.h" |
| +#include "chrome/browser/ui/passwords/manage_passwords_ui_controller.h" |
| #include "chrome/browser/ui/tabs/tab_strip_model.h" |
| #include "chrome/browser/ui/views/frame/browser_view.h" |
| #include "chrome/browser/ui/views/passwords/manage_passwords_bubble_view.h" |
| @@ -314,4 +315,17 @@ IN_PROC_BROWSER_TEST_F(ManagePasswordsBubbleViewTest, AutoSignin) { |
| SetupAutoSignin(local_credentials.Pass()); |
| EXPECT_TRUE(ManagePasswordsBubbleView::IsShowing()); |
| + ::testing::Mock::VerifyAndClearExpectations(&url_callback); |
| + |
| + ManagePasswordsBubbleView::CloseBubble(); |
| + EXPECT_FALSE(ManagePasswordsBubbleView::IsShowing()); |
| + content::RunAllPendingInMessageLoop(); |
| + |
| + // Open the bubble to manage accounts. |
| + EXPECT_EQ(password_manager::ui::MANAGE_STATE, GetController()->state()); |
| + EXPECT_CALL(url_callback, OnRequestDone(avatar_url)); |
|
Mike West
2015/03/18 08:33:51
This expectation is failing. Perhaps you need to p
vasilii
2015/03/18 09:25:09
No, we were opening the old UI without avatars.
|
| + ManagePasswordsBubbleView::ShowBubble( |
| + browser()->tab_strip_model()->GetActiveWebContents(), |
| + ManagePasswordsBubble::USER_ACTION); |
| + EXPECT_TRUE(ManagePasswordsBubbleView::IsShowing()); |
| } |