Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(12)

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

Created:
10 years, 6 months ago by davidben
Modified:
9 years, 6 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

Messages

Total messages: 4 (0 generated)
davidben
10 years, 6 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 ...
10 years, 6 months ago (2010-06-25 00:09:04 UTC) #2
davidben
Displaying the stringified error name (Fancy. I didn't know we had such a thing.) Could ...
10 years, 6 months ago (2010-06-25 01:09:22 UTC) #3
wtc
10 years, 6 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.

Powered by Google App Engine
This is Rietveld 408576698