Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(493)

Unified Diff: chrome/browser/mac/install_from_dmg.mm

Issue 9235084: Add OSSTATUS_LOG API (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/mac/install_from_dmg.mm
===================================================================
--- chrome/browser/mac/install_from_dmg.mm (revision 119461)
+++ chrome/browser/mac/install_from_dmg.mm (working copy)
@@ -22,6 +22,7 @@
#include "base/file_path.h"
#include "base/logging.h"
#include "base/mac/bundle_locations.h"
+#include "base/mac/mac_logging.h"
#import "base/mac/mac_util.h"
#include "base/mac/scoped_cftyperef.h"
#include "base/mac/scoped_nsautorelease_pool.h"
@@ -308,7 +309,8 @@
NULL, // pipe
&exit_status);
if (status != errAuthorizationSuccess) {
- LOG(ERROR) << "AuthorizationExecuteWithPrivileges install: " << status;
+ OSSTATUS_LOG(ERROR, status)
+ << "AuthorizationExecuteWithPrivileges install";
return false;
}
} else {
@@ -659,7 +661,7 @@
&disk_image_path_in_trash_c,
kFSFileOperationDefaultOptions);
if (status != noErr) {
- LOG(ERROR) << "FSPathMoveObjectToTrashSync: " << status;
+ OSSTATUS_LOG(ERROR, status) << "FSPathMoveObjectToTrashSync";
return;
}
@@ -677,7 +679,7 @@
kFNDirectoryModifiedMessage,
kNilOptions);
if (status != noErr) {
- LOG(ERROR) << "FNNotifyByPath: " << status;
+ OSSTATUS_LOG(ERROR, status) << "FNNotifyByPath";
return;
}
}

Powered by Google App Engine
This is Rietveld 408576698