Chromium Code Reviews
Help | Chromium Project | Sign in
(26)

Issue 2859026: Display the error code when certificates fail to add (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 11 months ago by David Benjamin
Modified:
3 years, 11 months ago
Reviewers:
wtc
CC:
chromium-reviews, ben+cc_chromium.org
Visibility:
Public.

Description

Display the error code when certificates fail to add We really want to make error strings, but this is a start. BUG=43906 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52499

Patch Set 1 #

Total comments: 2

Patch Set 2 : Show the stringified error code #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -4 lines) Patch
M chrome/app/generated_resources.grd View 1 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/ssl/ssl_add_cert_handler.cc View 1 3 chunks +9 lines, -2 lines 0 comments Download
Commit: CQ not working?

Messages

Total messages: 4 (0 generated)
David Benjamin
4 years, 11 months ago (2010-06-24 21:02:31 UTC) #1
wtc
This looks good, but I suggest two changes for consistency with our current network error ...
4 years, 11 months ago (2010-06-25 00:09:04 UTC) #2
David Benjamin
Displaying the stringified error name (Fancy. I didn't know we had such a thing.) Could ...
4 years, 11 months ago (2010-06-25 01:09:22 UTC) #3
wtc
4 years, 11 months ago (2010-06-25 01:22:51 UTC) #4
LGTM.

These are not standard SSL error codes.  These are the error codes used
by our network stack.  The stringized error codes make it clear that these
are not HTTP status codes.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld ec887be