Fixes the remaining net_unittests for OpenSSL, with the exceptions

net/base/x509_openssl_util.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/base/x509_openssl_util.h"
 
 #include "base/logging.h"
 #include "base/string_number_conversions.h"
 #include "base/string_piece.h"
 #include "base/string_util.h"
@@ skipping 131 matching lines @@
     } else if (c == '-') {
       found_hyphen = true;
     } else if (!IsAsciiDigit(c)) {
       LOG(WARNING) << "Invalid char " << c << " in hostname " << hostname;
       return false;
     }
     reference_name.push_back(c);
   }
   DCHECK(!reference_name.empty());
 
-  // TODO(joth): Add IP address support. See http://crbug.com/62973
   if (found_ip6_chars || !found_alpha) {
+    // For now we just do simple localhost IP address support, primarily as
+    // it's needed by the test server. TODO(joth): Replace this with full IP
+    // address support. See http://crbug.com/62973
+    if (hostname == "127.0.0.1" &&
+        std::find(cert_names.begin(), cert_names.end(), hostname)
+            != cert_names.end()) {

[wtc, 2010/12/01 22:50:05]

Nit: Chromium's style guide recommend folding long expressions
with operators at the end:
http://www.chromium.org/developers/coding-style#Code_Formatting

So I suggest putting != at the end of the previous line.

[joth, 2010/12/02 17:12:01]

Done.

+      DVLOG(1) << "Allowing localhost IP certificate: " << hostname;
+      return true;
+    }
     NOTIMPLEMENTED() << hostname;
     return false;
   }
 
   // |wildcard_domain| is the remainder of |host| after the leading host
   // component is stripped off, but includes the leading dot e.g.
   // "www.f.com" -> ".f.com".
   // If there is no meaningful domain part to |host| (e.g. it is an IP address
   // or contains no dots) then |wildcard_domain| will be empty.
   // We required at least 3 components (i.e. 2 dots) as a basic protection
@@ skipping 49 matching lines @@
   }
   DVLOG(1) << "Could not find any match for " << hostname
            << " (canonicalized as " << reference_name
            << ") in cert names " << JoinString(cert_names, '|');
   return false;
 }
 
 } // namespace x509_openssl_util
 
 } // namespace net