New issue
Advanced search Search tips

Issue 662485 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

PasswordSecurityLevelDowngradedFromHttpsIframe test flakes on Linux.

Project Member Reported by se...@chromium.org, Nov 4 2016

Issue description

Comment 1 by se...@chromium.org, Nov 4 2016

Summary: PasswordSecurityLevelDowngradedFromHttpsIframe test flakes on Linux. (was: PasswordSecurityLevelDowngradedFromIframe test flakes on Linux.)
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 4 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f8331eaf76e5d13a4d2c813614b4292c449e5615

commit f8331eaf76e5d13a4d2c813614b4292c449e5615
Author: sebsg <sebsg@chromium.org>
Date: Fri Nov 04 20:21:51 2016

Disable PasswordSecurityLevelDowngradedFromHttpsIframe on Linux.

BUG= 662485 

TBR=estark@chromium.org

Review-Url: https://codereview.chromium.org/2471623007
Cr-Commit-Position: refs/heads/master@{#429998}

[modify] https://crrev.com/f8331eaf76e5d13a4d2c813614b4292c449e5615/chrome/browser/ssl/chrome_security_state_model_client_browser_tests.cc

Components: UI>Browser>Omnibox>SecurityIndicators
Labels: Team-Security-UX Hotlist-HttpBad
Components: -UI>Browser>Omnibox>SecurityIndicators Internals>PageSecurityState
Project Member

Comment 5 by bugdroid1@chromium.org, Feb 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fc6825e49d0c5aa1a922cf3b260e67070835abfb

commit fc6825e49d0c5aa1a922cf3b260e67070835abfb
Author: estark <estark@chromium.org>
Date: Fri Feb 10 02:36:54 2017

Fix flaky password visibility browser test

We inject dummy scripts to wait until password visiblity notifications have been
sent from the renderer to the browser, but in the case of password fields in
OOPIFs, we need to inject dummy scripts into the OOPIFs, not just the main
frame.

BUG= 662485 

Review-Url: https://codereview.chromium.org/2686223002
Cr-Commit-Position: refs/heads/master@{#449529}

[modify] https://crrev.com/fc6825e49d0c5aa1a922cf3b260e67070835abfb/chrome/browser/ssl/security_state_tab_helper_browser_tests.cc

Comment 6 by est...@chromium.org, Feb 10 2017

Labels: M-58
Status: Fixed (was: Assigned)

Sign in to add a comment