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

Issue 2874002: Change the Windows CertDatabase behaviour to match Mac and NSS behaviour, whe... (Closed)

Created:
10 years, 6 months ago by rsleevi-old
Modified:
9 years, 7 months ago
Reviewers:
wtc, davidben
CC:
chromium-reviews, pam+watch_chromium.org, John Grabowski, cbentzel+watch_chromium.org, darin-cc_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Change Windows' application/x-x509-user-cert handling to allow any certificate that has a private key, regardless of where it was generated, rather than restricting it to keys generated via <keygen> in the current browsing session. BUG=148 TEST=None R=wtc Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=50695

Patch Set 1 #

Patch Set 2 : Actually delete keygen_handler this time #

Total comments: 3

Patch Set 3 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -319 lines) Patch
M net/base/cert_database_win.cc View 1 2 2 chunks +3 lines, -69 lines 0 comments Download
M net/base/keygen_handler.h View 1 2 2 chunks +0 lines, -52 lines 0 comments Download
D net/base/keygen_handler.cc View 1 chunk +0 lines, -36 lines 0 comments Download
M net/base/keygen_handler_mac.cc View 1 chunk +0 lines, -6 lines 0 comments Download
M net/base/keygen_handler_nss.cc View 1 chunk +0 lines, -5 lines 0 comments Download
M net/base/keygen_handler_unittest.cc View 2 chunks +0 lines, -42 lines 0 comments Download
M net/base/keygen_handler_win.cc View 4 chunks +0 lines, -95 lines 0 comments Download
M net/net.gyp View 1 chunk +0 lines, -1 line 0 comments Download
M net/third_party/mozilla_security_manager/nsKeygenHandler.cpp View 1 2 3 chunks +0 lines, -13 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
rsleevi-old
This is split from http://codereview.chromium.org/2668005, to make reviewing easier. There is a performance trade-off here, ...
10 years, 6 months ago (2010-06-19 06:36:07 UTC) #1
wtc
10 years, 6 months ago (2010-06-24 02:52:46 UTC) #2
LGTM!

David Benjamin will work on trying to not import unsolicited
application/x-x509-user-cert responses.  So he will be modifying this
code soon.

Please add
    R=wtc
    BUG=148
to the Description of the CL.

http://codereview.chromium.org/2874002/diff/14003/16001
File net/base/cert_database_win.cc (right):

http://codereview.chromium.org/2874002/diff/14003/16001#newcode13
net/base/cert_database_win.cc:13: #include "net/base/keygen_handler.h"
Remove "base/logging.h".
Remove "base/string_util.h".
Remove "net/base/keygen_handler.h".

http://codereview.chromium.org/2874002/diff/14003/16003
File net/base/keygen_handler.h (right):

http://codereview.chromium.org/2874002/diff/14003/16003#newcode8
net/base/keygen_handler.h:8: #include <map>
Remove <map>.

http://codereview.chromium.org/2874002/diff/14003/16009
File net/third_party/mozilla_security_manager/nsKeygenHandler.cpp (right):

http://codereview.chromium.org/2874002/diff/14003/16009#newcode53
net/third_party/mozilla_security_manager/nsKeygenHandler.cpp:53: #include
"net/base/keygen_handler.h"
Remove "net/base/keygen_handler.h".

Powered by Google App Engine
This is Rietveld 408576698