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

Issue 140034: Define the == operator for X509Certificate::Fingerprint.... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 10 months ago by wtc
Modified:
2 years, 11 months ago
Reviewers:
eroman
CC:
chromium-reviews_googlegroups.com, darin, willchan
Visibility:
Public.

Description

Define the == operator for X509Certificate::Fingerprint.

R=eroman
BUG=http://crbug.com/318
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=19029

Patch Set 1 #

Patch Set 2 : Upload before checkin #

Total comments: 1

Patch Set 3 : List operator== after data member #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -3 lines) Lint Patch
M net/base/x509_certificate.h View 1 2 2 chunks +6 lines, -0 lines 0 comments 0 errors Download
M net/http/http_network_transaction.cc View 1 1 chunk +1 line, -3 lines 0 comments 0 errors Download
Commit:

Messages

Total messages: 3
wtc
4 years, 10 months ago #1
eroman
lgtm
4 years, 10 months ago #2
wtc
4 years, 10 months ago #3
http://codereview.chromium.org/140034/diff/2001/2003
File net/base/x509_certificate.h (right):

http://codereview.chromium.org/140034/diff/2001/2003#newcode43
Line 43: bool operator==(const Fingerprint& other) const {
Eric, are we supposed to move this operator before the
data member?  Since this is a 'struct', it's not clear to
me if the "methods before data members" guideline in the
style guide applies...
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6