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

Unified Diff: chrome/browser/ui/views/passwords/manage_passwords_icon_view_interactive_uitest.cc

Issue 2818383004: Remove unnecessary virtual ManagePasswordsTest::view(). (Closed)
Patch Set: Created 3 years, 8 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
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 1295b741795361c2e7e54c568f8ca0a2b9b3347a..707d1b4830e5fdff85372459242e512ea939732a 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
@@ -22,16 +22,13 @@ class ManagePasswordsIconViewTest : public ManagePasswordsTest {
ManagePasswordsIconViewTest() {}
~ManagePasswordsIconViewTest() override {}
- ManagePasswordsIconView* view() override {
- BrowserView* browser_view = static_cast<BrowserView*>(browser()->window());
- return static_cast<ManagePasswordsIconViews*>(
- browser_view->toolbar()->location_bar()->manage_passwords_icon_view());
- }
-
password_manager::ui::State ViewState() { return GetView()->state_; }
ManagePasswordsIconViews* GetView() {
- return static_cast<ManagePasswordsIconViews*>(view());
+ return BrowserView::GetBrowserViewForBrowser(browser())
+ ->toolbar()
+ ->location_bar()
+ ->manage_passwords_icon_view();
}
base::string16 GetTooltipText() {

Powered by Google App Engine
This is Rietveld 408576698