Commit 388542e1 authored by Vadym Doroshenko's avatar Vadym Doroshenko Committed by Commit Bot

Replacing DCHECK with return.

With CL https://chromium-review.googlesource.com/c/chromium/src/+/1530980
NewPasswordFormManager::PasswordNoLongerGenerated might be called when
NewPasswordFormManager is not in generating state.

Bug: 886583, 866444

Change-Id: If69df52c77c10d195fad8b992462595942d45c11
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1561125Reviewed-by: default avatarMaxim Kolosovskiy <kolos@chromium.org>
Commit-Queue: Vadym Doroshenko <dvadym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#649857}
parent 4809135f
...@@ -421,7 +421,9 @@ void NewPasswordFormManager::PresaveGeneratedPassword( ...@@ -421,7 +421,9 @@ void NewPasswordFormManager::PresaveGeneratedPassword(
} }
void NewPasswordFormManager::PasswordNoLongerGenerated() { void NewPasswordFormManager::PasswordNoLongerGenerated() {
DCHECK(HasGeneratedPassword()); if (!HasGeneratedPassword())
return;
form_saver_->RemovePresavedPassword(); form_saver_->RemovePresavedPassword();
generated_password_.clear(); generated_password_.clear();
votes_uploader_.set_has_generated_password(false); votes_uploader_.set_has_generated_password(false);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment