New issue
Advanced search Search tips

Issue 679992 link

Starred by 1 user
Status: Fixed
Owner:
sdy@chromium.org
Closed: Jan 2017
Cc:
ligim...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

[Missing Tests]: Work around an AppKit bug that can crash Chrome after using the client certificate prompt.

Project Member Reported by ranjitkan@chromium.org, Jan 11 2017 
rsesek@, Automated tests for the below commit have been missing and would be really great to add them ASAP since these issues are high priority regressions.

Commit: https://codereview.chromium.org/2532203005

Ref. Bug: https://bugs.chromium.org/p/chromium/issues/detail?id=653093

Thank you!

Comment 1 by rsesek@chromium.org, Jan 11 2017

Owner: sdy@chromium.org
Assigning to the right person.
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 12 2017 

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

commit 145b27b6e14ae6dd557c823e39680d62f16b1cf3
Author: sdy <sdy@chromium.org>
Date: Thu Jan 12 00:45:55 2017

Add a test for our workaround of an AppKit crash in SFChooseIdentityPanel.

BUG= 679992 , 653093 

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

[modify] https://crrev.com/145b27b6e14ae6dd557c823e39680d62f16b1cf3/chrome/browser/ui/cocoa/ssl_client_certificate_selector_cocoa_browsertest.mm

Comment 3 by ligim...@chromium.org, Jan 12 2017

Labels: Pri-2
***Bulk Edit***

Considering Missing tests are P2, but please prioritize and add them asap.

Comment 4 by sdy@chromium.org, Jan 12 2017

Status: Fixed (was: Assigned)

Comment 5 by ranjitkan@chromium.org, Aug 8 2017

Components: Tests>Missing
►

Sign in to add a comment