Chromium Code Reviews| Index: chrome/browser/ui/views/passwords/manage_passwords_icon_view_interactive_uitest.cc |
| diff --git a/chrome/browser/ui/views/passwords/manage_passwords_icon_view_interactive_uitest.cc b/chrome/browser/ui/views/passwords/manage_passwords_icon_view_interactive_uitest.cc |
| index 6c360a491f94e387b80faafe3c08c1e5f2b4f077..52948d675de587cd9eca1c2cdf94fff61a10935a 100644 |
| --- a/chrome/browser/ui/views/passwords/manage_passwords_icon_view_interactive_uitest.cc |
| +++ b/chrome/browser/ui/views/passwords/manage_passwords_icon_view_interactive_uitest.cc |
| @@ -43,6 +43,10 @@ class ManagePasswordsIconViewTest : public ManagePasswordsTest { |
| return tooltip; |
| } |
| + const gfx::ImageSkia& GetImage() { |
|
msw
2016/01/28 23:27:42
nit: this change isn't necessary, afaict
varkha
2016/01/29 00:00:40
No, I need this or some other solution like making
msw
2016/01/29 00:02:01
Acknowledged.
|
| + return GetView()->GetImage(); |
| + } |
| + |
| private: |
| DISALLOW_COPY_AND_ASSIGN(ManagePasswordsIconViewTest); |
| }; |
| @@ -58,7 +62,7 @@ IN_PROC_BROWSER_TEST_F(ManagePasswordsIconViewTest, PendingState) { |
| EXPECT_TRUE(GetView()->visible()); |
| // No tooltip because the bubble is showing. |
| EXPECT_EQ(base::string16(), GetTooltipText()); |
| - const gfx::ImageSkia active_image = GetView()->GetImage(); |
| + const gfx::ImageSkia active_image = GetImage(); |
| } |
| IN_PROC_BROWSER_TEST_F(ManagePasswordsIconViewTest, ManageState) { |