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

Unified Diff: net/base/transport_security_state.cc

Issue 7972024: Update SHA1_LENGTH -> kSHA1Length to match previous change to SHA256_LENGTH. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/base/transport_security_state.cc
===================================================================
--- net/base/transport_security_state.cc (revision 102606)
+++ net/base/transport_security_state.cc (working copy)
@@ -11,11 +11,6 @@
#include <keyhi.h>
#include <pk11pub.h>
-// NSS leaks #defines from its headers which will upset base/sha1.h.
-#if defined(SHA1_LENGTH)
-#undef SHA1_LENGTH
-#endif
wtc 2011/09/24 01:39:41 Excellent!
-
#include "base/base64.h"
#include "base/json/json_reader.h"
#include "base/json/json_writer.h"
@@ -579,7 +574,7 @@
if (type_and_base64.find("sha1/") == 0 &&
base::Base64Decode(type_and_base64.substr(5, type_and_base64.size() - 5),
&hash_str) &&
- hash_str.size() == base::SHA1_LENGTH) {
+ hash_str.size() == base::kSHA1Length) {
SHA1Fingerprint hash;
memcpy(hash.data, hash_str.data(), sizeof(hash.data));
out->push_back(hash);

Powered by Google App Engine
This is Rietveld 408576698