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

Unified Diff: chrome/browser/password_manager/password_manager_browsertest.cc

Issue 19705013: [password autofill] Remove references to PasswordForm from RenderViewImpl (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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/password_manager/password_manager_browsertest.cc
diff --git a/chrome/browser/password_manager/password_manager_browsertest.cc b/chrome/browser/password_manager/password_manager_browsertest.cc
index 02d2cea3b04f6802743c0e78580db11a2829d867..9d1e663e1d960b4cb013cd6786961d39bfabd5ea 100644
--- a/chrome/browser/password_manager/password_manager_browsertest.cc
+++ b/chrome/browser/password_manager/password_manager_browsertest.cc
@@ -112,8 +112,109 @@ class PasswordManagerBrowserTest : public InProcessBrowserTest {
DISALLOW_COPY_AND_ASSIGN(PasswordManagerBrowserTest);
};
-
// Actual tests ---------------------------------------------------------------
+IN_PROC_BROWSER_TEST_F(PasswordManagerBrowserTest, PromptForNormalSubmit) {
+ ASSERT_TRUE(test_server()->Start());
+
+ GURL url = test_server()->GetURL("files/password/password_form.html");
+ ui_test_utils::NavigateToURL(browser(), url);
+
+ // Fill a form and submit through a <input type="submit"> button. Nothing
+ // special.
+ NavigationObserver observer(WebContents());
+ std::string fill_and_submit =
+ "document.getElementById('username_field').value = 'temp';"
+ "document.getElementById('password_field').value = 'random';"
+ "document.getElementById('input_submit_button').click()";
+ ASSERT_TRUE(content::ExecuteScript(RenderViewHost(), fill_and_submit));
+ observer.Wait();
+ EXPECT_TRUE(observer.infobar_shown());
+}
+
+IN_PROC_BROWSER_TEST_F(PasswordManagerBrowserTest,
+ PromptForSubmitUsingJavaScript) {
+ ASSERT_TRUE(test_server()->Start());
+
+ GURL url = test_server()->GetURL("files/password/password_form.html");
+ ui_test_utils::NavigateToURL(browser(), url);
+
+ // Fill a form and submit using <button> that calls submit() on the form.
+ // This should work regardless of the type of element, as long as submit() is
+ // called.
+ NavigationObserver observer(WebContents());
+ std::string fill_and_submit =
+ "document.getElementById('username_field').value = 'temp';"
+ "document.getElementById('password_field').value = 'random';"
+ "document.getElementById('submit_button').click()";
+ ASSERT_TRUE(content::ExecuteScript(RenderViewHost(), fill_and_submit));
+ observer.Wait();
+ EXPECT_TRUE(observer.infobar_shown());
+}
+
+IN_PROC_BROWSER_TEST_F(PasswordManagerBrowserTest, NoPromptForNavigation) {
+ ASSERT_TRUE(test_server()->Start());
+
+ GURL url = test_server()->GetURL("files/password/password_form.html");
+ ui_test_utils::NavigateToURL(browser(), url);
+
+ // Don't fill the password form, just navigate away. Shouldn't prompt.
+ NavigationObserver observer(WebContents());
+ ASSERT_TRUE(content::ExecuteScript(RenderViewHost(),
+ "window.location.href = 'done.html';"));
+ observer.Wait();
+ EXPECT_FALSE(observer.infobar_shown());
+}
+
+IN_PROC_BROWSER_TEST_F(PasswordManagerBrowserTest,
+ NoPromptForSubFrameNavigation) {
+ ASSERT_TRUE(test_server()->Start());
+
+ GURL url = test_server()->GetURL("files/password/multi_frames.html");
+ ui_test_utils::NavigateToURL(browser(), url);
+
+ // If you are filling out a password form in one frame and a different frame
+ // navigates, this should not trigger the infobar.
+ NavigationObserver observer(WebContents());
+ std::string fill =
+ "var first_frame = document.getElementById('first_frame');"
+ "var frame_doc = first_frame.contentDocument;"
+ "frame_doc.getElementById('username_field').value = 'temp';"
+ "frame_doc.getElementById('password_field').value = 'random';";
+ std::string navigate_frame =
+ "var second_iframe = document.getElementById('second_frame');"
+ "second_iframe.contentWindow.location.href = 'done.html';";
+
+ ASSERT_TRUE(content::ExecuteScript(RenderViewHost(), fill));
+ ASSERT_TRUE(content::ExecuteScript(RenderViewHost(), navigate_frame));
+ observer.Wait();
+ EXPECT_FALSE(observer.infobar_shown());
+}
+
+IN_PROC_BROWSER_TEST_F(PasswordManagerBrowserTest,
+ PromptAfterSubmitWithSubFrameNavigation) {
+ ASSERT_TRUE(test_server()->Start());
+
+ GURL url = test_server()->GetURL("files/password/multi_frames.html");
+ ui_test_utils::NavigateToURL(browser(), url);
+
+ // Make sure that we prompt to save password even if a sub-frame navigation
+ // happens first.
+ NavigationObserver observer(WebContents());
+ std::string navigate_frame =
+ "var second_iframe = document.getElementById('second_frame');"
+ "second_iframe.contentWindow.location.href = 'done.html';";
+ std::string fill_and_submit =
+ "var first_frame = document.getElementById('first_frame');"
+ "var frame_doc = first_frame.contentDocument;"
+ "frame_doc.getElementById('username_field').value = 'temp';"
+ "frame_doc.getElementById('password_field').value = 'random';"
+ "frame_doc.getElementById('input_submit_button').click();";
+
+ ASSERT_TRUE(content::ExecuteScript(RenderViewHost(), navigate_frame));
+ ASSERT_TRUE(content::ExecuteScript(RenderViewHost(), fill_and_submit));
+ observer.Wait();
+ EXPECT_TRUE(observer.infobar_shown());
+}
IN_PROC_BROWSER_TEST_F(PasswordManagerBrowserTest, PromptForXHRSubmit) {
#if defined(OS_WIN) && defined(USE_ASH)
« no previous file with comments | « chrome/browser/password_manager/password_manager.cc ('k') | chrome/browser/password_manager/password_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698