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

Unified Diff: base/mac/authorization_util.mm

Issue 2394683002: Update comments for AuthorizationExecuteWithPrivileges. (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « no previous file | chrome/installer/mac/app/AuthorizedInstall.m » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/mac/authorization_util.mm
diff --git a/base/mac/authorization_util.mm b/base/mac/authorization_util.mm
index a459bbf8fe4e05e4ef2de2cb748dc965b9726980..a3bc4f95969189c23526f63f4be6e3c78378ee83 100644
--- a/base/mac/authorization_util.mm
+++ b/base/mac/authorization_util.mm
@@ -108,7 +108,8 @@ OSStatus ExecuteWithPrivilegesAndGetPID(AuthorizationRef authorization,
pipe_pointer = &local_pipe;
}
-// Deprecated in OS X 10.7. https://crbug.com/592663.
+// AuthorizationExecuteWithPrivileges is deprecated in macOS 10.7, but no good
+// replacement exists. https://crbug.com/593133.
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wdeprecated-declarations"
// AuthorizationExecuteWithPrivileges wants |char* const*| for |arguments|,
« no previous file with comments | « no previous file | chrome/installer/mac/app/AuthorizedInstall.m » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698