Chromium Code Reviews Chromium Code Reviews
Issue 488083002:
[Password Manager] Fix to recognise failed login attempt for sites where content server pushes new … (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@branch_autofill_todo_20140813| Index: chrome/test/data/password/done_and_separate_login_form.html |
| diff --git a/chrome/test/data/password/done_and_separate_login_form.html b/chrome/test/data/password/done_and_separate_login_form.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..88b57f120bd52e152ebc0b8c56aeb97963bffec6 |
| --- /dev/null |
| +++ b/chrome/test/data/password/done_and_separate_login_form.html |
| @@ -0,0 +1,24 @@ |
| +<html> |
| +<head> |
| +<script type="text/javascript"> |
| +window.onload = function() { |
| + $('submit_separate').action = getLocationWithHttpSchemeSecuritySwapped(); |
| +} |
| + |
| +function getLocationWithHttpSchemeSecuritySwapped() { |
| + var protocol = ("http:" == window.location.protocol) ? "https://" : "http://"; |
| + var url = protocol + window.location.pathname + "done_and_separate_login_form.html"; |
|
vabr (Chromium)
2014/08/27 11:51:47
You should not need to add the filename, it should
vabr (Chromium)
2014/08/27 11:51:47
Did you forget to add window.location.host?
Pritam Nikam
2014/08/30 10:16:20
Done.
Pritam Nikam
2014/08/30 10:16:20
Done.
|
| + return url; |
| +} |
| +</script> |
| +</head> |
| +<body> |
| +Navigation complete. Below is the different login form pushed by server but with same action URL. |
| +The URL schem may or may not be same. Moreover, browser shall not promp user to save incorrect password. |
| +<form method="POST" id="to_separate"> |
| + <input type="text" id="username_separate" name="username_separate"> |
| + <input type="password" id="password_separate" name="password_separate"> |
| + <input type="submit" id="submit_separate" name="submit_separate"> |
| +</form> |
| +</body> |
| +</html> |
