Add OSSTATUS_LOG API

chrome/browser/platform_util_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.
 
 #include "chrome/browser/platform_util.h"
 
 #include <Carbon/Carbon.h>
 #import <Cocoa/Cocoa.h>
 #include <CoreServices/CoreServices.h>
 
 #include "base/file_path.h"
 #include "base/logging.h"
+#include "base/mac/mac_logging.h"
 #include "base/mac/scoped_aedesc.h"
 #include "base/sys_string_conversions.h"
 #include "googleurl/src/gurl.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/l10n/l10n_util_mac.h"
 
 namespace platform_util {
 
 void ShowItemInFolder(const FilePath& full_path) {
@@ skipping 10 matching lines @@
 //     http://crbug.com/32921
 //  2. Silent no-op for unassociated file types: http://crbug.com/50263
 // Instead, an AppleEvent is constructed to tell the Finder to open the
 // document.
 void OpenItem(const FilePath& full_path) {
   DCHECK([NSThread isMainThread]);
   NSString* path_string = base::SysUTF8ToNSString(full_path.value());
   if (!path_string)
     return;
 
-  OSErr status;
-
   // Create the target of this AppleEvent, the Finder.
   base::mac::ScopedAEDesc<AEAddressDesc> address;
   const OSType finderCreatorCode = 'MACS';
-  status = AECreateDesc(typeApplSignature,  // type
-                        &finderCreatorCode,  // data
-                        sizeof(finderCreatorCode),  // dataSize
-                        address.OutPointer());  // result
+  OSErr status = AECreateDesc(typeApplSignature,  // type
+                              &finderCreatorCode,  // data
+                              sizeof(finderCreatorCode),  // dataSize
+                              address.OutPointer());  // result
   if (status != noErr) {
-    LOG(WARNING) << "Could not create OpenItem() AE target";
+    OSSTATUS_LOG(WARNING, status) << "Could not create OpenItem() AE target";
     return;
   }
 
   // Build the AppleEvent data structure that instructs Finder to open files.
   base::mac::ScopedAEDesc<AppleEvent> theEvent;
   status = AECreateAppleEvent(kCoreEventClass,  // theAEEventClass
                               kAEOpenDocuments,  // theAEEventID
                               address,  // target
                               kAutoGenerateReturnID,  // returnID
                               kAnyTransactionID,  // transactionID
                               theEvent.OutPointer());  // result
   if (status != noErr) {
-    LOG(WARNING) << "Could not create OpenItem() AE event";
+    OSSTATUS_LOG(WARNING, status) << "Could not create OpenItem() AE event";
     return;
   }
 
   // Create the list of files (only ever one) to open.
   base::mac::ScopedAEDesc<AEDescList> fileList;
   status = AECreateList(NULL,  // factoringPtr
                         0,  // factoredSize
                         false,  // isRecord
                         fileList.OutPointer());  // resultList
   if (status != noErr) {
-    LOG(WARNING) << "Could not create OpenItem() AE file list";
+    OSSTATUS_LOG(WARNING, status) << "Could not create OpenItem() AE file list";
     return;
   }
 
   // Add the single path to the file list.  C-style cast to avoid both a
   // static_cast and a const_cast to get across the toll-free bridge.
   CFURLRef pathURLRef = (CFURLRef)[NSURL fileURLWithPath:path_string];
   FSRef pathRef;
   if (CFURLGetFSRef(pathURLRef, &pathRef)) {
     status = AEPutPtr(fileList.OutPointer(),  // theAEDescList
                       0,  // index
                       typeFSRef,  // typeCode
                       &pathRef,  // dataPtr
                       sizeof(pathRef));  // dataSize
     if (status != noErr) {
-      LOG(WARNING) << "Could not add file path to AE list in OpenItem()";
+      OSSTATUS_LOG(WARNING, status)
+          << "Could not add file path to AE list in OpenItem()";
       return;
     }
   } else {
     LOG(WARNING) << "Could not get FSRef for path URL in OpenItem()";
     return;
   }
 
   // Attach the file list to the AppleEvent.
   status = AEPutParamDesc(theEvent.OutPointer(),  // theAppleEvent
                           keyDirectObject,  // theAEKeyword
                           fileList);  // theAEDesc
   if (status != noErr) {
-    LOG(WARNING) << "Could not put the AE file list the path in OpenItem()";
+    OSSTATUS_LOG(WARNING, status)
+        << "Could not put the AE file list the path in OpenItem()";
     return;
   }
 
   // Send the actual event.  Do not care about the reply.
   base::mac::ScopedAEDesc<AppleEvent> reply;
   status = AESend(theEvent,  // theAppleEvent
                   reply.OutPointer(),  // reply
                   kAENoReply + kAEAlwaysInteract,  // sendMode
                   kAENormalPriority,  // sendPriority
                   kAEDefaultTimeout,  // timeOutInTicks
                   NULL, // idleProc
                   NULL);  // filterProc
   if (status != noErr) {
-    LOG(WARNING) << "Could not send AE to Finder in OpenItem()";
+    OSSTATUS_LOG(WARNING, status)
+        << "Could not send AE to Finder in OpenItem()";
   }
 }
 
 void OpenExternal(const GURL& url) {
   DCHECK([NSThread isMainThread]);
   NSString* url_string = base::SysUTF8ToNSString(url.spec());
   NSURL* ns_url = [NSURL URLWithString:url_string];
   if (!ns_url || ![[NSWorkspace sharedWorkspace] openURL:ns_url])
     LOG(WARNING) << "NSWorkspace failed to open URL " << url;
 }
@@ skipping 16 matching lines @@
 
 bool IsVisible(gfx::NativeView view) {
   // A reasonable approximation of how you'd expect this to behave.
   return (view &&
           ![view isHiddenOrHasHiddenAncestor] &&
           [view window] &&
           [[view window] isVisible]);
 }
 
 }  // namespace platform_util