Add OSSTATUS_LOG API

net/base/x509_cert_types_mac.cc

Index: net/base/x509_cert_types_mac.cc
===================================================================
--- net/base/x509_cert_types_mac.cc	(revision 119461)
+++ net/base/x509_cert_types_mac.cc	(working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 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.
 
@@ -10,6 +10,7 @@
 
 #include "base/logging.h"
 #include "base/i18n/icu_string_conversions.h"
+#include "base/mac/mac_logging.h"
 #include "base/utf_string_conversions.h"
 
 namespace net {
@@ -202,7 +203,7 @@
   OSStatus err = SecAsn1Decode(coder, ber_name_data, length, kNameTemplate,
                                &name);
   if (err) {
-    LOG(ERROR) << "SecAsn1Decode returned " << err << "; name=" << name;
+    OSSTATUS_LOG(ERROR, err) << "SecAsn1Decode name=" << name;

[wtc, 2012/01/28 00:06:26]

Nit: the message should say SecAsn1Decode failed.

How about
  OSSTATUS_LOG(ERROR, err) << "SecAsn1Decode failed; name=" << name;
?

Looking at this closely, I am not sure about the value of
printing the value of 'name' on failure.  'name' is a pointer
initialized to NULL, so if the function failed, 'name' is
probably still NULL.  So I think it is enough to just say
  OSSTATUS_LOG(ERROR, err) << "SecAsn1Decode failed";

     SecAsn1CoderRelease(coder);
     return false;
   }