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

Unified Diff: chrome/browser/ui/passwords/manage_passwords_test.h

Issue 444603003: Hook up the Mac password bubble to the browser and add browser tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix views build Created 6 years, 4 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/passwords/manage_passwords_test.h
diff --git a/chrome/browser/ui/views/passwords/manage_passwords_view_test.h b/chrome/browser/ui/passwords/manage_passwords_test.h
similarity index 79%
rename from chrome/browser/ui/views/passwords/manage_passwords_view_test.h
rename to chrome/browser/ui/passwords/manage_passwords_test.h
index 518674bcb6aa278d447b0d3ded4f7991a387f6c7..b04fca679bbe6fa327215f726f0cd278a50ba4dc 100644
--- a/chrome/browser/ui/views/passwords/manage_passwords_view_test.h
+++ b/chrome/browser/ui/passwords/manage_passwords_test.h
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_BROWSER_UI_VIEWS_PASSWORDS_MANAGE_PASSWORDS_VIEW_TEST_H_
-#define CHROME_BROWSER_UI_VIEWS_PASSWORDS_MANAGE_PASSWORDS_VIEW_TEST_H_
+#ifndef CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_TEST_H_
+#define CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_TEST_H_
#include "base/metrics/histogram_samples.h"
#include "base/test/histogram_tester.h"
@@ -12,14 +12,14 @@
#include "testing/gtest/include/gtest/gtest.h"
class ManagePasswordsUIControllerMock;
-class ManagePasswordsIconView;
+class ManagePasswordsIcon;
// Test class for the various password management view bits and pieces. Sets
// up a ManagePasswordsUIControllerMock, and provides some helper methods
// to poke at the bubble, icon, and controller's state.
-class ManagePasswordsViewTest : public InProcessBrowserTest {
+class ManagePasswordsTest : public InProcessBrowserTest {
public:
- ManagePasswordsViewTest() {}
+ ManagePasswordsTest() {}
// InProcessBrowserTest:
virtual void SetUpOnMainThread() OVERRIDE;
@@ -28,7 +28,7 @@ class ManagePasswordsViewTest : public InProcessBrowserTest {
ManagePasswordsUIControllerMock* controller();
// Get the icon view for the current WebContents.
- ManagePasswordsIconView* view();
+ virtual ManagePasswordsIcon* view() = 0;
// Execute the browser command to open the manage passwords bubble.
void ExecuteManagePasswordsCommand();
@@ -54,7 +54,7 @@ class ManagePasswordsViewTest : public InProcessBrowserTest {
autofill::PasswordForm test_form_;
base::HistogramTester histogram_tester_;
- DISALLOW_COPY_AND_ASSIGN(ManagePasswordsViewTest);
+ DISALLOW_COPY_AND_ASSIGN(ManagePasswordsTest);
};
-#endif // CHROME_BROWSER_UI_VIEWS_PASSWORDS_MANAGE_PASSWORDS_VIEW_TEST_H_
+#endif // CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_TEST_H_

Powered by Google App Engine
This is Rietveld 408576698