diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-24 23:08:04 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-24 23:08:04 +0000 |
| commit | c4aa8696d0be2c3f6d45478079db973c5808bbd9 (patch) | |
| tree | e24008c8ce3b80732dd6b4da33e0a7882f68a622 /chrome/browser/password_manager | |
| parent | 80da95a8e814a06177f37e7f41735752e6bf779a (diff) | |
| download | chromium_src-c4aa8696d0be2c3f6d45478079db973c5808bbd9.zip chromium_src-c4aa8696d0be2c3f6d45478079db973c5808bbd9.tar.gz chromium_src-c4aa8696d0be2c3f6d45478079db973c5808bbd9.tar.bz2 | |
Revert 67326 - Re-landing original change since it didn't seem to fix the failure.
Revert 67281 to fix failing FullLogin test - The earlier TabContentsWrapper changes had incomplete support for password manager.
This wires up Password Manager properly, making sure it observes all notifications that it needs to function.
http://crbug.com/63664
TEST=see bug
Review URL: http://codereview.chromium.org/5311006
TBR=ben@chromium.org
Review URL: http://codereview.chromium.org/5291004
TBR=andybons@chromium.org
Review URL: http://codereview.chromium.org/5347003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67337 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/password_manager')
| -rw-r--r-- | chrome/browser/password_manager/password_manager.cc | 7 | ||||
| -rw-r--r-- | chrome/browser/password_manager/password_manager.h | 3 |
2 files changed, 10 insertions, 0 deletions
diff --git a/chrome/browser/password_manager/password_manager.cc b/chrome/browser/password_manager/password_manager.cc index 3e7e5fd..b5762b8 100644 --- a/chrome/browser/password_manager/password_manager.cc +++ b/chrome/browser/password_manager/password_manager.cc @@ -151,6 +151,13 @@ void PasswordManager::DidStopLoading() { } } +void PasswordManager::DidNavigateAnyFramePostCommit( + const NavigationController::LoadCommittedDetails& details, + const ViewHostMsg_FrameNavigate_Params& params) { + if (params.password_form.origin.is_valid()) + ProvisionallySavePassword(params.password_form); +} + void PasswordManager::PasswordFormsFound( const std::vector<PasswordForm>& forms) { if (!delegate_->GetProfileForPasswordManager()) diff --git a/chrome/browser/password_manager/password_manager.h b/chrome/browser/password_manager/password_manager.h index 79ea5c9..0ebb451 100644 --- a/chrome/browser/password_manager/password_manager.h +++ b/chrome/browser/password_manager/password_manager.h @@ -50,6 +50,9 @@ class PasswordManager : public LoginModel, // WebNavigationObserver overrides. virtual void DidStopLoading(); + virtual void DidNavigateAnyFramePostCommit( + const NavigationController::LoadCommittedDetails& details, + const ViewHostMsg_FrameNavigate_Params& params); virtual void PasswordFormsFound( const std::vector<webkit_glue::PasswordForm>& forms); virtual void PasswordFormsVisible( |