diff options
| author | gcasto@chromium.org <gcasto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-04 18:06:39 +0000 |
|---|---|---|
| committer | gcasto@chromium.org <gcasto@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-04 18:06:39 +0000 |
| commit | 27d9110c3f42fddec09fc2621353a6757c1b31a7 (patch) | |
| tree | 680116ffc774f97384b37f6a7981d05b103f84bc /chrome/browser/autofill | |
| parent | c99441e77eb5e7752905ae61bb0ffd7ed06c4579 (diff) | |
| download | chromium_src-27d9110c3f42fddec09fc2621353a6757c1b31a7.zip chromium_src-27d9110c3f42fddec09fc2621353a6757c1b31a7.tar.gz chromium_src-27d9110c3f42fddec09fc2621353a6757c1b31a7.tar.bz2 | |
Fix crash where sync state is updated during tab destruction.
BUG=121895
TEST=None
Review URL: https://chromiumcodereview.appspot.com/9982009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130680 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/autofill')
| -rw-r--r-- | chrome/browser/autofill/autofill_manager.cc | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/chrome/browser/autofill/autofill_manager.cc b/chrome/browser/autofill/autofill_manager.cc index 7cfb8de..a85cd88 100644 --- a/chrome/browser/autofill/autofill_manager.cc +++ b/chrome/browser/autofill/autofill_manager.cc @@ -254,7 +254,11 @@ void AutofillManager::RenderViewCreated(content::RenderViewHost* host) { } void AutofillManager::OnStateChanged() { - UpdatePasswordSyncState(web_contents()->GetRenderViewHost()); + // It is possible for sync state to change during tab contents destruction. + // In this case, we don't need to update the renderer since it's going away. + if (web_contents() && web_contents()->GetRenderViewHost()) { + UpdatePasswordSyncState(web_contents()->GetRenderViewHost()); + } } void AutofillManager::DidNavigateMainFrame( |