Add OSSTATUS_LOG API

chrome/service/chrome_service_application_mac.mm

-// 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.
 
 #import "chrome/service/chrome_service_application_mac.h"
 
-#include "base/logging.h"
 #include "base/mac/foundation_util.h"
+#include "base/mac/mac_logging.h"
 #import "chrome/common/cloud_print/cloud_print_class_mac.h"
 #include "chrome/common/chrome_switches.h"
 
 @interface ServiceApplication ()
 - (void)setCloudPrintHandler;
 - (void)submitPrint:(NSAppleEventDescriptor*)event;
 @end
 
 @implementation ServiceApplication
 
@@ skipping 10 matching lines @@
 - (void)submitPrint:(NSAppleEventDescriptor*)event {
   std::string silent = std::string("--") + switches::kNoStartupWindow;
   // Set up flag so that it can be passed along with the Apple Event.
   CFStringRef silentLaunchFlag =
       CFStringCreateWithCString(NULL, silent.c_str(), kCFStringEncodingUTF8);
   CFStringRef flags[] = { silentLaunchFlag };
   // Argv array that will be passed.
   CFArrayRef passArgv =
       CFArrayCreate(NULL, (const void**) flags, 1, &kCFTypeArrayCallBacks);
   FSRef ref;
-  OSStatus status = noErr;
   CFURLRef* kDontWantURL = NULL;
   // Get Chrome's bundle ID.
   std::string bundleID =  base::mac::BaseBundleID();
   CFStringRef bundleIDCF =
       CFStringCreateWithCString(NULL, bundleID.c_str(), kCFStringEncodingUTF8);
   // Use Launch Services to locate Chrome using its bundleID.
-  status = LSFindApplicationForInfo(kLSUnknownCreator, bundleIDCF,
-                                    NULL, &ref, kDontWantURL);
+  OSStatus status = LSFindApplicationForInfo(kLSUnknownCreator, bundleIDCF,
+                                             NULL, &ref, kDontWantURL);
 
   if (status != noErr) {
-    LOG(ERROR) << "Failed to make path ref";
-    LOG(ERROR) << GetMacOSStatusErrorString(status);
-    LOG(ERROR) << GetMacOSStatusCommentString(status);

[Tristan Schmelcher 2, 2012/01/27 19:56:23]

Looks like this won't be logged in the new code. Is it not useful?

[Mark Mentovai, 2012/01/27 19:58:41]

Tristan Schmelcher 2 wrote:
It’s usually a sentence describing the error. It’s pretty verbose and not at all
tailored to the operation that was attempted. It’s usually just a copy of the
documentation string you’d get if you Googled the string returned by
GetMacOSStatusErrorString, so I left it out.

+    OSSTATUS_LOG(ERROR, status) << "Failed to make path ref";
     return;
   }
   // Actually create the Apple Event.
   NSAppleEventDescriptor* sendEvent =
       [NSAppleEventDescriptor
            appleEventWithEventClass:cloud_print::kAECloudPrintClass
                             eventID:cloud_print::kAECloudPrintClass
                    targetDescriptor:nil
                            returnID:kAutoGenerateReturnID
                       transactionID:kAnyTransactionID];
   // Pull the parameters out of AppleEvent sent to us and attach them
   // to our Apple Event.
   NSAppleEventDescriptor* parameters =
       [event paramDescriptorForKeyword:cloud_print::kAECloudPrintClass];
   [sendEvent setParamDescriptor:parameters
                      forKeyword:cloud_print::kAECloudPrintClass];
   LSApplicationParameters params = { 0,
                                      kLSLaunchDefaults,
                                      &ref,
                                      NULL,
                                      NULL,
                                      passArgv,
                                      NULL };
   AEDesc* initialEvent = const_cast<AEDesc*> ([sendEvent aeDesc]);
   params.initialEvent = static_cast<AppleEvent*> (initialEvent);
   // Send the Apple Event Using launch services, launching Chrome if necessary.
   status = LSOpenApplication(&params, NULL);
   if (status != noErr) {
-    LOG(ERROR) << "Unable to launch";
-    LOG(ERROR) << GetMacOSStatusErrorString(status);
-    LOG(ERROR) << GetMacOSStatusCommentString(status);
+    OSSTATUS_LOG(ERROR, status) << "Unable to launch";
   }
 }
 
 
 @end
 
 
 namespace chrome_service_application_mac {
 
 void RegisterServiceApp() {
   ServiceApplication* var =
       base::mac::ObjCCastStrict<ServiceApplication>(
           [ServiceApplication sharedApplication]);
   [var setCloudPrintHandler];
 }
 
 }  // namespace chrome_service_application_mac
-