New issue
Advanced search Search tips

Issue 894378 link

Starred by 0 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Oct 11
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug

Blocking:
issue 774955



Sign in to add a comment

chegg.com triggers DCHECK on login form

Project Member Reported by vabr@chromium.org, Oct 11

Issue description

Chrome Version       : 71.0.3578.0
URLs (if applicable) : https://www.chegg.com/auth?action=signup&redirect=https%3A%2F%2Fwww.chegg.com%2F

What steps will reproduce the problem?
1. Fill in the username and password
2. Click Student.

What is the expected result?
No DCHECK.

What happens instead of that?
The following DCHECK is hit in PasswordManager::OnLoginSuccessful:
  // If the form is eligible only for saving fallback, it shouldn't go here.
  DCHECK(!submitted_manager->GetPendingCredentials().only_for_fallback_saving);

Notes:
* This only affects the old parser. With chrome://flags/#new-password-form-parsing-for-saving enabled, there is no problem.
* Clicking the Student button hides the password field but seems to keep it in DOM.
 
Cc: dvadym@chromium.org
Vadym: FYI
Given that this is fixed with the new parser, I suggest marking this as won't fix. If I'll have a deeper look to understand why the DCHECK was hit, I'll update this bug.
Status: WontFix (was: Available)
Spoke with Vadym, who also does not seem to think investigating this is a top priority given that we launch the new parser. Therefore closing.

Sign in to add a comment