Add OSSTATUS_LOG API

remoting/host/user_authenticator_mac.cc

-// 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.
 
 #include "remoting/host/user_authenticator.h"
 
 #include <Security/Security.h>
 
 #include <string>
 
 #include "base/basictypes.h"
-#include "base/logging.h"
+#include "base/mac/mac_logging.h"
 
 namespace remoting {
 
 namespace {
 
 class UserAuthenticatorMac : public UserAuthenticator {
  public:
   UserAuthenticatorMac() {}
   virtual ~UserAuthenticatorMac() {}
   virtual bool Authenticate(const std::string& username,
@@ skipping 40 matching lines @@
                                         kAuthorizationFlagExtendRights,
                                         NULL);
   switch (status) {
     case errAuthorizationSuccess:
       return true;
 
     case errAuthorizationDenied:
       return false;
 
     default:
-      LOG(ERROR) << "AuthorizationCreate returned " << status;
+      OSSTATUS_LOG(ERROR, status) << "AuthorizationCreate";

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

Nit: should we add "failed" to the error message?
Or do you think "failed" is ambiguous here?  It could be
misinterpreted to mean authorization denied.

[Wez, 2012/01/28 00:11:52]

I think there's enough of a distinction between failed and denied for this to
make sense.

       return false;
   }
 }
 
 }  // namespace
 
 // static
 UserAuthenticator* UserAuthenticator::Create() {
   return new UserAuthenticatorMac();
 }
 
 }  // namespace remoting